On 19/10/15 13:31, wm4 wrote:
On Sun, 18 Oct 2015 17:59:43 +0200 Luca Barbato <[email protected]> wrote:--- avprobe.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/avprobe.c b/avprobe.c index a83fa68..2ab8ce0 100644 --- a/avprobe.c +++ b/avprobe.c @@ -589,8 +589,10 @@ static void show_packets(AVFormatContext *fmt_ctx) av_init_packet(&pkt); probe_array_header("packets", 0); - while (!av_read_frame(fmt_ctx, &pkt)) + while (!av_read_frame(fmt_ctx, &pkt)) { show_packet(fmt_ctx, &pkt); + av_packet_unref(&pkt); + } probe_array_footer("packets", 0); }So, was this always leaking, or is it preparation or something? Is it a bug fix or not? Surely the commit message field can be put to very good use here.
It is a memory leak, I guess I should update the message and candidate it to stable.
lu _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
