Opened 3 years ago

Closed 3 years ago

#2711 closed defect (fixed)

xa: memory eating

Reported by: ami_stuff Owned by:
Priority: normal Component: undetermined
Version: git-master Keywords: adpcm xa
Cc: Blocked By:
Blocking: Reproduced by developer: no
Analyzed by developer: no

Description

from a quick look it seems that xa_probe() from "libavformat/xa.c" is not executed

http://samples.mplayerhq.hu/A-codecs/MP3-pro/27%20MC%20Solaar%20-%20Rmi.mp3

C:\>ffmpeg -f xa -i "27 MC Solaar - Rmi.mp3" -f null -
ffmpeg version N-46283-g1475815 Copyright (c) 2000-2012 the FFmpeg developers
  built on Nov  2 2012 02:31:20 with gcc 4.5.0 (GCC) 20100414 (Fedora MinGW 4.5.
0-1.fc14)
  configuration: --prefix=/var/www/users/research/ffmpeg/snapshots/build --arch=
x86 --target-os=mingw32 --cross-prefix=i686-pc-mingw32- --cc='ccache i686-pc-min
gw32-gcc' --enable-w32threads --enable-memalign-hack --enable-runtime-cpudetect
--enable-cross-compile --enable-static --disable-shared --extra-libs='-lws2_32 -
lwinmm -lpthread' --extra-cflags='--static -I/var/www/users/research/ffmpeg/snap
shots/build/include' --extra-ldflags='-static -L/var/www/users/research/ffmpeg/s
napshots/build/lib' --enable-bzlib --enable-zlib --enable-gpl --enable-version3
--enable-nonfree --enable-libx264 --enable-libspeex --enable-libtheora --enable-
libvorbis --enable-libfaac --enable-libxvid --enable-libopencore-amrnb --enable-
libopencore-amrwb --enable-libmp3lame --enable-libvpx --disable-decoder=libvpx
  libavutil      52.  1.100 / 52.  1.100
  libavcodec     54. 70.100 / 54. 70.100
  libavformat    54. 35.100 / 54. 35.100
  libavdevice    54.  3.100 / 54.  3.100
  libavfilter     3. 21.106 /  3. 21.106
  libswscale      2.  1.102 /  2.  1.102
  libswresample   0. 16.100 /  0. 16.100
  libpostproc    52.  1.100 / 52.  1.100
[adpcm_ea_maxis_xa @ 0x1edcbc0] Invalid number of channels
    Last message repeated 1 times

Change History (2)

comment:1 Changed 3 years ago by cehoyos

  • Keywords adpcm added
  • Version changed from unspecified to git-master

Memory consumption could be considered too high.

$ valgrind ./ffmpeg_g -f xa -i 27\ MC\ Solaar\ -\ Rmi.mp3
==22420== Memcheck, a memory error detector
==22420== Copyright (C) 2002-2011, and GNU GPL'd, by Julian Seward et al.
==22420== Using Valgrind-3.7.0 and LibVEX; rerun with -h for copyright info
==22420== Command: ./ffmpeg_g -f xa -i 27\ MC\ Solaar\ -\ Rmi.mp3
==22420==
ffmpeg version N-54197-g015cc32 Copyright (c) 2000-2013 the FFmpeg developers
  built on Jun 25 2013 21:04:41 with gcc 4.7 (SUSE Linux)
  configuration: --enable-gpl --disable-indev=jack
  libavutil      52. 37.101 / 52. 37.101
  libavcodec     55. 17.100 / 55. 17.100
  libavformat    55. 10.100 / 55. 10.100
  libavdevice    55.  2.100 / 55.  2.100
  libavfilter     3. 77.101 /  3. 77.101
  libswscale      2.  3.100 /  2.  3.100
  libswresample   0. 17.102 /  0. 17.102
  libpostproc    52.  3.100 / 52.  3.100
[adpcm_ea_maxis_xa @ 0x72381a0] Invalid number of channels
    Last message repeated 1 times
[xa @ 0x722f3a0] max_analyze_duration 5000000 reached at 5000000 microseconds
[xa @ 0x722f3a0] Could not find codec parameters for stream 0 (Audio: adpcm_ea_maxis_xa, 42467584 Hz, 0 channels): unspecified number of channels
Consider increasing the value for the 'analyzeduration' and 'probesize' options
27 MC Solaar - Rmi.mp3: could not find codec parameters
==22420==
==22420== HEAP SUMMARY:
==22420==     in use at exit: 30,635 bytes in 7 blocks
==22420==   total heap usage: 37,917,547 allocs, 37,917,540 frees, 6,066,922,302 bytes allocated
==22420==
==22420== LEAK SUMMARY:
==22420==    definitely lost: 0 bytes in 0 blocks
==22420==    indirectly lost: 0 bytes in 0 blocks
==22420==      possibly lost: 0 bytes in 0 blocks
==22420==    still reachable: 30,635 bytes in 7 blocks
==22420==         suppressed: 0 bytes in 0 blocks
==22420== Rerun with --leak-check=full to see details of leaked memory
==22420==
==22420== For counts of detected and suppressed errors, rerun with: -v
==22420== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 2 from 2)

comment:2 Changed 3 years ago by cehoyos

  • Keywords xa added
  • Resolution set to fixed
  • Status changed from new to closed

Fixed by Paul B Mahol.

Note: See TracTickets for help on using tickets.