oss-sec mailing list archives
Re: mp3gain: NULL pointer dereference in sync_buffer (mpglibDBL/interface.c)
From: Agostino Sarubbo <ago () gentoo org>
Date: Thu, 14 Sep 2017 13:12:20 +0200
On giovedì 14 settembre 2017 11:51:42 CEST Dr. Thomas Orgis wrote:
I disagree. I am considering cleaning up mp3gain and omitting nearly all of the vulnerabilities by removing the decoder fork. Reason: rgain does not do what mp3gain did. Mp3gain can directly modify the MPEG frames so that the gain is changed also for decoders that do not support the added metadata (it additionally stores metadata to be able to revert the changes). While I am not regularily using this myself, I do think that it's a nifty hack that should not disappear. Maybe it can re-enter distros if it does not rely on an outdated internal decoder … This is becoming a bit off-topic … but I just wanted to note that the bug reports do serve a purpose in alerting me to that other copy of mpg123 code in the wild. Alrighty then, Thomas
Hello Thomas, the suggestion of removal was because of the dead status of the upstream project. If there will be people that fix the issues, will be great. Feel free to update this thread when you have news about. -- Agostino Sarubbo Gentoo Linux Developer
Current thread:
- mp3gain: NULL pointer dereference in sync_buffer (mpglibDBL/interface.c) Agostino Sarubbo (Sep 14)
- Re: mp3gain: NULL pointer dereference in sync_buffer (mpglibDBL/interface.c) Simon McVittie (Sep 14)
- Re: mp3gain: NULL pointer dereference in sync_buffer (mpglibDBL/interface.c) Agostino Sarubbo (Sep 14)
- Re: mp3gain: NULL pointer dereference in sync_buffer (mpglibDBL/interface.c) Dr. Thomas Orgis (Sep 14)
- Re: mp3gain: NULL pointer dereference in sync_buffer (mpglibDBL/interface.c) Agostino Sarubbo (Sep 14)
- Re: mp3gain: NULL pointer dereference in sync_buffer (mpglibDBL/interface.c) Agostino Sarubbo (Sep 14)
- Re: mp3gain: NULL pointer dereference in sync_buffer (mpglibDBL/interface.c) Simon McVittie (Sep 14)
- Re: mp3gain: NULL pointer dereference in sync_buffer (mpglibDBL/interface.c) Dr. Thomas Orgis (Sep 14)