Opened 3 years ago

Closed 3 years ago

Last modified 3 years ago

#9235 closed defect (fixed)

commit 958bea5 broke fate-flv-demux

Reported by: haihao Owned by:
Priority: normal Component: undetermined
Version: unspecified Keywords:
Cc: Blocked By:
Blocking: Reproduced by developer: no
Analyzed by developer: no

Description

commit 958bea5 caused a regression when running make fate-flv-demux V=2 KEEP=1

[flv @ 0x5612cb432e90] Packet corrupt (stream = 0, dts = 11678).
[NULL @ 0x5612cb434c20] Invalid NAL unit size (2771 > 1186).
[NULL @ 0x5612cb434c20] missing picture in access unit with size 1190
make: * [tests/Makefile:255: fate-flv-demux] Error 1

make fate-flv-demux V=2 KEEP=1 works well after resetting HEAD to 958bea5~1

The issue still exists after using the latest FFmpeg (commit 7fac6efa)

Change History (3)

comment:1 by U. Artie Eoff, 3 years ago

cc: Anton Khirnov <anton@khirnov.net>

comment:2 by U. Artie Eoff, 3 years ago

Resolution: fixed
Status: newclosed

Verified fixed in:
commit 6a9a4f34bd4ab10726d43f8961edfd99a5c47647
Author: Anton Khirnov <anton@khirnov.net>
Date: Mon May 10 09:15:29 2021 +0200

lavf/flvdec: normalize exporting date metadata


Export them in UTC, not the local timezone. This way the output is
the same everywhere. The timezone information stored in the file is
still ignored, since there seems to be no simple way to export it
correctly.


Format them according to ISO 8601, which we generally use for exporting
dates.


Fixes fate-flv-demux, which was broken since
958bea5248f87116b0dd080461aa70c14ea86cf0 on some platforms.

Last edited 3 years ago by Carl Eugen Hoyos (previous) (diff)

comment:3 by Carl Eugen Hoyos, 3 years ago

Wow, a regression was justified by a new bug...

Note: See TracTickets for help on using tickets.