Closed
Bug 240300
Opened 21 years ago
Closed 21 years ago
IDN: Domain in "Connection refused" dialog poorly encoded
Categories
(Core :: Internationalization, defect)
Core
Internationalization
Tracking
()
RESOLVED
FIXED
People
(Reporter: crispin, Assigned: jshin1987)
Details
(Keywords: intl)
Attachments
(2 files)
8.01 KB,
image/png
|
Details | |
1.41 KB,
patch
|
Biesinger
:
review+
darin.moz
:
superreview+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7b) Gecko/20040329 Galeon/1.3.14
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7b) Gecko/20040329 Galeon/1.3.14
Forwarded from http://bugzilla.gnome.org/show_bug.cgi?id=139341
If an IDN website isn't running, then the "Connection refused" dialog encodes
the URL incorrectly, and instead of getting the correct UTF-8 chars, you get
garbage.
Reproducible: Always
Steps to Reproduce:
1. Go to an IDN site that is in DNS, but isn't running (the reporter of the
original galeon bug gave http://刘莉.dyndns.info/ as an example (but
it may be running sometimes))
2. wait for the "Connection refused" dialog
Actual Results:
The dialog box has a very badly encoded dialog box.
Expected Results:
The dialog box should have the correctly displayed URL
Reporter | ||
Comment 1•21 years ago
|
||
Assignee | ||
Comment 2•21 years ago
|
||
Re comment #0:
Please, make sure to set View | Character Encoding to UTF-8 before typing
non-ASCII strings in bugzilla.
Comment 3•21 years ago
|
||
http://刘莉.dyndns.info/
(With UTF-8 Encoding)
Assignee | ||
Comment 4•21 years ago
|
||
Confirming and taking.
UTF-8 strings are zero-padded when converting to UTF-16.
Assignee | ||
Comment 5•21 years ago
|
||
re comment #4: that is, 'string' is used where 'AUTF8String' should be used or
something like that.
Status: NEW → ASSIGNED
OS: Linux → All
Hardware: PC → All
Comment 6•21 years ago
|
||
the symptom of comment 4 sounds more like AssignWithConversion is used where
CopyUTF8toUTF16 should be used. |string| instead of |AUTF8String| would cause
stripping of the high byte.
Assignee | ||
Comment 7•21 years ago
|
||
You're absolutely right. The screenshot confirmed that.
Assignee | ||
Comment 8•21 years ago
|
||
It seems like nhotta missed two places while making his patch for bug 122705.
Assignee | ||
Comment 9•21 years ago
|
||
Comment on attachment 150439 [details] [diff] [review]
patch
asking for r/sr
Attachment #150439 -
Flags: superreview?(darin)
Attachment #150439 -
Flags: review?(cbiesinger)
Updated•21 years ago
|
Attachment #150439 -
Flags: review?(cbiesinger) → review+
Updated•21 years ago
|
Attachment #150439 -
Flags: superreview?(darin) → superreview+
Assignee | ||
Comment 10•21 years ago
|
||
fixed on the trunk
Status: ASSIGNED → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•