On Sat, 7 Sep 2013, Luca Barbato wrote:
This partially reverts commit b634b36fcebfe16b837b6c4044f5d5cb99a75040.
---
My fault for not catching it before, Diego's fault for not fixing it timely.
To make it clear: glibc, musl, newlib, and everything else excluded openbsd
s/excluded/except/
and windows libc can CRASH if you pass to realloc a pointer provided by
memalign or posix_memalign.
"pass a pointer provided by ... to realloc"
The fact it does not happen in our codebase is because currently we always
"because we currently"
pass to realloc a pointer created by realloc.
"only pass pointers created by realloc to realloc", or perhaps "don't pass
any pointers created by av_malloc to av_realloc".
Please include this part explanation in the commit message, not only as an
annotation. Otherwise people will still stay confused and might try to
"fix" it again.
Downstreams were mislead by the previous wrong documentation already.
libavutil/mem.h | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
The patch itself is ok with me.
// Martin
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
