Closed
Bug 507379
Opened 15 years ago
Closed 15 years ago
hostname from SSL_RevealURL is null checked too late [@ nsDependentCString - nsNSSBadCertHandler]
Categories
(Core :: Security: PSM, defect)
Core
Security: PSM
Tracking
()
RESOLVED
FIXED
mozilla1.9.3a1
People
(Reporter: timeless, Assigned: timeless)
References
(Blocks 1 open bug, )
Details
(Keywords: coverity, crash)
Crash Data
Attachments
(1 file)
|
1.15 KB,
patch
|
KaiE
:
review+
|
Details | Diff | Splinter Review |
3168 nsNSSBadCertHandler(void *arg, PRFileDesc *sslSocket)
3200 char *hostname = SSL_RevealURL(sslSocket);
3201 charCleaner hostnameCleaner(hostname);
3202 nsDependentCString hostString(hostname);
3214 if (hostname && hostname[0] &&
|
||
Comment 2•15 years ago
|
||
Comment on attachment 391702 [details] [diff] [review]
patch
[Checkin: Comment 3]
r=kaie
Thanks for the patch
Attachment #391702 -
Flags: review?(kaie) → review+
|
|
||
Updated•15 years ago
|
Keywords: checkin-needed
|
||
Comment 3•15 years ago
|
||
Comment on attachment 391702 [details] [diff] [review]
patch
[Checkin: Comment 3]
http://hg.mozilla.org/mozilla-central/rev/525d2bb69a7c
Attachment #391702 -
Attachment description: patch → patch
[Checkin: Comment 3]
|
|
||
Updated•15 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.3a1
|
||
Updated•13 years ago
|
Crash Signature: [@ nsDependentCString - nsNSSBadCertHandler]
|
||
Updated•6 years ago
|
Blocks: coverity-analysis
You need to log in
before you can comment on or make changes to this bug.
Description
•