[FFmpeg-devel] "Lossless check failed" after applying suggested patch

[Thoralf Dassler]

Hi again,

as suggested by Carl Eugen, I applied the below patch and this solves 
the original problem (see below). Now, on navigating through frames with 
`mplayer <filename>' (arrow keys, page up/page down), I get a new 
error/warning; same if I do `mplayer <filename> -channels 6'.

Could it be my file is damaged -- I could upload a sample if required? 
Anyway, the new error/warning is:

[truehd @ 0xde50e0]Lossless check failed - expected 6e, calculated cc.
A: 623.5 V: 623.5 A-V:  0.004 ct:  0.047   0/  0 40%  3%  2.6% 0 0
[truehd @ 0xde50e0]Lossless check failed - expected ef, calculated 5b.
A: 684.6 V: 684.6 A-V:  0.022 ct:  0.058   0/  0 33%  3%  4.8% 0 0
[truehd @ 0xde50e0]Lossless check failed - expected 60, calculated 97.
A: 745.7 V: 745.6 A-V:  0.077 ct:  0.064   0/  0 37%  3%  5.3% 0 0
[truehd @ 0xde50e0]Lossless check failed - expected 3c, calculated c9.
A: 813.0 V: 812.9 A-V:  0.068 ct:  0.063   0/  0 34%  3%  5.3% 0 0
[truehd @ 0xde50e0]Lossless check failed - expected 65, calculated 11.
A: 874.0 V: 874.0 A-V:  0.025 ct:  0.038   0/  0 ??% ??% ??,?% 0 0
[truehd @ 0xde50e0]Lossless check failed - expected ac, calculated 31.
A: 940.0 V: 939.9 A-V:  0.057 ct:  0.034   0/  0 ??% ??% ??,?% 0 0
[truehd @ 0xde50e0]Lossless check failed - expected a7, calculated 06.
A:1007.2 V:1007.1 A-V:  0.093 ct:  0.028   0/  0 ??% ??% ??,?% 0 0
[truehd @ 0xde50e0]Lossless check failed - expected ad, calculated 24.
A:1077.3 V:1077.2 A-V:  0.099 ct:  0.034   0/  0 ??% ??% ??,?% 0 0
[truehd @ 0xde50e0]Lossless check failed - expected 00, calculated 5b.

etc.

Thnaks,
Thoralf

********************

Hi!

Es ist nicht erforderlich, ein Sample bereitzustellen (das Problem tritt 
mit jedem TrueHD sample und mplayer -channels 2 auf, wobei channels 2 
default ist - ich teste leider immer mit channels 8).

Das sollte in nächster Zeit wie gesagt gelöst werden, entweder indem die 
Anfrage ganz verschwindet, oder indem der Patch unten applied wird.
(inlined)

Danke für den Report, Carl Eugen

diff --git a/libavcodec/mlpdec.c b/libavcodec/mlpdec.c
index 50826d0..b13d079 100644
--- a/libavcodec/mlpdec.c
+++ b/libavcodec/mlpdec.c
@@ -340,6 +340,7 @@ static int read_major_sync(MLPDecodeContext *m, 
GetBitContext *gb)
              m->avctx->channel_layout = 
ff_truehd_layout(mh.channels_thd_stream1);
          }
          if (m->avctx->channels &&
+            !m->avctx->request_channels && 
!m->avctx->request_channel_layout &&
              
av_get_channel_layout_nb_channels(m->avctx->channel_layout) != 
m->avctx->channels) {
              m->avctx->channel_layout = 0;
              av_log_ask_for_sample(m->avctx, "Unknown channel layout.");

*******************

On Sun, May 22, 2011 at 11:22:46AM +0100, Thoralf Dassler wrote:

> >  Hi,
> >
> >  On certain files I receive this error/warning from FFmpeg TrueHD, played
> >  with mplayer<filename>. I would like to upload as ample, but the email
> >  then exceeds the 400k limit.
> >
> >  The full error/warning is this (there are hundreds when watching the movie):
> >
> >  [truehd @ 0xde32a0]Unknown channel layout.If you want to help, upload a
> >  sample of this file toftp://upload.ffmpeg.org/MPlayer/incoming/  and
> >  contact the ffmpeg-devel mailing list.
> >
> >  My mplayer/ffmpeg combination is of today, from svn and git.