Closed
Bug 394271
Opened 18 years ago
Closed 18 years ago
two public SSL functions require PRFD* to point to SSL layer
Categories
(NSS :: Libraries, defect, P2)
Tracking
(Not tracked)
RESOLVED
FIXED
3.11.8
People
(Reporter: nelson, Assigned: nelson)
Details
Attachments
(1 file, 1 obsolete file)
2.98 KB,
patch
|
julien.pierre
:
review+
wtc
:
superreview+
|
Details | Diff | Splinter Review |
libSSL's methods that get called through NSPRs PRIOMethods table require
that the PRFileDescriptor (PRFD) argument must be the SSL layer's PRFD.
Nearly all other SSL functions that have PRFD arguments permit, but do not
require, that the PRFD be the SSL layer's PRFD. They will work even if
called with the PRFD from another layer in the stack, PROVIDED that there
is only one SSL layer in the stack (which is the usual case).
However, there are two functions, new in NSS 3.11, that require that the
PRFD argument be precisely the SSL layer PRFD. They are
SSL_ReHandshakeWithTimeout and SSL_ForceHandshakeWithTimeout.
These are the only two callers to ssl_SetTimeout, also new in NSS 3.11.
They require that the PRFD be the SSL layer's, rather than another layer
in the stack, because ssl_SetTimeout calls ssl_GetPrivate to get the
address of the sslSocket structure. Most other non-PRIOMethod functions
call ssl_FindSocket instead of ssl_GetPrivate for this purpose.
ssl_GetPrivate requires that its argument be an SSL layer PRFD, while
ssl_FindSocket searches the stack for the SSL layer PRFD if the PRFD
address passed to it is not an SSL layer PRFD.
I propose to
a) move ssl_SetTimeout to sslsecur.c, to be in the same file with its callers,
b) make it a static function, and
c) make it call ssl_FindSocket instead of ssl_GetPrivate.
Assignee | ||
Comment 1•18 years ago
|
||
This patch does what I proposed in comment 0.
Attachment #278903 -
Flags: superreview?(wtc)
Attachment #278903 -
Flags: review?(julien.pierre.boogz)
Assignee | ||
Updated•18 years ago
|
OS: Windows XP → All
Priority: -- → P2
Hardware: PC → All
Target Milestone: --- → 3.11.8
Comment 2•18 years ago
|
||
Comment on attachment 278903 [details] [diff] [review]
patch v1
r=wtc. We also need to remove the function declaration
in sslimpl.h.
Attachment #278903 -
Flags: superreview?(wtc) → review+
Assignee | ||
Comment 3•18 years ago
|
||
I'm asking Wan-Teh for SR again. Wan-Teh, if you are unable to set the
SR flag, please don't cancel the SR request, and instead let's press this
issue to get it fixed.
Attachment #278903 -
Attachment is obsolete: true
Attachment #278983 -
Flags: superreview?(wtc)
Attachment #278983 -
Flags: review?
Attachment #278903 -
Flags: review?(julien.pierre.boogz)
Assignee | ||
Updated•18 years ago
|
Attachment #278983 -
Flags: review? → review?(julien.pierre.boogz)
Comment 4•18 years ago
|
||
Comment on attachment 278983 [details] [diff] [review]
patch v2, removes func from header file
I can't set superreview+ on this patch. The dropdown box for
superreview only has empty (cancel request) and '?'.
This patch is correct.
Updated•18 years ago
|
Attachment #278983 -
Flags: review?(julien.pierre.boogz) → review+
Updated•18 years ago
|
Attachment #278983 -
Flags: superreview?(wtc) → superreview+
Assignee | ||
Comment 5•18 years ago
|
||
On trunk:
Checking in sslimpl.h; new revision: 1.60; previous revision: 1.59
Checking in sslsecur.c; new revision: 1.39; previous revision: 1.38
Checking in sslsock.c; new revision: 1.54; previous revision: 1.53
Assignee | ||
Comment 6•18 years ago
|
||
On branch:
Checking in sslimpl.h; new revision: 1.42.2.13; previous revision: 1.42.2.12
Checking in sslsecur.c; new revision: 1.34.2.5; previous revision: 1.34.2.4
Checking in sslsock.c; new revision: 1.44.2.9; previous revision: 1.44.2.8
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•