If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Path in shell\open\command registry key in installer SetAsDefault function should be quoted

RESOLVED FIXED in seamonkey2.15

Status

SeaMonkey
Installer
RESOLVED FIXED
7 years ago
5 years ago

People

(Reporter: mcsmurf, Assigned: mcsmurf)

Tracking

Trunk
seamonkey2.15
x86
Windows XP

SeaMonkey Tracking Flags

(seamonkey2.15 fixed)

Details

Attachments

(2 attachments)

(Assignee)

Description

7 years ago
I noticed the path to the application in the shell\open\command registry key does not get quoted (for example below the StartMenuInternet key). This should be changed as an application can also be installed in a path containing a space.
(Assignee)

Comment 1

7 years ago
Created attachment 482026 [details] [diff] [review]
Firefox patch
Assignee: nobody → bugzilla
Attachment #482026 - Flags: review?(rstrong)
(Assignee)

Updated

7 years ago
Attachment #482026 - Flags: review?(rstrong) → review?(robert.bugzilla)
(Assignee)

Comment 2

7 years ago
Created attachment 482028 [details] [diff] [review]
SeaMonkey Patch
Attachment #482028 - Flags: review?(bugzilla)
(In reply to comment #0)
> I noticed the path to the application in the shell\open\command registry key
> does not get quoted (for example below the StartMenuInternet key). This should
> be changed as an application can also be installed in a path containing a
> space.
These specific paths only need to be quoted if they have command line arguments.
Attachment #482026 - Flags: review?(robert.bugzilla) → review-
Assignee: bugzilla → installer
Component: NSIS Installer → Installer
Product: Toolkit → SeaMonkey
QA Contact: nsis.installer → xpi-packages
(Assignee)

Comment 4

7 years ago
Guess I'll have to investigate this further then. I discovered this bug after setting SeaMonkey as default browser when the path to seamonkey.exe contained spaces (I got "(Null) not a valid win32 application" after clicking on the Internet start menu entry). Then I added quotes around the path in shell\open\command and it worked fine. Maybe something else triggered that as on another PC (same OS, same service pack) it seems to work fine with spaces inside the path.
There is nothing wrong with adding quotes but it definitely isn't required for these keys and Firefox has been doing it this way iirc since the 2.0 cycle without a single bug report.
(Assignee)

Comment 6

7 years ago
Ok, I found out what is causing this, it's a Windows bug and me having tested something else a few weeks ago: The path to my SeaMonkey installation was C:\Documents and Settings\Frank\Desktop\.... . I had also created a file with the name "Documents" on the drive C:. After renaming (or deleting) that file it worked again. So yeah, it's a special case :)
Comment on attachment 482028 [details] [diff] [review]
SeaMonkey Patch

Cancelling review until the Firefox side is sorted.
Attachment #482028 - Flags: review?(bugzilla)
(Assignee)

Comment 8

7 years ago
The Firefox side is sorted IMHO, it's a bug(?) or a feature in Windows, see Comment 6. The quotes are needed.
(Assignee)

Comment 9

5 years ago
Comment on attachment 482028 [details] [diff] [review]
SeaMonkey Patch

The quotes are needed, see Comment 6 and http://msdn.microsoft.com/en-us/library/windows/desktop/cc144109%28v=vs.85%29.aspx on this:
"Use quotation marks appropriately to ensure that spaces in the command line are not misinterpreted. [...] Note that in this value, quotation marks are placed around the path because it contains embedded spaces. Omitting these quotation marks could cause the command line to be misinterpreted."
Attachment #482028 - Flags: review?(philip.chee)

Comment 10

5 years ago
Comment on attachment 482028 [details] [diff] [review]
SeaMonkey Patch

rs=me based on code inspection. I assume that you've tested this patch!
Attachment #482028 - Flags: review?(philip.chee) → review+
(Assignee)

Comment 11

5 years ago
Tested the patch and works fine. Pushed to comm-central: https://hg.mozilla.org/comm-central/rev/05edecfafc41
Assignee: installer → bugzilla
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
(Assignee)

Comment 12

5 years ago
Settings flags to make tracking of fixed bugs easier.
status-seamonkey2.15: --- → fixed
Target Milestone: --- → seamonkey2.15
You need to log in before you can comment on or make changes to this bug.