Closed
Bug 265373
Opened 21 years ago
Closed 21 years ago
compiler warnings in pki3hack on Solaris AMD64
Categories
(NSS :: Libraries, defect, P2)
Tracking
(Not tracked)
RESOLVED
FIXED
3.9.4
People
(Reporter: julien.pierre, Assigned: julien.pierre)
Details
(Keywords: regression)
Attachments
(1 file)
1.00 KB,
patch
|
Details | Diff | Splinter Review |
"pki3hack.c", line 102: warning: implicit function declaration: NSSRWLock_LockWrite
"pki3hack.c", line 104: warning: implicit function declaration:
NSSRWLock_UnlockWrite
"pki3hack.c", line 1045: warning: implicit function declaration: NSSRWLock_LockRead
"pki3hack.c", line 1049: warning: implicit function declaration:
NSSRWLock_UnlockRead
The fix is to include nssrwlk.h , and already checked in on the tip.
Assignee | ||
Comment 1•21 years ago
|
||
Assignee | ||
Comment 2•21 years ago
|
||
Checking in pki3hack.c;
/cvsroot/mozilla/security/nss/lib/pki/pki3hack.c,v <-- pki3hack.c
new revision: 1.79.14.3; previous revision: 1.79.14.2
done
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•21 years ago
|
Priority: -- → P2
Target Milestone: --- → 3.9.4
Updated•21 years ago
|
OS: SunOS → Solaris
Comment 3•21 years ago
|
||
This bug was introduced in rev. 1.79.14.2 of pki3hack.c,
a fix for bug 252702. So the "bug introduced" version
is NSS 3.9.3.
Fortunately both NSSRWLock_LockWrite and NSSRWLock_LockRead
return void, so we don't need to respin NSS 3.9.3 for this.
Version: 3.9.2 → 3.9.3
Updated•21 years ago
|
Severity: normal → trivial
Keywords: regression
Assignee | ||
Comment 5•21 years ago
|
||
timeless, NSS bugs are filed only against NSS releases, not browser releases. If
you want this fix in a client product, the client product needs to move up to a
tag that includes it. This was fixed in 3.9.4, so pulling NSS with a tag of
NSS_3_9_4_RTM would include this fix .
Comment 6•21 years ago
|
||
I put this fix in NSS_CLIENT_TAG (Mozilla 1.8 Beta)
yesterday.
You need to log in
before you can comment on or make changes to this bug.
Description
•