lzf: update pointer p after realloc
This fixes heap-use-after-free detected by AddressSanitizer.
Reviewed-by: Luca Barbato <lu_zero@gentoo.org>
Signed-off-by: Andreas Cadhalpun <Andreas.Cadhalpun@googlemail.com>
(cherry picked from commit bb6a7b6f75
)
Signed-off-by: Andreas Cadhalpun <Andreas.Cadhalpun@googlemail.com>
This commit is contained in:
@@ -53,6 +53,7 @@ int ff_lzf_uncompress(GetByteContext *gb, uint8_t **buf, int64_t *size)
|
|||||||
ret = av_reallocp(buf, *size);
|
ret = av_reallocp(buf, *size);
|
||||||
if (ret < 0)
|
if (ret < 0)
|
||||||
return ret;
|
return ret;
|
||||||
|
p = *buf + len;
|
||||||
}
|
}
|
||||||
|
|
||||||
bytestream2_get_buffer(gb, p, s);
|
bytestream2_get_buffer(gb, p, s);
|
||||||
@@ -75,6 +76,7 @@ int ff_lzf_uncompress(GetByteContext *gb, uint8_t **buf, int64_t *size)
|
|||||||
ret = av_reallocp(buf, *size);
|
ret = av_reallocp(buf, *size);
|
||||||
if (ret < 0)
|
if (ret < 0)
|
||||||
return ret;
|
return ret;
|
||||||
|
p = *buf + len;
|
||||||
}
|
}
|
||||||
|
|
||||||
av_memcpy_backptr(p, off, l);
|
av_memcpy_backptr(p, off, l);
|
||||||
|
Reference in New Issue
Block a user