4 ms·
Xz.git: Fix Linux Landlock Feature Test in Autotools
- jamespwilliams 3y agoCan you spot the intentional bug in this commit?
- JSDevOps 3y agoIt was checking if the C compiled to detect the landlock feature, and there was a single character in the middle of the code that made it always fail to compile and therefore silently leave the feature disabled.
- michelangelo 3y agoThis is just spectacular. Evil, of course, but spectacular nevertheless. + #include <sys/prctl.h> +. <<<<<<<<<<<<<<<<<<<<<<<< THIS IS IT + void my_sandbox(void) + { +
- fargle 3y agowhy bother with a . instead of a   or some other unicode hackdodgery?
- olliej 3y agoA few reasons have been previously discussed: plenty of diff viewers highlight non-standard characters being a big one, the other bit being if someone does see this error it seems entirely plausible as a mistake. Choosing something atypical in a way that is clearly trying to be invisible rather than being an accident will draw a lot more attention than “oops I must have typo’d it”
- atombender 3y agoThread from yesterday (221 comments): https://news.ycombinator.com/item?id=39874404 https://news.ycombinator.com/item?id=39874404