Opened 16 years ago

Closed 16 years ago

#1089 closed defect (fixed)

Null pointer reference in ad_liba52.c:62

Reported by: rgsteele@… Owned by: r_togni@…
Priority: normal Component: ad
Version: HEAD Severity: normal
Keywords: Cc: rgsteele@…
Blocked By: Blocking:
Reproduced by developer: no Analyzed by developer: no

Description

I'm running the latest svn on x86_64 Hardy Heron. When attempting to play a video with AC3 encoded audio, mplayer crashes with a segmentation fault at libmpcodecs/ad_liba52.c:62.

62 int c=demux_getc(sh_audio->ds);

(gdb) print sh_audio->ds
$4 = (demux_stream_t *) 0x0

Attachments (5)

mplayer backtrace.txt (2.1 KB ) - added by rgsteele@… 16 years ago.
gdb backtrace
system info.txt (2.7 KB ) - added by rgsteele@… 16 years ago.
output of uname -a, ls -l /lib/libc[.-]*, etc.
mplayer patch.txt (466 bytes ) - added by rgsteele@… 16 years ago.
Patch
unpatchedmplayeroutputwithlavf.txt (19.6 KB ) - added by rgsteele@… 16 years ago.
./mplayer -demuxer lavf /home/ryan/Desktop/HVDVD_TS/FEATURE*EVO > ~/unpatchedmplayeroutputwithlavf.txt 2>&1
mypatchedmplayeroutput.txt (27.1 KB ) - added by rgsteele@… 16 years ago.
./mplayer /home/ryan/Desktop/HVDVD_TS/FEATURE*EVO > ~/mypatchedmplayeroutput.txt 2>&1

Download all attachments as: .zip

Change History (11)

by rgsteele@…, 16 years ago

Attachment: mplayer backtrace.txt added

gdb backtrace

comment:1 by rgsteele@…, 16 years ago

by rgsteele@…, 16 years ago

Attachment: system info.txt added

output of uname -a, ls -l /lib/libc[.-]*, etc.

comment:2 by rgsteele@…, 16 years ago

by rgsteele@…, 16 years ago

Attachment: mplayer patch.txt added

Patch

comment:3 by rgsteele@…, 16 years ago

Working with uau on #mplayerdev I found that running mplayer with "-demuxer lavf" is a workaround for the bug.

I've attached a patch for mplayer.c which seems to fix the bug. Treat it with suspicion -- I really don't know what I'm doing. :)

Note that with either fix the video plays back correctly, but the audio is just intermittent glitches, with a number of errors appearing on the terminal. I'll attach the output for each in case it helps, but I suspect this is a separate issue.

by rgsteele@…, 16 years ago

./mplayer -demuxer lavf /home/ryan/Desktop/HVDVD_TS/FEATURE*EVO > ~/unpatchedmplayeroutputwithlavf.txt 2>&1

comment:4 by rgsteele@…, 16 years ago

by rgsteele@…, 16 years ago

Attachment: mypatchedmplayeroutput.txt added

./mplayer /home/ryan/Desktop/HVDVD_TS/FEATURE*EVO > ~/mypatchedmplayeroutput.txt 2>&1

comment:5 by rgsteele@…, 16 years ago

comment:6 by rgsteele@…, 16 years ago

Resolution: fixed
Status: newclosed

From http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/2008-May/057536.html:

Thanks, though I think the real bug is elsewhere (in the AVI demuxer
more precisely). Fixed differently in SVN r26908.

Greetings,
Reimar Döffinger

Note: See TracTickets for help on using tickets.