When CurrentPage/findInPage(stringToFind, forward, matchCase) is invoked with stringToFind==null, then JVM crashes

VERIFIED FIXED

Status

Core Graveyard
Java APIs to WebShell
P3
critical
VERIFIED FIXED
17 years ago
5 years ago

People

(Reporter: Oleg Khokhlov, Assigned: Ashu Kulkarni)

Tracking

({crash})

Trunk
x86
Windows NT
crash

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

17 years ago
When CurrentPage/findInPage(java.lang.String stringToFind, boolean forward, 
boolean matchCase) method is invoked with stringToFind==null, then JVM crashes:

#
# HotSpot Virtual Machine Error, EXCEPTION_ACCESS_VIOLATION
#
# Error ID: 4F533F57494E13120E43505002B7
#
abnormal program termination

Build Date & Platform:
Mozilla M13, Webclient source from June,6; WinNT 4.0 with SP4.

Comment 1

17 years ago
M13 is pretty old. . . Have you tried a nightly build?

Comment 2

17 years ago
As I know, webclient is still not work with any
builds, exclude M13

Comment 3

17 years ago
Adding crash keyword.
Keywords: crash

Comment 4

17 years ago
And this one :-)

Gerv
Status: UNCONFIRMED → NEW
Ever confirmed: true

Comment 5

17 years ago
a
Status: NEW → ASSIGNED
(Reporter)

Comment 6

17 years ago
I tested it on M16, and JVM still crashes.

Comment 7

17 years ago
Find is Ashu's.
Assignee: edburns → ashuk
Status: ASSIGNED → NEW
(Assignee)

Comment 8

17 years ago
Created attachment 11831 [details] [diff] [review]
Diffs in CurrentPageImpl.cpp and jni_util.h
(Assignee)

Comment 9

17 years ago
Fixed. Checked in changes. Pls see attachment for patch. Checks to see if Find 
string is non-null before setting in the SearchContext object.
Status: NEW → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED
(Reporter)

Comment 10

16 years ago
I mark it as verified.
Status: RESOLVED → VERIFIED
Component: Java APIs to WebShell → Java APIs to WebShell
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.