search

sdroege / rtsp-types

RTSP (RFC 7826) types and parsers/serializers
MIT License
26 stars 13 forks source link

don't panic on invalid Content-Length #12

Closed scottlamb closed 3 years ago

scottlamb commented 3 years ago

I feel like I'm fighting nom here, so I'm curious to see if you have a nicer fix. But at least it works!

sdroege commented 3 years ago

I feel like I'm fighting nom here, so I'm curious to see if you have a nicer fix. But at least it works!

This seems correct. Because getting the header does not really use nom, you can't make use of built-in mechanisms like map_res but have to handle the error yourself.

sdroege commented 3 years ago

Thanks for the fix and the fuzzing setup!