If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

crash [@ vorbis_dsp_clear] if !vi

VERIFIED WORKSFORME

Status

()

Core
Audio/Video
--
critical
VERIFIED WORKSFORME
7 years ago
6 years ago

People

(Reporter: timeless, Unassigned)

Tracking

({coverity, crash})

Trunk
coverity, crash
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(crash signature)

(Reporter)

Description

7 years ago
235 void vorbis_dsp_clear(vorbis_dsp_state *v){
236   int i;
237   if(v){
238     vorbis_info *vi=v->vi;
this code worries that vi might be null:
239     codec_setup_info *ci=(codec_setup_info *)(vi?vi->codec_setup:NULL);
240     private_state *b=(private_state *)v->backend_state;
241 
242     if(v->pcm){
this code simply crashes:
243       for(i=0;i<vi->channels;i++)
vi can't be NULL if pcm is non-NULL. vi->channels is required to allocate pcm in the first place.
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → INVALID
(Reporter)

Comment 2

7 years ago
ok, kinda unfortunate, i'm likely to be switching to a new coverity in the future at which point any annotations i make in the current one will be lost.
Status: RESOLVED → VERIFIED
Resolution: INVALID → WORKSFORME
(Assignee)

Updated

6 years ago
Crash Signature: [@ vorbis_dsp_clear]
You need to log in before you can comment on or make changes to this bug.