121565 - clicks4u.com - site spawns infinite timers locking up Browser

Status: RESOLVED FIXED

(Tech Evangelism Graveyard :: English US, defect, P3)

Description

[Bob Clary [:bc] (inactive)]

from bug 120475#c14

------- Additional Comment #14 From Stuart Parmenter 2002-01-23 19:13 -------

The problem is in
http://www.clicks4u.com/watermark.js

at the bottom, it calls setInterval(), which sucks because it really wants 
setTimeout, but that isn't what is killing us.

the code:
  if (navigator.appName == "Netscape") {
    if (navigator.appVersion.indexOf("6") == -1) {
      wm = document.jsbrand;
    }
  }

should be checking for nav4

it should be doing (indexOf("5") != -1) for NS6/moz and then using |wm = 
document.getElementById("jsbrand")|

What happens is it fails to set wm, then calls setInterval again.
So the first time through, we now have 2 repeating timers... then 4.. 8... 16...

Unless JS can detect this and stop it, there isn't much we can do.

==

Contact Info: [e-mail]

Comment 1

[Bob Clary [:bc] (inactive)]

This problem is not Mozilla specific. If you run IE long enough the timers send
the CPU to 100%, so there is good motivation for them to fix the script. ->
Change Summary to reflect any browser.

An example of this problem appears in http://www.jimamidon.com/affiliates-quick.html

Contacted late January, follow up in March.

Comment 2

[Phil Schwartau]

Note there are other evangelism issues at this site.
|document.all|is used without any Moz/N6 codepath. 

In fact, |document.all| is in functions that get called by 
error handlers at the site. In Moz/N6, this can cause infinite
recursion if the error handler is ever triggered.

See http://bugzilla.mozilla.org/show_bug.cgi?id=120475#c2
and the discussion following -

Comment 3

[Zach Lipton [:zach]]

Mass reassign of all tech-evangelism us general bugs assigned to bc to 
doron except bc's P1 bugs. You may search for this mass reassign (it is 
305 bugs) by searching for the keyword 'greeneggsandham'

Comment 4

[Bob Clary [:bc] (inactive)]

not fixed.

Comment 5

[Matthias Versen [:Matti]]

dead domain