News for hackers. Stuff that matters.
Re:OpenSSL added new C parser code [...] without doing any b... by rebelwebmaster on Wednesday November 02, 2022 @12:56PM (#33439036)
Isn't openssl included in the oss-fuzz project? If hanno caught it this quickly with his fuzzer, would seem to be surprising if they didn't also.
Re:OpenSSL added new C parser code [...] without doing any b... by nwellnhof on Wednesday November 02, 2022 @01:43PM (#33439931)
Just because a project uses oss-fuzz, you can't assume it has good fuzz coverage. In this case, they probably should have written a specialized fuzz target for the Punycode parser. Parsers like this are easy to fuzz and such bugs are typically caught very quickly, often in mere seconds. With a more general fuzz target, it can take much longer to come up with input that triggers the bug.
Re:OpenSSL added new C parser code [...] without doing any b... by kdbg on Wednesday November 02, 2022 @03:44PM (#33441863)
It is, it'll build a few fuzzers hitting different areas[0]. The important function in many of those `.c` files is `FuzzerTestOneInput` which is effectively the entrypoint for a single fuzz test.

Taking a look at x509.c[1] which I believe is the most likely to be able to reach the punnycode parser. (I am not at all familiar with the codebase). You can see that the OpenSSL fuzzer is basically doing a holistic fuzz (I assume the i2d* and d2i* functions exercise the parser), that is its just invoking key entrypoints that in theory can exercise all the rest of the functionality with the correct inputs.

Hanno's fuzzer on the other hand, is explicitly only testing the `ossl_punnycode_decode` function[3].

Given the breadth of the fuzzer, I think its very possible OSS-Fuzz just didn't hit it.

[0] https://github.com/openssl/openssl/blob/master/fuzz/

[1] https://github.com/openssl/openssl/blob/master/fuzz/x509.c

[2] https://twitter.com/hanno/status/1587775675397726209/photo/2

Re:OpenSSL added new C parser code [...] without doing any b... by kramerger on Wednesday November 02, 2022 @04:17PM (#33442366)
Given how much horse power and experience they have, this is very disappointing.
Re:OpenSSL added new C parser code [...] without doing any b... by kelnos on Wednesday November 02, 2022 @05:05PM (#33442900)
"They" who? Even since Heartbleed, the OpenSSL project is still woefully underfunded given its importance to... well, everything on the internet.
Re:OpenSSL added new C parser code [...] without doing any b... by kramerger on Thursday November 03, 2022 @08:57AM (#33450265)
I meant OSS-fuzz, i.e. Google & co