On 22.08.2015 18:58, wm4 wrote: > On Sat, 8 Aug 2015 13:32:22 +0200 > Andreas Cadhalpun <andreas.cadhal...@googlemail.com> wrote: > >> Signed-off-by: Andreas Cadhalpun <andreas.cadhal...@googlemail.com> >> --- >> libavdevice/openal-dec.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/libavdevice/openal-dec.c b/libavdevice/openal-dec.c >> index b552158..cab1eb6 100644 >> --- a/libavdevice/openal-dec.c >> +++ b/libavdevice/openal-dec.c >> @@ -204,7 +204,7 @@ static int read_packet(AVFormatContext* ctx, AVPacket >> *pkt) >> fail: >> /* Handle failure */ >> if (pkt->data) >> - av_destruct_packet(pkt); >> + av_free_packet(pkt); >> if (error_msg) >> av_log(ctx, AV_LOG_ERROR, "Error: %s\n", error_msg); >> return error; > > LGTM
Pushed. > (Actually the old code looks buggy... can't use av_destruct_packet() on > a packet allocated with av_new_packet().) Yes, that looked fishy. Best regards, Andreas _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
