Properly handle maximum number of CSI params

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 CSI params #7

Closed Aaron1011 closed 7 years ago

Aaron1011 commented 7 years ago

This commit ensures that 'num_params' is never incremented beyond MAX_PARAMS. Additionally, an extra parameter is added to the corresponding OSC test (17 params or MAX_PARAMS + 1) to ensure that the proper behavior is tested.

This bug was found with afl.

jwilm commented 7 years ago

Thanks again @Aaron1011! Looks good :+1: