Closed
Bug 1210380
Opened 9 years ago
Closed 9 years ago
[Coverity 1291990] Code can never be reached because of a logical contradiction in ssl3_HandleServerNameXtn
Categories
(NSS :: Libraries, defect)
NSS
Libraries
Tracking
(firefox44 affected)
RESOLVED
FIXED
3.21
Tracking | Status | |
---|---|---|
firefox44 | --- | affected |
People
(Reporter: ttaubert, Assigned: ttaubert)
References
(Blocks 1 open bug)
Details
In ssl3_HandleServerNameXtn(), |listLenBytes| as returned by ssl3_ConsumeHandshakeNumber() is at least 1. That means that |listCount| is always at least incremented once and we can remove:
> if (!listCount) {
> return SECFailure; /* nothing we can act on */
> }
Assignee | ||
Comment 1•9 years ago
|
||
Patch at: https://codereview.appspot.com/269930043
Assignee: nobody → ttaubert
Status: NEW → ASSIGNED
Flags: needinfo?(martin.thomson)
Flags: needinfo?(ekr)
Comment 3•9 years ago
|
||
Checked in. Fixing the other thing could have compatibility risks so we can do that separately. (I don't think that it does, but a separate commit is still wise.) https://hg.mozilla.org/projects/nss/rev/5fcf4660d2af
Flags: needinfo?(martin.thomson)
Target Milestone: --- → 3.21
Assignee | ||
Comment 4•9 years ago
|
||
Should we resolve this as fixed and do the rest over in bug 1210484?
You need to log in
before you can comment on or make changes to this bug.
Description
•