jgaeddert / liquid-dsp

digital signal processing library for software-defined radios
http://liquidsdr.org
MIT License
1.89k stars 443 forks source link

Possible NULL pointer dereference #166

Closed ashley-b closed 5 years ago

ashley-b commented 5 years ago

The check for _q == NULL is done after _q->M

https://github.com/jgaeddert/liquid-dsp/blob/b1f8da417960198af87b244337f0dc06a2df65db/src/modem/src/modem_common.c#L370-L376

jgaeddert commented 5 years ago

Good catch. I meant to check that _q->symbol_map wasn't NULL, not the main object.

Fixed in be24d2b821426faa942155e360cea44b34a73297