Henrik Gemal Reporter
	Description • 23 years ago

going to:
http://www.bgbank.dk/link/konkurrence
crashes mozilla build 20011106

TB37679342Y

	Diego Biurrun
	Comment 1 • 23 years ago

Same here with 2001110612 under Linux. No talkback triggered.

	R.K.Aa.
	Comment 3 • 23 years ago

adding to summary

	av (gone)
	Comment 5 • 23 years ago

--> JavaScript based on the stack trace.

	Phil Schwartau
	Comment 6 • 23 years ago

I get the same stack trace as Andrei with a debug build 2001-11-06 on WinNT.
cc'ing Brendan on this one - 

	Brendan Eich [:brendan] Assignee
	Comment 7 • 23 years ago

Argh, js_strchr is toxic nowadays, given dependent strings.  This should be
fixed ASAP for 0.9.6.

/be

	Brendan Eich [:brendan] Assignee
	Comment 8 • 23 years ago

khanson, jband, shaver: please r= and sr= for 0.9.6 so Asa can a=.  Thanks,

/be

	John Bandhauer
	Comment 9 • 23 years ago

Comment on attachment 57043 [details] [diff] [review]
minimal proposed fix for 0.9.6

sr=jband

	Mike Shaver (:shaver -- probably not reading bugmail closely)
	Comment 10 • 23 years ago

Comment on attachment 57043 [details] [diff] [review]
minimal proposed fix for 0.9.6

Yeah, r=shaver, but let's keep it open for a better fix post-0.9.6, OK?

	Brendan Eich [:brendan] Assignee
	Comment 11 • 23 years ago

shaver: le plus certainement, better fix may keep js_strnchr for the safe
nul-terminated uses, but avoid flattening in do_replace by providing a
js_FindChar or some such.

Asa: a= for checkin today?  Thanks,

/be

	Asa Dotzler [:asa]
	Comment 12 • 23 years ago

a=asa (on behalf of drivers) for checkin to 0.9.6

	Brendan Eich [:brendan] Assignee
	Comment 13 • 23 years ago

Checked in for 0.9.6, lowering severity and setting priority for 0.9.7.

/be

	Phil Schwartau
	Comment 14 • 23 years ago

QA note: 

Verified Fixed with the minimal fix that has been checked in so far.
Using binaries 20011111xx Linux, 20011113xx WinNT, 20011115xx Mac9.1.
Site now loads without crashing (whether or not Flash plug-in is installed).

	Mike Shaver (:shaver -- probably not reading bugmail closely)
	Comment 16 • 23 years ago

Comment on attachment 58502 [details] [diff] [review]
proposed larger fix, good for 0.9.7 trunk

r=shaver.

	John Bandhauer
	Comment 17 • 23 years ago

Comment on attachment 58502 [details] [diff] [review]
proposed larger fix, good for 0.9.7 trunk

sr=jband

	Brendan Eich [:brendan] Assignee
	Comment 18 • 23 years ago

Fix checked in, thanks.

/be

	Phil Schwartau
	Comment 19 • 23 years ago

Verified Fixed using Mozilla trunk binaries 20011120xx WinNT, Linux, Mac 9.1
Site now loads without crashing (whether or not Flash plug-in is installed).