Closed Bug 386164 Opened 18 years ago Closed 17 years ago

crashreporter expects "Vendor" to be existing

Categories

(Toolkit :: Crash Reporting, defect)

x86
All
defect
Not set
normal

Tracking

()

RESOLVED DUPLICATE of bug 396209

People

(Reporter: wolfiR, Unassigned)

Details

crashreporter expects "Vendor" to be existing Vendor is an optional field in application.ini but if it's not set the crash reporter on Windows saves to the registry key Software\Crash Reporter and therefore treats Vendor to be mandatory.
I wonder if http://mxr.mozilla.org/seamonkey/source/toolkit/airbag/client/crashreporter_win.cpp#699 was intended to fall back to Software\Mozilla\Crash Reporter but doesn't do it because Vendor could be found in gQueryParameters but is empty?
Aehm, forgot my proposal: Fall back to product/Name/ProductName if Vendor is empty/not defined.
Looks like we should probably be testing the string here to see if it's an empty string: http://mxr.mozilla.org/seamonkey/source/toolkit/xre/nsAppRunner.cpp#2524 But this might be obviated by bent's fix over on bug 396486.
Rolled into the patch on bug 396209.
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.