Opened 12 months ago

Last modified 11 months ago

#55632 assigned defect

idutils @4.6_0: mkid always fails with "Illegal instruction: 4"

Reported by: jlargentaye (John de Largentaye) Owned by: yanjingfeng@…
Priority: Normal Milestone:
Component: ports Version:
Keywords: highsierra haspatch Cc: ryandesign (Ryan Schmidt)
Port: idutils

Description

On macOS 10.13, idutils compiles and installs succesfully, but the 'mkid' executable is completely broken, immediately failing like:

$ mkid
Illegal instruction: 4

Attachments (4)

idutils-install.log (156.4 KB) - added by jlargentaye (John de Largentaye) 12 months ago.
output of "port -v install idutils"
mkid_2018-01-04-105241_JohnLs-MacBook-Pro.crash (10.8 KB) - added by jlargentaye (John de Largentaye) 11 months ago.
DiagnosticReports crash file for mkid
Portfile-idutils.diff (725 bytes) - added by ryandesign (Ryan Schmidt) 11 months ago.
patch-lib-vasnprintf.c.diff (2.0 KB) - added by ryandesign (Ryan Schmidt) 11 months ago.

Download all attachments as: .zip

Change History (9)

Changed 12 months ago by jlargentaye (John de Largentaye)

Attachment: idutils-install.log added

output of "port -v install idutils"

comment:1 Changed 11 months ago by ryandesign (Ryan Schmidt)

Keywords: highsierra added
Owner: set to yanjingfeng@…
Status: newassigned

Please attach the crash log from ~/Library/Logs/DiagnosticReports

Changed 11 months ago by jlargentaye (John de Largentaye)

DiagnosticReports crash file for mkid

comment:2 Changed 11 months ago by ryandesign (Ryan Schmidt)

Aha:

%n used in a non-immutable format string

Yes, that is a new security measure in High Sierra that deliberately causes programs to crash if they use this potentially insecure code.

The code in question is part of vasnprintf, which is part of gnulib. The problem and the fix are discussed here: https://lists.gnu.org/archive/html/bug-gnulib/2017-07/msg00056.html

The developers of idutils will need to update to a newer version of gnulib—and it looks like they just did so 75 minutes ago: http://git.savannah.gnu.org/cgit/idutils.git/commit/?id=9a4ea9a365589ba52b4b020800e7b9718697989e

Now they just need to release a new version of idutils containing that fix.

Changed 11 months ago by ryandesign (Ryan Schmidt)

Attachment: Portfile-idutils.diff added

Changed 11 months ago by ryandesign (Ryan Schmidt)

Attachment: patch-lib-vasnprintf.c.diff added

comment:4 Changed 11 months ago by ryandesign (Ryan Schmidt)

Cc: ryandesign added

comment:5 Changed 11 months ago by jlargentaye (John de Largentaye)

Thanks!

Note: See TracTickets for help on using tickets.