Closed
Bug 248435
Opened 20 years ago
Closed 20 years ago
CERT_DecodeGeneralName not in public header file
Categories
(NSS :: Libraries, defect, P2)
Tracking
(Not tracked)
RESOLVED
FIXED
3.10
People
(Reporter: nelson, Assigned: wtc)
References
Details
Attachments
(1 file)
|
(deleted),
patch
|
rrelyea
:
review+
julien.pierre
:
superreview+
|
Details | Diff | Splinter Review |
The fix for bug 122712 exported CERT_DecodeGeneralName among other symbols,
but it left the declaration of that function in a private header file.
This has apparently been blocking progress on PSM bug 230655 since February.
But apparently, John didn't file an NSS bug about this. :(
|
Reporter | |
Comment 1•20 years ago
|
||
Nominating this for 3.9.3
Priority: -- → P2
Target Milestone: --- → 3.9.3
|
Assignee | |
Comment 2•20 years ago
|
||
Moved to 3.10, unless Julien wants the fix in 3.9.4.
Severity: blocker → normal
Status: NEW → ASSIGNED
Target Milestone: 3.9.3 → 3.10
|
|
Assignee | |
Comment 3•20 years ago
|
||
Moved the declarations of CERT_DecodeGeneralName
and CERT_EncodeGeneralName (also exported) from
the private header genname.h to the public header
cert.h.
xconst.c uses these two functions and therefore
needs to include cert.h now.
|
|
Assignee | |
Updated•20 years ago
|
Attachment #164211 -
Flags: superreview?(julien.pierre.bugs)
Attachment #164211 -
Flags: review?(rrelyea0264)
|
||
Updated•20 years ago
|
Attachment #164211 -
Flags: superreview?(julien.pierre.bugs) → superreview+
|
|
Assignee | |
Comment 4•20 years ago
|
||
Patch checked in on the NSS trunk (NSS 3.10).
Checking in cert.h;
/cvsroot/mozilla/security/nss/lib/certdb/cert.h,v <-- cert.h
new revision: 1.48; previous revision: 1.47
done
Checking in genname.h;
/cvsroot/mozilla/security/nss/lib/certdb/genname.h,v <-- genname.h
new revision: 1.9; previous revision: 1.8
done
Checking in xconst.c;
/cvsroot/mozilla/security/nss/lib/certdb/xconst.c,v <-- xconst.c
new revision: 1.8; previous revision: 1.7
done
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
|
||
Updated•20 years ago
|
Attachment #164211 -
Flags: review?(rrelyea) → review+
You need to log in
before you can comment on or make changes to this bug.
Description
•