Closed
Bug 273551
Opened 20 years ago
Closed 18 years ago
firefox windows installer writes invalid registry entries
Categories
(Firefox :: Installer, defect)
Tracking
()
RESOLVED
WORKSFORME
People
(Reporter: ezs, Unassigned)
Details
Attachments
(1 file, 2 obsolete files)
380 bytes,
text/plain;charset=UTF-16LE
|
Details |
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7.5) Gecko/20041107 Firefox/1.0 Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7.5) Gecko/20041107 Firefox/1.0 The Windows Installer writes two invalid entries to the Windows Registry: HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\Windows\CurrentVersion\Uninstall\Mozilla Firefox (1.0) | NoModify HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\Windows\CurrentVersion\Uninstall\Mozilla Firefox (1.0) | NoRepair The install creates these as DWORD values with just 0x00 in the value; to be truly valid they need a full DWORD. "NoModify"=dword:00000000 "NoRepair"=dword:00000000 This is a cosmetic issue - but causes problems with repackaging software. Reproducible: Always Steps to Reproduce: Install Firefox 1.0 on Windows XP Actual Results: Bad registry values are created. The Product functions perfectly. Expected Results: The expected behaviour is for the install to write correct DWORD values.
Reporter | ||
Comment 1•20 years ago
|
||
Reporter | ||
Comment 2•20 years ago
|
||
The attachment is snafued - so here's the cut and paste of a working registry: <snip> Windows Registry Editor Version 5.00 [HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\Windows\CurrentVersion\Uninstall\Mozilla Firefox (1.0)] "NoModify"=dword:00000000 "NoRepair"=dword:00000000 </snip>
Attachment #168139 -
Attachment mime type: text/plain → text/plain;charset=UTF-16LE
Comment 3•19 years ago
|
||
The "Size" profile string has to be set for REG_BINARY and REG_DWORD values in the installer's config.it or dwSize will be set to 0 and later passed to RegSetValueEx creating invalid registry entries. References: http://lxr.mozilla.org/mozilla/source/browser/installer/windows/config.it#1017 http://lxr.mozilla.org/mozilla/source/toolkit/mozapps/installer/windows/wizard/setup/ifuncns.c#2126 http://lxr.mozilla.org/mozilla/source/toolkit/mozapps/installer/windows/wizard/setup/ifuncns.c#1904 Alternatively set dwSize fixed to 4 for DWORD values. Sorry, no patch as I don't have a CVS set up.
Comment 4•19 years ago
|
||
here's the simple patch, untested
Comment 5•19 years ago
|
||
I assume you checked this isn't a dupe. setting ->NEW
Status: UNCONFIRMED → NEW
Ever confirmed: true
Updated•19 years ago
|
Assignee: bugs → nobody
QA Contact: bugzilla → installer
Comment 6•19 years ago
|
||
Michael, are you still interestd in fixing this? If so, you should ask for review from mconnor@mozilla.com by setting the appropriate flag on the attachment.
Comment 7•19 years ago
|
||
Comment on attachment 177959 [details] [diff] [review] config.it fixes OK, the patch is now outdated and I can't update it till later today. Should be pretty straightforward though. Just set the Size profile string to 4 bytes.
Attachment #177959 -
Flags: review?(mconnor)
Comment 8•19 years ago
|
||
Comment on attachment 177959 [details] [diff] [review] config.it fixes please request review on the new patch when its ready
Attachment #177959 -
Flags: review?(mconnor)
Updated•19 years ago
|
Attachment #177959 -
Attachment is obsolete: true
Comment 9•19 years ago
|
||
updated bandaid patch, real fix in 314904
Attachment #201776 -
Flags: review?(mconnor)
Comment 10•18 years ago
|
||
This is fixed with the new installer that landed - bug 326580. resolving -> wfm
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → WORKSFORME
Comment 11•17 years ago
|
||
Comment on attachment 201776 [details] [diff] [review] config.it fixes v2 This file does not even exist anymore.
Attachment #201776 -
Attachment is obsolete: true
Attachment #201776 -
Flags: review?(mconnor)
You need to log in
before you can comment on or make changes to this bug.
Description
•