Closed
Bug 13952
Opened 25 years ago
Closed 25 years ago
The escape sequences unicode in wallet.properties can't display correctly
Categories
(SeaMonkey :: General, defect, P3)
Tracking
(Not tracked)
VERIFIED
FIXED
M14
People
(Reporter: rchen, Assigned: morse)
References
Details
The escape sequences unicode in wallet.properties can't display correctly.
I put the string \u30d5\u30a1\u30a4\u30eb which is "File" in Japanese to
firstPassword in wallet.properties. When I ran apprunner and go to
edit/wallet/select/Safe From Fill, the dialog shows .... for these characters.
This test was on 09/15/99 build on Japanese NT.
Updated•25 years ago
|
Assignee: saari → morse
Comment 2•25 years ago
|
||
This is in the wallet area. Reassign to morse first. Morse, please contact
either tao or rchen for help. Thanks
Assignee | ||
Updated•25 years ago
|
Status: NEW → ASSIGNED
Target Milestone: M14
Assignee | ||
Comment 3•25 years ago
|
||
Tao or rchen, HELP. What do I do here? How can I reproduce this? How do I fix
it?
Hi Steve, can you point out where the code is (file and function)? I would like
to take a look at the code and see if I can fix it.
Assignee | ||
Comment 5•25 years ago
|
||
wallet.properties is in extensions/wallet/src. The code that reads that
properties file is in extensions/wallet/src/wallet.cpp. The way to trigger it
is to select the menu item of edit/wallet/display-wallet-contents. The function
in wallet.cpp is Wallet_SetKey.
Comment 6•25 years ago
|
||
I take a look at the code and here is my finding...
1. The function declaration
PUBLIC char*
Wallet_Localize(char* genericString) {
is BAD BAD. Change it to PUBLIC PRUnichar* . char* is not good for non ASCII
data.
2. The implementation of Wallet_Localize seems ok untill
714 ret = bundle->GetStringFromName(ptrtmp, &ptrv);
this line is probably ok.But the next line is bad
724 return v.ToNewCString();
The ToNewCSting function will damage the unicode data and you should not use it,
instead, you should return thr PRUnichar* by calling GetUnicode().
You should not use char* as your data type, which is BAD BAD. You should use
PRUnichar* as your data type as possible as you can. And you will find most of
the Gecko API use PRUnichar* but not char*.
ToNewCString will damage data. Don't use it!!!!
Assignee | ||
Updated•25 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 7•25 years ago
|
||
Now fixed in wallet, single signon, and cookies.
Updated•20 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•