summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJames Almer <jamrial@gmail.com>2020-04-14 17:31:26 -0300
committerJames Almer <jamrial@gmail.com>2020-04-15 14:12:38 -0300
commite7eb379d98f6b8be9b19afd71fc5b3473eca4a47 (patch)
treee38b231f9de94fd2c6b5de203a45610e9a425152
parent18f5256c0dc6cfb41e649ab80b0577ba33852d76 (diff)
avformat/movenc: remove call to av_copy_packet_side_data() when concatenating eac3 syncframes
This generates a potential memory leak, and mixes side data from the last packet with other properties from the first. Keep all the properties from the first packet only in the output packet instead. Signed-off-by: James Almer <jamrial@gmail.com>
-rw-r--r--libavformat/movenc.c2
1 files changed, 0 insertions, 2 deletions
diff --git a/libavformat/movenc.c b/libavformat/movenc.c
index bc8d08044e..bf3e4fa2ce 100644
--- a/libavformat/movenc.c
+++ b/libavformat/movenc.c
@@ -520,8 +520,6 @@ concatenate:
memcpy(info->pkt.data + info->pkt.size - pkt->size, pkt->data, pkt->size);
info->num_blocks += num_blocks;
info->pkt.duration += pkt->duration;
- if ((ret = av_copy_packet_side_data(&info->pkt, pkt)) < 0)
- goto end;
if (info->num_blocks != 6)
goto end;
av_packet_unref(pkt);