Opened 6 years ago
Last modified 5 years ago
#55632 assigned defect
idutils @4.6_0: mkid always fails with "Illegal instruction: 4"
| Reported by: | jlargentaye (John de Largentaye) | Owned by: | |
|---|---|---|---|
| Priority: | Normal | Milestone: | |
| Component: | ports | Version: | |
| Keywords: | highsierra haspatch | Cc: | ryandesign (Ryan Carsten 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)
Change History (10)
Changed 6 years ago by jlargentaye (John de Largentaye)
| Attachment: | idutils-install.log added |
|---|
comment:1 Changed 6 years ago by ryandesign (Ryan Carsten Schmidt)
| Keywords: | highsierra added |
|---|---|
| Owner: | set to yanjingfeng@… |
| Status: | new → assigned |
Please attach the crash log from ~/Library/Logs/DiagnosticReports
Changed 6 years ago by jlargentaye (John de Largentaye)
| Attachment: | mkid_2018-01-04-105241_JohnLs-MacBook-Pro.crash added |
|---|
DiagnosticReports crash file for mkid
comment:2 follow-up: 3 Changed 6 years ago by ryandesign (Ryan Carsten 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.
comment:3 Changed 6 years ago by ryandesign (Ryan Carsten Schmidt)
| Keywords: | haspatch added |
|---|
Replying to ryandesign:
The problem and the fix are discussed here: https://lists.gnu.org/archive/html/bug-gnulib/2017-07/msg00056.html
Specifically
Changed 6 years ago by ryandesign (Ryan Carsten Schmidt)
| Attachment: | Portfile-idutils.diff added |
|---|
Changed 6 years ago by ryandesign (Ryan Carsten Schmidt)
| Attachment: | patch-lib-vasnprintf.c.diff added |
|---|
comment:4 Changed 6 years ago by ryandesign (Ryan Carsten Schmidt)
| Cc: | ryandesign added |
|---|
comment:6 Changed 5 years ago by kurthindenburg (Kurt Hindenburg)
| Owner: | yanjingfeng@… deleted |
|---|
output of "port -v install idutils"