Closed
Bug 185167
Opened 22 years ago
Closed 21 years ago
Display of BMPString (UTF-8) encoded fields of certificate is bad
Categories
(Core Graveyard :: Security: UI, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: t8m, Assigned: jgmyers)
Details
Attachments
(3 files)
10.99 KB,
image/png
|
Details | |
1.47 KB,
application/octet-stream
|
Details | |
2.17 KB,
patch
|
KaiE
:
review+
sspitzer
:
superreview+
|
Details | Diff | Splinter Review |
When you try to display Details of certificate which has some fields with
non-ascii characters (encoded as BMPString). The display is wrong as it was
encoded with iso8859-1 encoding.
See attached screenshot.
Updated•22 years ago
|
Version: unspecified → 2.4
Assignee | ||
Comment 3•21 years ago
|
||
Assignee | ||
Updated•21 years ago
|
Attachment #138792 -
Attachment description: P → Proposed fix
Attachment #138792 -
Flags: review?(ssaux)
Comment 5•21 years ago
|
||
Comment on attachment 138792 [details] [diff] [review]
Proposed fix
I'm probably not the best person to give reviews on PSM. Kai Engert would be
the best person.
Attachment #138792 -
Flags: review?(ssaux) → review+
Assignee | ||
Updated•21 years ago
|
Attachment #138792 -
Flags: superreview?(scc)
Assignee | ||
Updated•21 years ago
|
Attachment #138792 -
Flags: superreview?(scc) → superreview?(sspitzer)
Comment 6•21 years ago
|
||
Comment on attachment 138792 [details] [diff] [review]
Proposed fix
sr=sspitzer, but can we also get kai to review?
Attachment #138792 -
Flags: superreview?(sspitzer)
Attachment #138792 -
Flags: superreview+
Attachment #138792 -
Flags: review?(kaie)
Attachment #138792 -
Flags: review+
Comment 7•21 years ago
|
||
Comment on attachment 138792 [details] [diff] [review]
Proposed fix
r=kaie
Attachment #138792 -
Flags: review?(kaie) → review+
Assignee | ||
Comment 8•21 years ago
|
||
Fix checked in.
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Comment 9•21 years ago
|
||
When I try to verify this bug, I still find one problem left, John.
After import of the certificate, everything's correct except for the copy of the
name of the certificate inside the title bar of the "Certificate Viewer"
windows, which incorrectly reads "Certificate Viewer:LudÄ_k RaÄiek" instead of
the proper czech characters.
As the original content of the cert CN is BMP, it has somewhere been correctly
converted to utf-8, but that string is later somewhere wrongly interpreted as
ISO-8859-1 before display.
Shall we reopen, or treat this in a separate bug ?
Just like bug 197009, it would be good to get this inside NSS 3.9.1, and then in
1.7.b.
I think this is even more important than bug 197009, because it seems to concern
all certs that have utf-8/BMP content, and by application of RFC3280, CA are
required to switch to the use of UTF-8 for certificates issued after 31/12/2003,
and this is respected by some people.
Assignee | ||
Comment 10•21 years ago
|
||
I would treat as a separate bug.
Comment 11•21 years ago
|
||
I tried to import the bug into FireFox 0.8 and the problem is still present.
Assignee | ||
Comment 12•21 years ago
|
||
the certificate titlebar viewing issue is bug 234856
Updated•8 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•