diff options
| author | Andreas Rheinhardt <andreas.rheinhardt@gmail.com> | 2020-01-24 23:48:28 +0100 |
|---|---|---|
| committer | James Almer <jamrial@gmail.com> | 2020-01-26 12:41:32 -0300 |
| commit | 2899995a6fdd4a4f7365333d951f86cc0da8eec5 (patch) | |
| tree | 2744fb5a899687369ef16343c180c577a715f142 /libavformat/av1.c | |
| parent | d4bbc0db011151191ebc5e14af4be28127715cbd (diff) | |
| download | ffmpeg-2899995a6fdd4a4f7365333d951f86cc0da8eec5.tar.gz | |
avformat/av1, avc, hevc: Remove av_freep()
ff_av1_filter_obus_buf() and ff_avc_parse_nal_units_buf() both have a
pointer-to-pointer parameter which they use to pass a newly allocated
buffer to the caller. And both functions freed what this pointer points to
before overwriting it. But no caller of these functions used this feature,
but some had to initialize the pointer just because of this. So remove
it and update the documentation of ff_av1_filter_obus_buf() wrt this fact.
ff_hevc_annexb2mp4_buf in contrast did not free the pointer. This has been
documented, too.
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
Signed-off-by: James Almer <jamrial@gmail.com>
Diffstat (limited to 'libavformat/av1.c')
| -rw-r--r-- | libavformat/av1.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/libavformat/av1.c b/libavformat/av1.c index 876fd92733..709484b361 100644 --- a/libavformat/av1.c +++ b/libavformat/av1.c @@ -82,7 +82,6 @@ int ff_av1_filter_obus_buf(const uint8_t *in, uint8_t **out, int *size) memset(buf + len, 0, AV_INPUT_BUFFER_PADDING_SIZE); - av_freep(out); *out = buf; *size = len; |