Opened 10 years ago

Last modified 8 years ago

#1337 new defect

[PATCH] -demuxer lavf and dimensions not set crash

Reported by: mennucc1@… Owned by: reimar
Priority: normal Component: demuxer
Version: HEAD Severity: normal
Keywords: Cc: compn, jojelino@…
Blocked By: Blocking:
Reproduced by developer: Analyzed by developer:

Description

hi,

the command
./mencoder -ss 8 -ovc lavc -demuxer lavf -oac copy -o /tmp/pianta.avi -of lavf /data4/tv/MIIB.mpeg
(the file is at the above URL)
dies of a division by zero; here following is the backtrace

#0 av_frac_add (f=0x8dd8270, incr=<value optimized out>) at utils.c:87
#1 0x0825cbb2 in compute_pkt_fields2 (st=0x8dd8250, pkt=0xbf9eac64)

at utils.c:2590

#2 0x0825feb8 in av_interleaved_write_frame (s=0x8ddb9d0, pkt=0xbf9eac64)

at utils.c:2688

#3 0x080ac48f in write_chunk (stream=0x8dd7990, len=384,

flags=<value optimized out>, dts=0, pts=0) at libmpdemux/muxer_lavf.c:261

#4 0x080a111c in muxer_write_chunk (s=0x8dd7990, len=0, flags=0,

dts=-9.2233720368547758e+18, pts=-9.2233720368547758e+18)
at libmpdemux/muxer.c:125

#5 0x0809b7c8 in main (argc=148732304, argv=0xbf9eaff4) at mencoder.c:1385

a simple patch avoids this problem

a.

Attachments (1)

utils.c.diff (344 bytes) - added by mennucc1@… 10 years ago.
patch

Download all attachments as: .zip

Change History (10)

Changed 10 years ago by mennucc1@…

patch

comment:1 Changed 10 years ago by mennucc1@…

comment:2 Changed 10 years ago by compn

sample url is 404

comment:3 Changed 9 years ago by mennucc1@…

sorry, I deleted that sample; but the patch may be worth nonetheless

comment:4 Changed 8 years ago by compn

  • Owner changed from r_togni@… to reimar

comment:5 Changed 8 years ago by compn

  • Cc patriotact@… added
  • Summary changed from crash when using '-ss -demuxer lavf ' to [PATCH] crash when using '-ss -demuxer lavf '

comment:6 Changed 8 years ago by compn

  • Cc jojelino@… added

comment:7 Changed 8 years ago by compn

comment:8 Changed 8 years ago by reimar

Unfortunately the patch is not a proper solution.
There should never be a 0 there, the real difficulty is figuring how it got there and fix the source of it.

comment:9 Changed 8 years ago by compn

  • Summary changed from [PATCH] crash when using '-ss -demuxer lavf ' to [PATCH] -demuxer lavf and dimensions not set crash

try mplayer demuxer over lavf ?

update title.

Note: See TracTickets for help on using tickets.