Closed Bug 333392 Opened 14 years ago Closed 13 years ago

false positive [@ nsXBLService::GetBinding - nsXBLService::BindingReady]

Categories

(Core :: XBL, defect)

x86
Linux
defect
Not set

Tracking

()

RESOLVED WONTFIX

People

(Reporter: timeless, Unassigned)

References

(Blocks 1 open bug, )

Details

(Keywords: coverity, crash)

Crash Data

Attachments

(1 file)

found by coverity

i don't think this crash can actually happen, and perhaps coverity should deal with it better.

whether or not someone wants to fix the code, the comment on 829 needs to move *after* the NS_IMETHODIMP nsXBLService::BindingReady function which is *not* internal.
Attached patch Should assertSplinter Review
The scan warning is bogus, and this assert should make clear why.
Attachment #217804 - Flags: superreview?(bugmail)
Attachment #217804 - Flags: review?(bugmail)
Status: UNCONFIRMED → NEW
Ever confirmed: true
Attachment #217804 - Flags: superreview?(bugmail)
Attachment #217804 - Flags: superreview+
Attachment #217804 - Flags: review?(bugmail)
Attachment #217804 - Flags: review+
Checked in the assert; the bug as a whole should probably be wontfix or something...
Per comment #2, then...
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → WONTFIX
Crash Signature: [@ nsXBLService::GetBinding - nsXBLService::BindingReady]
You need to log in before you can comment on or make changes to this bug.