Closed
Bug 396196
Opened 17 years ago
Closed 11 years ago
Use wrapper executable with no extra options for OS Integration
Categories
(Firefox :: Shell Integration, defect)
Firefox
Shell Integration
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: dveditz, Unassigned)
References
Details
(Keywords: sec-want, Whiteboard: [sg:want P2])
As seen in the various inter-program URI-launching problems recently (bug 384384 mitigated by adding -osint, bug 395942 quicktime ignoring the -osint addition) we should create a launcher program for OS Integration that does not take extra arguments beyond the minimum necessary to load a URI. That wrapper can then call the real firefox executable with the -osint -url "%s" params, and any other code building on our platform can call the internal exe if they need the -chrome param or other fancy stuff.
Spun off from bug 395942 comment 7 and later discussion.
Flags: blocking-firefox3?
Updated•17 years ago
|
Whiteboard: [sg:want P2]
Comment 1•17 years ago
|
||
passing arbitrary arguments is dangerous on linux.
shouldn't os be all?
Reporter | ||
Comment 3•17 years ago
|
||
Mike: we need an owner here, and please make this block Firefox 3
Assignee: nobody → mconnor
Updated•17 years ago
|
Flags: blocking-firefox3? → blocking-firefox3+
Updated•17 years ago
|
Target Milestone: --- → Firefox 3 M11
Updated•17 years ago
|
Priority: -- → P5
Comment 4•17 years ago
|
||
Would like to see this, but I wonder if its too late to muck with this for Fx3. Jim, thoughts?
Assignee: mconnor → jmathies
Flags: wanted-firefox3+
Flags: blocking-firefox3-
Flags: blocking-firefox3+
Target Milestone: Firefox 3 Mx → Firefox 3 M11
Comment 5•17 years ago
|
||
Well, imho, it feels like a pretty big change to make at beta 2. But once I finish up my other higher priority bugs I can try to roll something together. From my understanding this would touch on the installer, all the application registration stuff in firefox, and in some places some of the new helper app handling code which filters the running app out of helper app registrations. Am I missing anything?
Comment 6•17 years ago
|
||
Beta3 is over. Removing TM.
Target Milestone: Firefox 3 beta3 → ---
Version: unspecified → Trunk
Updated•16 years ago
|
Assignee: jmathies → nobody
Updated•16 years ago
|
Flags: wanted-firefox3.1?
Priority: P5 → --
Comment 8•11 years ago
|
||
Daniel, how important is this bug for security these days? Is it still wanted after 5 years?
Flags: wanted-firefox3.5? → needinfo?(dveditz)
Reporter | ||
Comment 9•11 years ago
|
||
This was an attempt to protect us from misbehaving apps like QuickTime (see bug 395942), but if an application is doing its own thing they might work around such a wrapper anyways.
Status: NEW → RESOLVED
Closed: 11 years ago
Flags: needinfo?(dveditz)
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•