Closed
Bug 334193
Opened 18 years ago
Closed 18 years ago
nsContinueDespiteCertError's default case should set rv
Categories
(Core :: Security: PSM, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: timeless, Assigned: timeless)
References
(Blocks 1 open bug, )
Details
(Keywords: coverity)
Attachments
(1 obsolete file)
currently both code paths result in PR_FALSE being returned, but purify and valgrind would complain about the jump
Attachment #218650 -
Flags: superreview?(bzbarsky)
Attachment #218650 -
Flags: review?(kengert)
Updated•18 years ago
|
Attachment #218650 -
Flags: superreview?(bzbarsky) → superreview+
Comment 2•18 years ago
|
||
Comment on attachment 218650 [details] [diff] [review] set rv r=kengert patch can't hurt and adds clarity
Attachment #218650 -
Flags: review?(kengert) → review+
Comment on attachment 218650 [details] [diff] [review] set rv mozilla/security/manager/ssl/src/nsNSSIOLayer.cpp 1.108
Attachment #218650 -
Attachment is obsolete: true
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Updated•6 years ago
|
Blocks: coverity-analysis
You need to log in
before you can comment on or make changes to this bug.
Description
•