From patchwork Fri Sep 18 01:04:25 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Lance Wang X-Patchwork-Id: 22461 Return-Path: X-Original-To: patchwork@ffaux-bg.ffmpeg.org Delivered-To: patchwork@ffaux-bg.ffmpeg.org Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by ffaux.localdomain (Postfix) with ESMTP id 346AD44A9B0 for ; Fri, 18 Sep 2020 04:11:30 +0300 (EEST) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id F3D5168B8FD; Fri, 18 Sep 2020 04:11:29 +0300 (EEST) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from mail-oi1-f194.google.com (mail-oi1-f194.google.com [209.85.167.194]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id D78F368B7E1 for ; Fri, 18 Sep 2020 04:11:22 +0300 (EEST) Received: by mail-oi1-f194.google.com with SMTP id x69so4914553oia.8 for ; Thu, 17 Sep 2020 18:11:22 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=YVGaEYePPL6WpWtzB1EyLxdDE//4Eg1UmhqvYuMih6Y=; b=s5jXeEdVC3jKcGpYRRdqo/NRnos8evyARidSyU4sYhOJXrVOd46sdl8Mgj1NNTThgr dfOLGuN/F+z/6SOuWU7HQsc8eYb0K1XKXxXmZEHKPYk+DzhJX10xOtn/SpQkNehstgL5 WvRUsXPIXWgXVyefdeMWZbdZoHl7nZXvc4b9Z/Z1L3kV3gAz4pfRo5VT7C+f5BDTTY5h W57OC9jNEsKeNkOC24JsgG+OMtnWgG1X0DvVOUscVX3R9AFMD32cekgyqLQUdznU6W1K tYumOFRC78hbsPQ9QDA7erV1Y/9/3nBa1B9FawmOoeoYmhiRrXeu+I0eQoZ91z7mctXh jMIQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=YVGaEYePPL6WpWtzB1EyLxdDE//4Eg1UmhqvYuMih6Y=; b=Ccoxe35xaQ3SEM4/B6buOSanki4AQzh+BEZ0H/1+NV10K8h1sVQENZaCBEGQaYSe5b Y/W3k+kRcLGVRRJcREd+zZpZ1TpoUOre0hL+91G3ZJd3YWfskXmLJ5sSvVZVprNlbkYo vWMDxhuGbu7gntHy1qlOBRBsdJL0IXoHRuHpBVZGePHg+jzuw3oevJSkyDHkUzppT4ss ARkNVtTYqiJICDXMZeuc+5AvcdZyy9rommZ1sH2i2dponScCT1gX9ZwJQ+rf/xS+vC+5 UdlpEp5fuF12QRxM9JrQtdy1nbOysxT2DTdc8SKTFAL5yaJxyL8fb88b/c7nPst7thfC nfDQ== X-Gm-Message-State: AOAM533trG1ItJ0BcS5NEDgAQZgVxIluI7ZH1qqExBjjMHCmepN5Khyy lmNTInoyKsXbcrL9/wKgXYkXqzDNGQU= X-Google-Smtp-Source: ABdhPJwhOwIzcclU9zv9oLSyZWdpYF/qb5k2p7UyHaF5H/aWWX9Nsv03CoPqmeAQmG1aVPbqfsH9JQ== X-Received: by 2002:a17:90b:3351:: with SMTP id lm17mr10466963pjb.151.1600391090915; Thu, 17 Sep 2020 18:04:50 -0700 (PDT) Received: from vpn2.localdomain ([161.117.202.209]) by smtp.gmail.com with ESMTPSA id a3sm838576pfl.213.2020.09.17.18.04.49 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Thu, 17 Sep 2020 18:04:50 -0700 (PDT) From: lance.lmwang@gmail.com To: ffmpeg-devel@ffmpeg.org Date: Fri, 18 Sep 2020 09:04:25 +0800 Message-Id: <1600391066-15693-2-git-send-email-lance.lmwang@gmail.com> X-Mailer: git-send-email 1.8.3.1 In-Reply-To: <1600391066-15693-1-git-send-email-lance.lmwang@gmail.com> References: <1600391066-15693-1-git-send-email-lance.lmwang@gmail.com> Subject: [FFmpeg-devel] [PATCH 2/3] avdevice/lavfi: unref the frame on failure X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: FFmpeg development discussions and patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: FFmpeg development discussions and patches Cc: Limin Wang MIME-Version: 1.0 Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" From: Limin Wang Signed-off-by: Limin Wang --- libavdevice/lavfi.c | 19 ++++++++++++------- 1 file changed, 12 insertions(+), 7 deletions(-) diff --git a/libavdevice/lavfi.c b/libavdevice/lavfi.c index 268dc0d..a4b510f 100644 --- a/libavdevice/lavfi.c +++ b/libavdevice/lavfi.c @@ -432,7 +432,7 @@ static int lavfi_read_packet(AVFormatContext *avctx, AVPacket *pkt) if (st->codecpar->codec_type == AVMEDIA_TYPE_VIDEO) { size = av_image_get_buffer_size(frame->format, frame->width, frame->height, 1); if ((ret = av_new_packet(pkt, size)) < 0) - return ret; + goto fail; av_image_copy_to_buffer(pkt->data, size, (const uint8_t **)frame->data, frame->linesize, frame->format, frame->width, frame->height, 1); @@ -440,7 +440,7 @@ static int lavfi_read_packet(AVFormatContext *avctx, AVPacket *pkt) size = frame->nb_samples * av_get_bytes_per_sample(frame->format) * frame->channels; if ((ret = av_new_packet(pkt, size)) < 0) - return ret; + goto fail;; memcpy(pkt->data, frame->data[0], size); } @@ -449,18 +449,19 @@ static int lavfi_read_packet(AVFormatContext *avctx, AVPacket *pkt) int size; uint8_t *metadata = av_packet_pack_dictionary(frame_metadata, &size); - if (!metadata) - return AVERROR(ENOMEM); + if (!metadata) { + ret = AVERROR(ENOMEM); + goto fail; + } if ((ret = av_packet_add_side_data(pkt, AV_PKT_DATA_STRINGS_METADATA, metadata, size)) < 0) { av_freep(&metadata); - return ret; + goto fail;; } } if ((ret = create_subcc_packet(avctx, frame, min_pts_sink_idx)) < 0) { - av_frame_unref(frame); - return ret; + goto fail; } pkt->stream_index = stream_idx; @@ -468,6 +469,10 @@ static int lavfi_read_packet(AVFormatContext *avctx, AVPacket *pkt) pkt->pos = frame->pkt_pos; av_frame_unref(frame); return size; +fail: + av_frame_unref(frame); + return ret; + } #define OFFSET(x) offsetof(LavfiContext, x)