[crasher]crashes when changing IME

VERIFIED FIXED in M12

Status

()

P3
major
VERIFIED FIXED
19 years ago
9 years ago

People

(Reporter: David.R.Gardiner, Assigned: ftang)

Tracking

({inputmethod})

Trunk
x86
Windows NT
inputmethod
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [PDT-])

(Reporter)

Description

19 years ago
I'm using the NT5 IMEs to test out typing in Chinese Simplified into
web forms.

If you type something in English, then change to the Chinese IME (via the icon
in the status bar tray) you can enter Chinese chars.

When you change back to English, viewer crashes.

The problem seems to be an empty string doesn't convert very well to UniCode.

At around Line 3770 in widget\src\windows\nsWindow.cpp, this code has a problem
if the string is empty (which seems to happen when you do that language switch).

the application bombs when it tries to append a null to the string, but because
the string was empty, no space was allocated in the first place.

-dave
(Assignee)

Updated

19 years ago
Status: NEW → ASSIGNED
Target Milestone: M12
(Assignee)

Updated

19 years ago
Summary: Viewer crashes when changing IME → [dogfood][crashser]Viewer crashes when changing IME

Updated

19 years ago
Whiteboard: [PDT-]

Comment 1

19 years ago
Putting on PDT- radar.  Viewer is a tool.  Not a product for dogfood.  Please
use the latest Mozilla build at: http://www.mozilla.org/binaries.html

Thanks!

Updated

19 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 19 years ago
Resolution: --- → WONTFIX

Comment 2

19 years ago
I18N team does not have the bandwidth to provide IME support in viewer.
We're only supporting apprunner/mozilla.exe.  If this problem occurs in
apprunner/mozilla.exe, please report it as another bug.
Removing "[dogfood]" from Summary and marking WONTFIX.

Updated

19 years ago
Summary: [dogfood][crashser]Viewer crashes when changing IME → [crasher]Viewer crashes when changing IME
(Assignee)

Updated

19 years ago
Status: RESOLVED → REOPENED
Summary: [crasher]Viewer crashes when changing IME → [crasher]crashes when changing IME
(Assignee)

Updated

19 years ago
Status: REOPENED → ASSIGNED
(Assignee)

Updated

19 years ago
Resolution: WONTFIX → ---
(Assignee)

Comment 3

19 years ago
This is a valid bug. There are nothing related to viewer. Take the word viewer
from the summary. Reopen this bug. This is a crash bug. It just happen that the
reporter hit the crash in viewer. We can reproduce the same thing in
apprunner....

It should not be a [PDT-]. It is a crashing bug.
(Assignee)

Comment 4

19 years ago
it is a valid bug. Very good catch.
(Assignee)

Updated

19 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 19 years ago19 years ago
Resolution: --- → FIXED
(Assignee)

Comment 5

19 years ago
fix and check in.

Updated

19 years ago
Status: RESOLVED → VERIFIED

Comment 6

19 years ago
I verified this in 120108 Win32 build.
Keywords: inputmethod
You need to log in before you can comment on or make changes to this bug.