Opened 9 years ago

Closed 8 years ago

Last modified 19 months ago

#4786 closed defect (fixed)

MLP misdetected for dvd_pcm

Reported by: Alexander E. Patrakov Owned by:
Priority: normal Component: avformat
Version: git-master Keywords: mpegps mlp
Cc: Blocked By:
Blocking: Reproduced by developer: yes
Analyzed by developer: no

Description

Summary of the bug:
How to reproduce:

$ ffprobe ATS_01_1.AOB
ffprobe version 2.7.2 Copyright (c) 2007-2015 the FFmpeg developers
  built with gcc 5.2.0 (GCC)
  configuration: --prefix=/usr --disable-debug --disable-static --disable-stripping --enable-avisynth --enable-avresample --enable-fontconfig --enable-gnutls --enable-gpl --enable-libass --enable-libbluray --enable-libfreetype --enable-libfribidi --enable-libgsm --enable-libmodplug --enable-libmp3lame --enable-libopencore_amrnb --enable-libopencore_amrwb --enable-libopenjpeg --enable-libopus --enable-libpulse --enable-libschroedinger --enable-libsoxr --enable-libspeex --enable-libssh --enable-libtheora --enable-libv4l2 --enable-libvorbis --enable-libvpx --enable-libwebp --enable-libx264 --enable-libx265 --enable-libxvid --enable-shared --enable-version3 --enable-x11grab
  libavutil      54. 27.100 / 54. 27.100
  libavcodec     56. 41.100 / 56. 41.100
  libavformat    56. 36.100 / 56. 36.100
  libavdevice    56.  4.100 / 56.  4.100
  libavfilter     5. 16.101 /  5. 16.101
  libavresample   2.  1.  0 /  2.  1.  0
  libswscale      3.  1.101 /  3.  1.101
  libswresample   1.  2.100 /  1.  2.100
  libpostproc    53.  3.100 / 53.  3.100
Input #0, mpeg, from 'ATS_01_1.AOB':
  Duration: 00:00:22.96, start: 0.007333, bitrate: 3568 kb/s
    Stream #0:0[0xa1]: Audio: pcm_dvd, 48000 Hz, 6 channels, s16, 4608 kb/s

However the stream is actually MLP, not PCM, and is playable by VLC.

Attachments (2)

ATS_01_1.AOB (2.0 MB ) - added by Alexander E. Patrakov 9 years ago.
patchmpegmlp.diff (493 bytes ) - added by Carl Eugen Hoyos 9 years ago.

Download all attachments as: .zip

Change History (6)

by Alexander E. Patrakov, 9 years ago

Attachment: ATS_01_1.AOB added

comment:1 by Alexander E. Patrakov, 9 years ago

well, the output in the initial report was for a 10-megabyte sample, and the first 2 megabytes are attached because of the file size limit in this instance of Trac.

by Carl Eugen Hoyos, 9 years ago

Attachment: patchmpegmlp.diff added

comment:2 by Carl Eugen Hoyos, 9 years ago

Component: undeterminedavformat
Keywords: mpegps mlp added
Reproduced by developer: set
Status: newopen
Version: unspecifiedgit-master

I just committed a patch that allows forcing mlp with the following command line:

$ ffmpeg -acodec mlp -i ATS_01_1.AOB​ out.wav

PoC attached that matches my original patches for issue 1731 on the mailing list:
http://thread.gmane.org/gmane.comp.video.ffmpeg.devel/142613
http://thread.gmane.org/gmane.comp.video.ffmpeg.devel/142629

comment:3 by Carl Eugen Hoyos, 8 years ago

Resolution: fixed
Status: openclosed

Should be fixed in fb9036b3142e06631a70810c3d779f8e2d9f180c - thank you for the sample!

comment:4 by Balling, 19 months ago

I will just point out that this is real mlp (after -c copy you see 0xF8726FBB) and has metadata that gives 24 bit (TrueHD does not have it).

Yes, it is not lossless with what Plex +EAE.exe produce. WHY??

Plex Transcoder.exe -i ATS_01_1.AOB -c:a pcm_s24le fe2e121.wav

ffmpeg.exe -i ATS_01_1.AOB -c:a pcm_s24le daqwdaw1.wav

Note: See TracTickets for help on using tickets.