filter_audio icon indicating copy to clipboard operation
filter_audio copied to clipboard

mem overflow bug

Open zoff99 opened this issue 3 years ago • 2 comments

https://github.com/irungentoo/filter_audio/blob/ada2f4fdc04940cdeee47caffe43add4fa017096/filter_audio.c#L320-L321

memset tries to write twice the bytes of the actual buffer size

A/libc: FORTIFY: memset: prevented 640-byte write into 320-byte buffer

should say:

memset(temp, 0, nsx_samples*sizeof(int16_t)); 

thanks to @kaefert for helping me find this

zoff99 avatar Dec 03 '20 07:12 zoff99

https://github.com/isotoxin/filter_audio/blob/37e81bcc934d3556ca52e8dfbd0a2396c868a684/filter_audio.c#L331

fixed here

https://github.com/isotoxin/filter_audio/commit/4adf0d2c175040c86f5a757bcffd190db224438d

zoff99 avatar Dec 03 '20 07:12 zoff99

Will this get merged?

ghost avatar Dec 20 '20 13:12 ghost