Closed Bug 334233 Opened 14 years ago Closed 14 years ago
_Script Error Reporter would use rv uninitialized if !message && !report
I don't think this is supposed to happen. I believe coverity has ways that you can hint to it about conditions you want, so we could either use that hint, or just set rv to NS_ERROR_NOT_AVAILABLE or something so that the code doesn't trigger the case (which seems cheaper to me than adding a coverity specific marker).
Note that the assert is needed, imo. The other change is optional.
Priority: -- → P4
Summary: NS_ScriptErrorReporter would use rv uninitialized if !message && !report → [FIX]NS_ScriptErrorReporter would use rv uninitialized if !message && !report
Target Milestone: --- → mozilla1.9alpha
Comment on attachment 218599 [details] [diff] [review] I think this is the way to go r+sr=jst
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.