Closed Bug 389501 Opened 17 years ago Closed 17 years ago

fetch malware data with the url classifier

Categories

(Toolkit :: Safe Browsing, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: dcamp, Assigned: dcamp)

References

Details

Attachments

(1 file, 1 obsolete file)

This patch adds a malware warden that keeps the goog-malware-sha128 table up to date.
OS: Mac OS X → All
Hardware: PC → All
Version: unspecified → Trunk
Attached patch v1 (obsolete) — Splinter Review
apparently I didn't attach this.
Attachment #274549 - Flags: review?(tony)
Attachment #274549 - Attachment is patch: true
Attachment #274549 - Attachment mime type: application/octet-stream → text/plain
Comment on attachment 274549 [details] [diff] [review]
v1

In shutdown, you need to remove the pref observers or we leak as browser windows close.  dbaron added this in bug 386665.

r+ with that change.
Attachment #274549 - Flags: review?(tony) → review+
Attached patch v2Splinter Review
This removes the prefs observers in shutdown.

I also changed the pref's default to 'true', so I'm re-requesting review.
Attachment #274549 - Attachment is obsolete: true
Attachment #275924 - Flags: review?(tony)
Attachment #275924 - Flags: review?(tony) → review+
Checking in browser/app/profile/firefox.js;
/cvsroot/mozilla/browser/app/profile/firefox.js,v  <--  firefox.js
new revision: 1.193; previous revision: 1.192
done
Checking in browser/components/safebrowsing/content/application.js;
/cvsroot/mozilla/browser/components/safebrowsing/content/application.js,v  <--  application.js
new revision: 1.13; previous revision: 1.12
done
RCS file: /cvsroot/mozilla/browser/components/safebrowsing/content/malware-warden.js,v
done
Checking in browser/components/safebrowsing/content/malware-warden.js;
/cvsroot/mozilla/browser/components/safebrowsing/content/malware-warden.js,v  <--  malware-warden.js
initial revision: 1.1
done
Checking in browser/components/safebrowsing/content/sb-loader.js;
/cvsroot/mozilla/browser/components/safebrowsing/content/sb-loader.js,v  <--  sb-loader.js
new revision: 1.21; previous revision: 1.20
done
Checking in browser/components/safebrowsing/src/nsSafebrowsingApplication.js;
/cvsroot/mozilla/browser/components/safebrowsing/src/nsSafebrowsingApplication.js,v  <--  nsSafebrowsingApplication.js
new revision: 1.9; previous revision: 1.8
done
Checking in toolkit/components/url-classifier/content/listmanager.js;
/cvsroot/mozilla/toolkit/components/url-classifier/content/listmanager.js,v  <--  listmanager.js
new revision: 1.22; previous revision: 1.21
done
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Attachment #275924 - Attachment is patch: true
Attachment #275924 - Attachment mime type: application/octet-stream → text/plain
Blocks: 384941
Product: Firefox → Toolkit
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: