Closed Bug 389501 Opened 19 years ago Closed 18 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: 18 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: