News for hackers. Stuff that matters.
Re:Xz/liblzma: Bash-stage Obfuscation Explained by jghn on Sunday March 31, 2024 @03:04AM (#39882088)
Not only were they a co-maintainer, but if you're relying on code review to ensure correctness and security, you've already lost the battle.

Code reviews are more about education and de-siloing.

Re:Xz/liblzma: Bash-stage Obfuscation Explained by ajross on Sunday March 31, 2024 @09:57AM (#39884306)
Yeah, no. Code review isn't going to catch all bugs, but it does catch a ton as long as it's done sincerely and well. You'd have an extremely hard time trying to sneak code with a syntax problem like this into Linux, for example. The community values and rewards nitpickery and fine-toothing, and for good reason.
Re:Xz/liblzma: Bash-stage Obfuscation Explained by barfbagginus on Sunday March 31, 2024 @12:56PM (#39885870)
Assume your co-contibutor was not always malicious. They passed all past vetting efforts. But their motives have changed due to a secret cause - they're being extorted by a criminal holding seriously damaging material over them and their family.

What other controls would you use to prevent them contributing malicious commits, besides closely reading your co-contributor's commits, and disallowing noisy commits that you don't fully comprehend and vouch for?

We assume that it'd be unethical to surveil the contributor well enough to detect the change in alliance. That would violate their privacy.

Is it reasonable to say, "game over, I lose" in that context? To that end, we might argue that an embedded mole will always think of ways to fool our review, so this kind of compromise is fatal.

But let's assume it's not game over. You have an advanced persistent threat, and you've got a chance to defeat them. What, besides reviewing the code, do you do?

Re:Xz/liblzma: Bash-stage Obfuscation Explained by malkia on Sunday March 31, 2024 @04:02PM (#39887548)
Corporative espionage comes to mind....
Re:Xz/liblzma: Bash-stage Obfuscation Explained by edflsafoiewq on Sunday March 31, 2024 @01:00PM (#39885893)
In open source, code review is absolutely about correctness and security.
Re:Xz/liblzma: Bash-stage Obfuscation Explained by ted_dunning on Monday April 01, 2024 @02:44AM (#39891459)
In some open source, this is true.

In an awful lot of open source, code review is a vanishingly rare commodity because there aren't enough committers left.