From patchwork Tue Apr 9 13:05:09 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: James Almer X-Patchwork-Id: 47983 Delivered-To: ffmpegpatchwork2@gmail.com Received: by 2002:a05:6a20:9c8d:b0:1a7:a0dc:8de5 with SMTP id mj13csp337596pzb; Tue, 9 Apr 2024 06:06:03 -0700 (PDT) X-Forwarded-Encrypted: i=2; AJvYcCV5WcS76gOVL7WB6LMqhGRZjBTGva4DBWkH4onsShZWIq4OhqEZZofKOcieFB2xajDtudTtdVOLjcwXuLTeyIGTJJaWd/xK29g0LQ== X-Google-Smtp-Source: AGHT+IGPK5EdsZW2Po3DApCo0KUvZ87iOXvvTVB0MMGmkjGXPfXta+mnNlNzfzhYMP3MN6AiuJ23 X-Received: by 2002:a05:6512:3c8e:b0:514:88df:88b9 with SMTP id h14-20020a0565123c8e00b0051488df88b9mr12156091lfv.45.1712667963504; Tue, 09 Apr 2024 06:06:03 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1712667963; cv=none; d=google.com; s=arc-20160816; b=QfUYH7oNbvhU+2vTxYou3gy4nd/iIUMN1KgwzNuZ2HZT0OwgcELYAxQgxS7AR0Da28 PDnfAZbLvP1dUwW7Ozd9opUZjRiy0QeklHkH4OmBYjslhXEGAujVtj8yxnXcP1hjgO3z zpGLZF9BYs0jrSzmePdDVTCl1gtfY7KCUklfy200/b0Bou1U76BFpj3vRnU/GmzVk3IN 4Bk5V3zYPvDHT7WJY+CMUxwOg+GKr4GztpqHSUxWS7bn0/TniOkPLTJFhKKf3+QQakg7 bBKBLSWUejmg0nC/whrVXxaZA5Xf2nPkhLSzI9FNSSGsc1qU1HnrCowEisf9KBCziIE0 ZaMA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:content-transfer-encoding:reply-to:list-subscribe :list-help:list-post:list-archive:list-unsubscribe:list-id :precedence:subject:mime-version:message-id:date:to:from :dkim-signature:delivered-to; bh=KzaaIBns+K6m/pGM12jp2UGdwnQ1PEVi2t45gfOu92A=; fh=YOA8vD9MJZuwZ71F/05pj6KdCjf6jQRmzLS+CATXUQk=; b=R0NHD/hPBbc5bxrRAR1OOWAFC2vxk0hWXnvJBugd9LpSxzbQdpHQOKhBlHMAFTYl3L IB9n8D4PKydHicB43slKUHARsDomt3nLZ0JwQBAjWQmnpqAT7XKHTGTgh0H5dnZS9+nB nX/5q09s34q/NYLrGEGQ9x/Iq/KnBmgJqHg1c8xPRwWDI7p3KW1O7kRult67ziaCGIt1 pS9q0MXWx8FtAwyrI1FkgjwPF+RmRGVGG5+j6lsVh3+gskqhZsR7WWm8YuX91IBa6jsk beuXTqti2EO0l4+KL4i9mFtjcPBxYkuxC8yM/Mk39Ukgm3KXeiRHEKaOgalK+RTEnlX6 5n4Q==; dara=google.com ARC-Authentication-Results: i=1; mx.google.com; dkim=neutral (body hash did not verify) header.i=@gmail.com header.s=20230601 header.b=U4AZN96k; spf=pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) smtp.mailfrom=ffmpeg-devel-bounces@ffmpeg.org; dmarc=fail (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org. [79.124.17.100]) by mx.google.com with ESMTP id x22-20020a1709064a9600b00a51feccc737si420329eju.893.2024.04.09.06.05.39; Tue, 09 Apr 2024 06:06:03 -0700 (PDT) Received-SPF: pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) client-ip=79.124.17.100; Authentication-Results: mx.google.com; dkim=neutral (body hash did not verify) header.i=@gmail.com header.s=20230601 header.b=U4AZN96k; spf=pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) smtp.mailfrom=ffmpeg-devel-bounces@ffmpeg.org; dmarc=fail (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 4B46A68D18D; Tue, 9 Apr 2024 16:05:37 +0300 (EEST) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from mail-pf1-f174.google.com (mail-pf1-f174.google.com [209.85.210.174]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id C232A689AEE for ; Tue, 9 Apr 2024 16:05:30 +0300 (EEST) Received: by mail-pf1-f174.google.com with SMTP id d2e1a72fcca58-6e703e0e5deso4785680b3a.3 for ; Tue, 09 Apr 2024 06:05:30 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1712667927; x=1713272727; darn=ffmpeg.org; h=content-transfer-encoding:mime-version:message-id:date:subject:to :from:from:to:cc:subject:date:message-id:reply-to; bh=B+rIFf2pjdr9bdyXPWZ1zqz5ITDKJQq3ZEWiwlhtdYM=; b=U4AZN96km+xs4vW0SPkl5C6SpcSanx1tuV/a+N+sqWv7tYRFyqBuicK2fcke6lXjro XYF/nMA7Et5UFoKQItefoVUInAyl25Su0yZTqs58C6RXZ9Wrb6dhxBY8caxAxNWYZ5DY oM0N+KXc0QP0QaZXhRKoqSCd05NNlnLGtsbly7oYt63hEMkZwmBC8UeLfKe5283xQrpa x8MamO/a0fJtmdMf+hb3ekd+fWWPxxZvB1vXsLTmQIgBl7vQFADjo6Iejt4VqBfxf/Xo ux9YdKLsm1ZLXjk0JthymPbaVBL+NCJHxHnnle7Cs79Q2NMOi+0vJHlzfiQdYVN+W4WP +Gzw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1712667927; x=1713272727; h=content-transfer-encoding:mime-version:message-id:date:subject:to :from:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=B+rIFf2pjdr9bdyXPWZ1zqz5ITDKJQq3ZEWiwlhtdYM=; b=j5p/R9z8pQWQuOAlJ6+RWxnkXK7CjB+vcBmbAvHOM882BDFbR0vseneCls1U96xFtr j9+OX8jZrFSb5GpvqPYGhLM0C4GMol2Uc0s/JtVAIFIuwM3XYBG9IKFTSoC4A0mnJfup WnridryrOSYrX3i4pMYiuDrkXhamqUWvnsgUSOeZDnkqMPi2XB4JkQ9vw8CcYFcPkmwV vtllJKXb+V6ADQijynPQBgqD5pD4idLenxpgbNXZSYrQ1JNkpIWJMtWkUbp3uHjxJaQs FJDJ0b+YseR4Z9PkeD3tOdOsg6vjHSk3asXLE3Wn/XcjcgTPo0VeXm8E3klCbKSqbThc 91AA== X-Gm-Message-State: AOJu0Yzm+5RurZR593zVVYT2viZD5PtdAIwwaOPCM9H0EZ2tQiUcFXuS HfuZj8HctmTRdnAt0uh6lPN6gsWOHlkBD33DucSuwDXM96fEIJcw81zebVnj X-Received: by 2002:a05:6a00:a13:b0:6e6:89ad:1233 with SMTP id p19-20020a056a000a1300b006e689ad1233mr13087883pfh.2.1712667927130; Tue, 09 Apr 2024 06:05:27 -0700 (PDT) Received: from localhost.localdomain ([190.194.167.233]) by smtp.gmail.com with ESMTPSA id fa12-20020a056a002d0c00b006e694719fa0sm6492990pfb.147.2024.04.09.06.05.25 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 09 Apr 2024 06:05:26 -0700 (PDT) From: James Almer To: ffmpeg-devel@ffmpeg.org Date: Tue, 9 Apr 2024 10:05:09 -0300 Message-ID: <20240409130509.1461-1-jamrial@gmail.com> X-Mailer: git-send-email 2.44.0 MIME-Version: 1.0 Subject: [FFmpeg-devel] [PATCH] avcodec/libsvt1: check return value of sned/receive functions X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.29 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 Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" X-TUID: MCjq3vhd9cqJ Signed-off-by: James Almer --- libavcodec/libsvtav1.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/libavcodec/libsvtav1.c b/libavcodec/libsvtav1.c index 105c3369c0..6ff893cf10 100644 --- a/libavcodec/libsvtav1.c +++ b/libavcodec/libsvtav1.c @@ -486,6 +486,7 @@ static int eb_send_frame(AVCodecContext *avctx, const AVFrame *frame) { SvtContext *svt_enc = avctx->priv_data; EbBufferHeaderType *headerPtr = svt_enc->in_buf; + EbErrorType svt_ret; int ret; if (!frame) { @@ -524,7 +525,9 @@ static int eb_send_frame(AVCodecContext *avctx, const AVFrame *frame) if (avctx->gop_size == 1) headerPtr->pic_type = EB_AV1_KEY_PICTURE; - svt_av1_enc_send_picture(svt_enc->svt_handle, headerPtr); + svt_ret = svt_av1_enc_send_picture(svt_enc->svt_handle, headerPtr); + if (svt_ret != EB_ErrorNone) + return svt_print_error(avctx, svt_ret, "Error sending a frame to encoder"); return 0; } @@ -579,6 +582,8 @@ static int eb_receive_packet(AVCodecContext *avctx, AVPacket *pkt) svt_ret = svt_av1_enc_get_packet(svt_enc->svt_handle, &headerPtr, svt_enc->eos_flag); if (svt_ret == EB_NoErrorEmptyQueue) return AVERROR(EAGAIN); + else if (svt_ret != EB_ErrorNone) + return svt_print_error(avctx, svt_ret, "Error getting an output packet from encoder"); #if SVT_AV1_CHECK_VERSION(2, 0, 0) if (headerPtr->flags & EB_BUFFERFLAG_EOS) {