this post was submitted on 22 Jul 2024
255 points (96.7% liked)

Technology

59396 readers
2618 users here now

This is a most excellent place for technology news and articles.


Our Rules


  1. Follow the lemmy.world rules.
  2. Only tech related content.
  3. Be excellent to each another!
  4. Mod approved content bots can post up to 10 articles per day.
  5. Threads asking for personal tech support may be deleted.
  6. Politics threads may be removed.
  7. No memes allowed as posts, OK to post as comments.
  8. Only approved bots from the list below, to ask if your bot can be added please contact us.
  9. Check for duplicates before posting, duplicates may be removed

Approved Bots


founded 1 year ago
MODERATORS
you are viewing a single comment's thread
view the rest of the comments
[–] [email protected] 43 points 3 months ago (2 children)

The driver was signed, the issue was with a configuration file for that's not part of the driver.

[–] [email protected] 8 points 3 months ago* (last edited 3 months ago) (2 children)

A configuration file shouldn't crash the kernel. I don't understand how this solution could pass the certification. I don't know the criteria of course, but on the surface it sounds like Crowdstrike created a workaround, and Microsoft either missed or allowed it.

[–] [email protected] 6 points 3 months ago (1 children)

How would you prove that no input exists that could crash a piece of code? The potential search space is enormous. Microsoft can't prevent drivers from accepting external input, so there's always a risk that something could trigger an undetected error in the code. Microsoft certainly ought to be fuzz testing drivers it certifies but that will only catch low hanging fruit. Unless they can see the source code, it's hard to determine for sure that there are no memory safety bugs.

The driver developers are the ones with the source code and should have been using analysis tools to find these kinds of memory safety errors. Or they could have written it in a memory safe language like Rust.

[–] [email protected] 1 points 3 months ago (1 children)

You don't need to prove that no input can crash the code. "Exhaustive testing is not possible" is one of the core testing principles, ISTQB teaches that. As far as we know, the input was a file filled with zeroes, and not some subtle configuration or instruction. That can definitely be expected, tested, and handled.

[–] [email protected] 3 points 3 months ago (1 children)

As far as we know, the input was a file filled with zeroes

CrowdStrike have said that was not the problem:

This is not related to null bytes contained within Channel File 291 or any other Channel File.

That said, their preliminary incident review doesn't give us much to go on as to what was wrong with the file.

You're speculating that it was something easy to test for by a third party. It certainly could have been but I would hope it's a more subtle bug which, as you say, can't be exhaustively tested for. Source code analysis definitely would have surfaced this bug so either they didn't bother looking or didn't bother fixing it.

[–] [email protected] 2 points 3 months ago

You’re speculating that it was something easy to test for by a third party.

Based on the data that I have, which is of course very limited! I didn't know about the recent news regarding the null bytes, thank you for sharing this info.

[–] [email protected] 5 points 3 months ago

AFAIK, blue screen doesn't mean kernel crash. Hell, windows crashing isn't even rare.

Certification doesn't mean it has Microsoft seal of approval either, only that it comes from a certified and approved vendor, with some checks at best.

Config files are not part of the driver, ever. How do you think you can change the settings of you GPU without asking Microsoft?

But hey, if you are so willing to blame Microsoft for the one time it's not their fault, may I talk to you about our Lord Savior Linux? In my office we only knew because of the memes.

[–] mriguy 3 points 3 months ago

Maybe it should be. At least part of the package that’s signed.