On 01/13/2017 12:36 AM, Luca Barbato wrote:
> On 12/01/2017 21:36, John Stebbins wrote:
>> ---
>>  libavformat/asfdec.c | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
> How about:
>
> Correctly iterate over the ASF streams
>
> (what makes me wonder is why there are more mapped streams than file
> streams?)

I can't tell you the exact reasons, but with the sample, 2 AVStreams are 
created with asf_read_stream_properties 1
AVStream is created with asf_read_picture and 1 AVStream is created with 
ff_id3v2_parse_apic. The one created with
ff_id3v2_parse_apic does not have a corresponding AFStream.


>> diff --git a/libavformat/asfdec.c b/libavformat/asfdec.c
>> index 1c50ad6..d602af8 100644
>> --- a/libavformat/asfdec.c
>> +++ b/libavformat/asfdec.c
>> @@ -1485,7 +1485,7 @@ static int asf_read_packet(AVFormatContext *s, 
>> AVPacket *pkt)
>>              asf->return_subpayload = 0;
>>              return 0;
>>          }
>> -        for (i = 0; i < s->nb_streams; i++) {
>> +        for (i = 0; i < asf->nb_streams; i++) {
>>              ASFPacket *asf_pkt = &asf->asf_st[i]->pkt;
>>              if (asf_pkt && !asf_pkt->size_left && asf_pkt->data_size) {
>>                  if (asf->asf_st[i]->span > 1 &&
>>
> _______________________________________________
>

-- 
John      GnuPG fingerprint: D0EC B3DB C372 D1F1 0B01  83F0 49F1 D7B2 60D4 D0F7

Attachment: signature.asc
Description: OpenPGP digital signature

_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to