Properly handle maximum number of OSC parameters

alacritty / vte

Parser for virtual terminal emulators

https://docs.rs/vte/

Apache License 2.0

242 stars 56 forks source link

Properly handle maximum number of OSC parameters #6

Closed Aaron1011 closed 7 years ago

Aaron1011 commented 7 years ago

Previously, osc_num_params would be incremented when the final parameter finished parsing, even if it was already at MAX_PARAMS. This commit ensures that it is not incremented beyond MAX_PARAMS.

I found this bug with american fuzzy lop

jwilm commented 7 years ago

This looks great; thanks for the patch!