Opened 8 years ago

Closed 8 years ago

#5364 closed defect (fixed)

artefacts/errors with odd-sized flx

Reported by: ami_stuff Owned by:
Priority: normal Component: avcodec
Version: git-master Keywords: flic
Cc: Blocked By:
Blocking: Reproduced by developer: yes
Analyzed by developer: no

Description

http://www.datafilehost.com/d/e1f00679

C:\>d:\ffmpeg -i D:\333_333.flx 1\out.avi
ffmpeg version N-79107-g30d1213 Copyright (c) 2000-2016 the FFmpeg developers
  built with gcc 5.3.0 (GCC)
  configuration: --enable-gpl --enable-version3 --disable-w32threads --enable-av
isynth --enable-bzlib --enable-fontconfig --enable-frei0r --enable-gnutls --enab
le-iconv --enable-libass --enable-libbluray --enable-libbs2b --enable-libcaca --
enable-libdcadec --enable-libfreetype --enable-libgme --enable-libgsm --enable-l
ibilbc --enable-libmodplug --enable-libmfx --enable-libmp3lame --enable-libopenc
ore-amrnb --enable-libopencore-amrwb --enable-libopenjpeg --enable-libopus --ena
ble-librtmp --enable-libschroedinger --enable-libsnappy --enable-libsoxr --enabl
e-libspeex --enable-libtheora --enable-libtwolame --enable-libvidstab --enable-l
ibvo-amrwbenc --enable-libvorbis --enable-libvpx --enable-libwavpack --enable-li
bwebp --enable-libx264 --enable-libx265 --enable-libxavs --enable-libxvid --enab
le-libzimg --enable-lzma --enable-decklink --enable-zlib
  libavutil      55. 19.100 / 55. 19.100
  libavcodec     57. 28.103 / 57. 28.103
  libavformat    57. 29.100 / 57. 29.100
  libavdevice    57.  0.101 / 57.  0.101
  libavfilter     6. 39.102 /  6. 39.102
  libswscale      4.  0.100 /  4.  0.100
  libswresample   2.  0.101 /  2.  0.101
  libpostproc    54.  0.100 / 54.  0.100
Input #0, flic, from 'D:\333_333.flx':
  Duration: N/A, start: 0.000000, bitrate: N/A
    Stream #0:0: Video: flic, rgb555le, 333x333, 20 tbr, 20 tbn, 20 tbc
Output #0, avi, to '1\out.avi':
  Metadata:
    ISFT            : Lavf57.29.100
    Stream #0:0: Video: mpeg4 (FMP4 / 0x34504D46), yuv420p, 333x333, q=2-31, 200
 kb/s, 20 fps, 20 tbn, 20 tbc
    Metadata:
      encoder         : Lavc57.28.103 mpeg4
    Side data:
      cpb: bitrate max/min/avg: 0/0/200000 buffer size: 0 vbv_delay: -1
Stream mapping:
  Stream #0:0 -> #0:0 (flic (native) -> mpeg4 (native))
Press [q] to stop, [?] for help
[flic @ 0000000000454200] In chunk FLI_COPY : source data (222444 bytes) bigger
than image, skipping chunk
    Last message repeated 1 times
frame=  244 fps=0.0 q=7.7 size=     486kB time=00:00:12.20 bitrate= 326.2kbits/s
[flic @ 0000000000454200] In chunk FLI_COPY : source data (222444 bytes) bigger
than image, skipping chunk
    Last message repeated 1 times
frame=  491 fps=491 q=6.2 size=     822kB time=00:00:24.55 bitrate= 274.3kbits/s
[flic @ 0000000000454200] In chunk FLI_COPY : source data (222444 bytes) bigger
than image, skipping chunk
D:\333_333.flx: I/O error
frame=  600 fps=486 q=10.6 Lsize=     982kB time=00:00:30.00 bitrate= 268.0kbits
/s speed=24.3x
video:962kB audio:0kB subtitle:0kB other streams:0kB global headers:0kB muxing o
verhead: 2.069649%

Change History (7)

comment:1 by Carl Eugen Hoyos, 8 years ago

Component: undeterminedavcodec
Keywords: flic added
Reproduced by developer: set
Status: newopen
Version: unspecifiedgit-master

Patch sent.

Are you able to verify that the files are valid / can be decoded with another decoder?

in reply to:  1 comment:2 by ami_stuff, 8 years ago

Replying to cehoyos:

Patch sent.

Thanks.

Are you able to verify that the files are valid / can be decoded with another decoder?

Yes, the files decodes correctly in Ulead VideoStudio.

comment:3 by Carl Eugen Hoyos, 8 years ago

Thank you for the confirmation!

Please consider to test my patch with more samples, as often it is based on pure guessing;-)

comment:4 by ami_stuff, 8 years ago

Here are some more samples which I created. Unfortunately I can't test your patch atm, so if these files decodes correctly please lgtm your patch.

http://www.datafilehost.com/d/07a61521

comment:5 by Carl Eugen Hoyos, 8 years ago

I'd still need a pal8 sample with an even width not dividable by four.

comment:7 by Carl Eugen Hoyos, 8 years ago

Resolution: fixed
Status: openclosed

Should be fixed in de1a0d43595b98c1093c9fc39de0e96fba8e9a06 - thank you for the samples!

Note: See TracTickets for help on using tickets.