[Vorbis-dev] Add vorbis_dsp_init() ?

Conrad Parker conrad at metadecks.org
Sat Mar 14 22:18:51 PDT 2009


Hi,

This mozilla bug report is a crash triggered by a Vorbis file with
corrupt headers:

https://bugzilla.mozilla.org/show_bug.cgi?id=481601

The patch to fix the crash adds a new vorbis_dsp_init() function to
libvorbis, and calls
that from fs_vorbis_init() in libfishsound:

https://bug481601.bugzilla.mozilla.org/attachment.cgi?id=366150

The public function it adds is:

void vorbis_dsp_init(vorbis_dsp_state *v){
  memset(v,0,sizeof(*v));
}

Is it ok to commit the libvorbis patch to svn.xiph.org/trunk/vorbis?

Conrad.


More information about the Vorbis-dev mailing list