[MPlayer-users] release of ogmtools 0.5

[D Richard Felker III]

On Wed, Jul 24, 2002 at 06:08:21PM +0200, Moritz Bunkus wrote:
> [Automatic answer: RTFM (read DOCS, FAQ), also read DOCS/bugreports.html]
> Hi folks.
> 
> A few days ago I released the first version of 'oggmerge' and
> 'ogmdemux'. I've packaged both (and the new 'ogminfo') together into
> the ogmtools. They are not finished by far - but I'd like to release
> often and early. So here's v0.5.
> 
> I encourage everyone to try them AFTER reading the README (which is
> included below) and to provide feedback about bugs, missing
> functionality (if it isn't already mentioned in the README) and so on.
> 
> http://www.bunkus.org/videotools/ogmtools/

ogmmerge segfaults for me:

(gdb) run -v -o test.ogm foo.avi bar.ogg
Starting program: /tmp/ogmtools/ogmmerge -v -o test.ogm foo.avi bar.ogg
Opening foo.avi for reading (type AVI (Audio/Video Interleaved))...
Opening bar.ogg for reading (type Vorbis audio embedded in OGG)...

Program received signal SIGSEGV, Segmentation fault.
vorbis_page_out (state=0x80521a8) at vorbis.c:117
117       page = vstate->pages->page;
(gdb) list
112
113       vstate = (vorbis_state_t *)state->private;
114
115       if (vstate->pages == NULL) return NULL;
116
117       page = vstate->pages->page;
118       vstate->pages = vstate->pages->next;
119
120       return page;
121     }
(gdb) print *vstate
$1 = {oy = {data = 0x0, storage = 0, fill = 0, returned = 0, unsynced = 0, headerbytes = 0, bodybytes = 0}, serialno = 0,
  old_granulepos = 0, first_page = 1, samplerate = 0, pages = 0x6c727473}
(gdb) print *vstate->pages
Cannot access memory at address 0x6c727473.
(gdb) print vstate->pages
$2 = (vorbis_page_t *) 0x6c727473
(gdb) bt
#0  vorbis_page_out (state=0x80521a8) at vorbis.c:117
#1  0x8049d4d in main (argc=6, argv=0xbffffc34) at ogmmerge.c:423
(gdb)

Clearly vstate->pages contains a bogus pointer. Perhaps it's used
uninitialized??

The files I'm using are the same ones I uploaded to mphq as
vorbismux-fail.tar.gz. foo.avi was generated by mencoder and bar.ogg
by latest release of oggenc.

Rich