Closed Bug 141820 Opened 22 years ago Closed 22 years ago

Don't quote parameters when calling DosStart Session

Categories

(NSPR :: NSPR, defect)

x86
OS/2
defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: mkaply, Assigned: wtc)

Details

Attachments

(1 file)

The OS/2 PR_CreateOS2Process incorrectly adds quotes to parameters.

For instance, if we pass two parameters -p 345 and hostname to CMD.EXE, it 
invokes it as:

CMD.EXE "-p 345" hostname

This is wrong. I'm not sure why the quoting was added in, but it doesn't work.

So we need to remove it.
Attached patch Fix for problemSplinter Review
Remove all the quote stuff
Fix checked into the tip and NSPRPUB_PRE_4_2_CLIENT_BRANCH of NSPR.
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Target Milestone: --- → 4.2.1
a=asa (on behalf of drivers) for checkin to the 1.0 branch
Michael, could you take care of checking your fix into
the mozilla 1.0 branch?
Yep. I'll put it on the branch.
Fix checked in to branch with second a= from chofmann
Verified
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: