Closed Bug 1231124 Opened 4 years ago Closed 4 years ago
[Static Analysis][Uninitialized pointer field] Function Audio
Timeline Event from Audio Event Timeline .h
The Static Analysis tool Coverity added that mCurve is not initialized in constructor leading to an undefined value for pointer.
Hello Johnny, Could you please take a look other this patch? THX
Attachment #8696679 - Flags: review?(jst)
Comment on attachment 8696679 [details] [diff] [review] Bug 1231124 .diff Use nullptr, not NULL, r=me As far as I see, the code is safe here though, since mCurve is always set if mType is AudioTimelineEvent::SetValueCurve
Attachment #8696679 - Flags: review?(jst) → review+
Thanks for the review, i've updated the patch.
Attachment #8696679 - Attachment is obsolete: true
You need to log in before you can comment on or make changes to this bug.