In the AddonInstall constructor (for the XPI provider), if the install is a local file, this.hash is checked to see whether a hash of the file should be computed. aHash is passed as a parameter, but is never set as this.hash (for local installs only). Therefore, even if a hash is passed in, it will never be checked.
blocking2.0: --- → ?
Whiteboard: [rewrite] → [AddonsRewrite]
Version: unspecified → Trunk
Doesn't seem to be blocking any features in Firefox 4, would be nice to fix though.
blocking2.0: ? → -
Assignee: nobody → bmcbride
Status: NEW → ASSIGNED
Attachment #448656 - Flags: review?(dtownsend)
Comment on attachment 448656 [details] [diff] [review] One-line fix Can you add a test too please?
Attachment #448656 - Flags: review?(dtownsend) → review+
Fixed by the patch in bug 570200
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Flags: in-testsuite? → in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.3a5
Target Milestone: mozilla1.9.3a5 → mozilla1.9.3
Tinderbox looks greenish in our area. Marking as verified fixed based on automated test results.
Status: RESOLVED → VERIFIED
Target Milestone: mozilla1.9.3 → mozilla1.9.3a6
You need to log in before you can comment on or make changes to this bug.