On 01/12/2017 05:31 AM, Diego Biurrun wrote:
> On Wed, Jan 11, 2017 at 12:22:10PM -0700, John Stebbins wrote:
>> --- a/libavformat/dv.c
>> +++ b/libavformat/dv.c
>> @@ -478,7 +478,7 @@ static int dv_read_header(AVFormatContext *s)
>> static int dv_read_packet(AVFormatContext *s, AVPacket *pkt)
>> {
>> - int size;
>> + int size, result;
>> RawDVContext *c = s->priv_data;
>>
>> @@ -487,7 +487,10 @@ static int dv_read_packet(AVFormatContext *s, AVPacket
>> *pkt)
>> if (!c->dv_demux->sys)
>> return AVERROR(EIO);
>> size = c->dv_demux->sys->frame_size;
>> - if (avio_read(s->pb, c->buf, size) <= 0)
>> + result = avio_read(s->pb, c->buf, size);
>> + if (result == AVERROR_EOF)
>> + return result;
>> + if (result <= 0)
>> return AVERROR(EIO);
> nit: The variable could have smaller scope.
Ok. Will submit update to fix.
>
> Is there a specific problem that this fixes?
Yes. HandBrake shows that encodes do not finish successfully because we
receive EIO from libav instead of EOF.-- John GnuPG fingerprint: D0EC B3DB C372 D1F1 0B01 83F0 49F1 D7B2 60D4 D0F7
signature.asc
Description: OpenPGP digital signature
_______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
