Yeah, my read was that they changed an interface to include an optional paramete... | Hacker News

Hacker Newsnew | past | comments | ask | show | jobs | submit

		cptskippy on Aug 7, 2024 \| parent \| context \| favorite \| on: CrowdStrike Official RCA is now out [pdf] Yeah, my read was that they changed an interface to include an optional parameter but never actually tested the underlying code by providing said optional parameter. The bug in clients (sensors) wasn't due to regex, the regex was in their integration unit testing which also had a bug and was never supplying the 21st parameter to the client code.

flamedoge on Aug 7, 2024 [–]

regex isn't probably a good thing in a kernel boot code considering it's NP hard

cptskippy on Aug 8, 2024 | [–]

That's true statement but what does it have to do with the RCA? From what I read it appears the regex was in the integration tests for the template.

Guidelines | FAQ | Lists | API | Security | Legal | Apply to YC | Contact