On Fri, Nov 21, 2014 at 12:32 PM, Luca Barbato <[email protected]> wrote:
> On 19/11/14 07:28, Vittorio Giovara wrote:
>> On Mon, Nov 17, 2014 at 12:22 AM, Vittorio Giovara
>> <[email protected]> wrote:
>>> CC: [email protected]
>>> Bug-Id: CID 29261
>>> ---
>>>  libavformat/mpegenc.c   |  2 +-
>>>  tests/ref/lavf/mpg      |  6 +++---
>>>  tests/ref/lavf/ts       |  4 ++--
>>>  tests/ref/seek/lavf-mpg | 12 ++++++------
>>>  tests/ref/seek/lavf-ts  | 16 ++++++++--------
>>>  5 files changed, 20 insertions(+), 20 deletions(-)
>>>
>>> diff --git a/libavformat/mpegenc.c b/libavformat/mpegenc.c
>>> index fed03ed..9640893 100644
>>> --- a/libavformat/mpegenc.c
>>> +++ b/libavformat/mpegenc.c
>>> @@ -1057,7 +1057,7 @@ retry:
>>>          es_size              -= stream->premux_packet->unwritten_size;
>>>          stream->premux_packet = stream->premux_packet->next;
>>>      }
>>> -    if (es_size)
>>> +    if (stream->premux_packet && es_size)
>>>          stream->premux_packet->unwritten_size -= es_size;
>>>
>>>      if (remove_decoded_packets(ctx, s->last_scr) < 0)
>>> diff --git a/tests/ref/lavf/mpg b/tests/ref/lavf/mpg
>>> index e4c8ae0..400f3fa 100644
>>> --- a/tests/ref/lavf/mpg
>>> +++ b/tests/ref/lavf/mpg
>>> @@ -1,3 +1,3 @@
>>> -7df31ba8a5909e3c88b1d1a3f93c4ec2 *./tests/data/lavf/lavf.mpg
>>> -372736 ./tests/data/lavf/lavf.mpg
>>> -./tests/data/lavf/lavf.mpg CRC=0xdd24439e
>>> +6130e03eb58b397b0e93768ce1fe8bc2 *./tests/data/lavf/lavf.mpg
>>> +370688 ./tests/data/lavf/lavf.mpg
>>> +./tests/data/lavf/lavf.mpg CRC=0x756e6236
>>
>> This seems to break fate, even if I update the ref files, on a few
>> architectures (as spotted on oracle).
>> Does anyone have any idea why?
>
> Which architectures? Sounds extremely fishy.

It does, that's why I am asking ^^
It happens with gcc on any architecture (arm and x86_64 too).

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

Reply via email to