From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTP id C0207455FE for ; Thu, 9 Mar 2023 20:16:54 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id B09BA68BAA5; Thu, 9 Mar 2023 22:16:51 +0200 (EET) Received: from iq.passwd.hu (iq.passwd.hu [217.27.212.140]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 5E53E688373 for ; Thu, 9 Mar 2023 22:16:45 +0200 (EET) Received: from localhost (localhost [127.0.0.1]) by iq.passwd.hu (Postfix) with ESMTP id 2D5F7E6E8E for ; Thu, 9 Mar 2023 21:16:46 +0100 (CET) X-Virus-Scanned: amavisd-new at passwd.hu Received: from iq.passwd.hu ([127.0.0.1]) by localhost (iq.passwd.hu [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 0UemKnNGjoC5 for ; Thu, 9 Mar 2023 21:16:41 +0100 (CET) Received: from iq (iq [217.27.212.140]) by iq.passwd.hu (Postfix) with ESMTPS id C816FE6538 for ; Thu, 9 Mar 2023 21:16:41 +0100 (CET) Date: Thu, 9 Mar 2023 21:16:41 +0100 (CET) From: Marton Balint To: FFmpeg development discussions and patches In-Reply-To: Message-ID: <70b44e81-cb5b-2d7f-3b19-8ad85eae348a@passwd.hu> References: MIME-Version: 1.0 Subject: Re: [FFmpeg-devel] [PATCH] libavformat/aea EOF Patch 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 Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: On Thu, 9 Mar 2023, asivery wrote: > Thank you very much, indeed that works as well. > Here's the updated patch. Will apply, thanks. Marton > > Best regards. > > --- > diff --git a/libavformat/aea.c b/libavformat/aea.c > index f4b39e4f9e..d16217381b 100644 > --- a/libavformat/aea.c > +++ b/libavformat/aea.c > @@ -90,13 +90,7 @@ static int aea_read_header(AVFormatContext *s) > > static int aea_read_packet(AVFormatContext *s, AVPacket *pkt) > { > - int ret = av_get_packet(s->pb, pkt, s->streams[0]->codecpar->block_align); > - > - pkt->stream_index = 0; > - if (ret <= 0) > - return AVERROR(EIO); > - > - return ret; > + return av_get_packet(s->pb, pkt, s->streams[0]->codecpar->block_align); > } > > const AVInputFormat ff_aea_demuxer = { > > ------- Original Message ------- > On Thursday, March 9th, 2023 at 12:28 AM, Marton Balint wrote: > > >> >> On Sat, 4 Mar 2023, asivery wrote: >> >>> Could someone please take a look? This would improve AEA demuxing, right now every AEA file makes ffmpeg crash with an error. >> >> >> The way I see it, you could simply replace the whole body of >> aea_read_packet() with this single line: >> >> return av_get_packet(s->pb, pkt, s->streams[0]->codecpar->block_align); >> >> >> Regards, >> Marton >> >>> Thank you in advance :) >>> ------- Original Message ------- >>> On Saturday, October 22nd, 2022 at 7:43 PM, asivery asivery@protonmail.com wrote: >>> >>>> Hello, >>>> This patch aims to fix a problem I've noticed while working with AEA (Sony ATRAC1 comtainer) files. >>>> Right now FFmpeg always exits with an "Input/Output error" when dealing with AEA files. >>>> This patch solves that issue by returning AVERROR_EOF once the end of file is encountered, instead of always returning AVERROR(EIO). >>>> >>>> I am sending this patch again because of an incorrect mime type of the first one. >>>> >>>> Best regards,Asivery >>>> _______________________________________________ >>>> ffmpeg-devel mailing list >>>> ffmpeg-devel@ffmpeg.org >>>> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel >>> >>> To unsubscribe, visit link above, or email >>> ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe". >> >> _______________________________________________ >> ffmpeg-devel mailing list >> ffmpeg-devel@ffmpeg.org >> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel >> >> To unsubscribe, visit link above, or email >> ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe". > _______________________________________________ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > To unsubscribe, visit link above, or email > ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe". > _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".