10831 loader emulator ls command fails to detect file types

Review Request #1774 — Created April 19, 2019 and submitted May 10, 2019, 4:48 p.m.

Description

10831 loader emulator ls command fails to detect file types

Testing Done

Issues

Description	From	Last Updated
asprintf() will allocate space, so is this malloc() needed?	yyang	May 7, 2019, 2:16 p.m.

Ship it!

usr/src/common/ficl/emu/loader_emu.c (Diff revision 1)

Show all issues

asprintf() will allocate space, so is this malloc() needed?

hm, I am pretty sure I did nuke it, but here it is..:)thanks.

-	63f177d2bcf18801f0e278fecc4e68d5c50939f7
+	9a023c2ec268877df88a942de1ce2884793d7f8b

usr/src/common/ficl/emu/loader_emu.c (Diff revision 2)

Ideally the code should also deal with the case of buf == NULL, but I think it's out of scope of this fix.

Ship it!

Ship it!

Ship it!

-	9a023c2ec268877df88a942de1ce2884793d7f8b
+	af1a324ac28c267c1882eb849e7286160d953cb4

You have a pending review.