Closed
Bug 394254
Opened 18 years ago
Closed 18 years ago
nsInstallTrigger::HandleContent needs to nullcheck referringURI
Categories
(Core Graveyard :: Installer: XPInstall Engine, defect)
Core Graveyard
Installer: XPInstall Engine
Tracking
(Not tracked)
VERIFIED
DUPLICATE
of bug 393309
People
(Reporter: Biesinger, Assigned: Biesinger)
Details
(Keywords: crash, topcrash)
Attachments
(1 file)
1.24 KB,
patch
|
Details | Diff | Splinter Review |
There can be a null referrer property, so the code needs to nullcheck it before using it
this doesn't exactly match the line numbers from the topcrash breakpad report but is what I can reproduce.
Comment 1•18 years ago
|
||
That was fixed in bug 393309
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → DUPLICATE
Assignee | ||
Comment 2•18 years ago
|
||
Not quite sure whether passing an empty host in this case is the right thing to do...
Assignee: nobody → cbiesinger
Status: RESOLVED → ASSIGNED
Attachment #278869 -
Flags: superreview?(dveditz)
Attachment #278869 -
Flags: review?(mano)
Resolution: DUPLICATE → ---
Assignee | ||
Comment 3•18 years ago
|
||
Comment on attachment 278869 [details] [diff] [review]
patch
that's what I get for loading cvsblame URIs from socorro
Attachment #278869 -
Flags: superreview?(dveditz)
Attachment #278869 -
Flags: review?(mano)
Assignee | ||
Updated•18 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 18 years ago → 18 years ago
Resolution: --- → DUPLICATE
Updated•18 years ago
|
Status: RESOLVED → VERIFIED
Updated•9 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•