[Bug 42514] New: start.exe incorrectly treats multiple quoted arguments as the console title (breaking .e.g URL opening in League of Legends)

[wine-bugs at winehq.org]

https://bugs.winehq.org/show_bug.cgi?id=42514

            Bug ID: 42514
           Summary: start.exe incorrectly treats multiple quoted arguments
                    as the console title  (breaking .e.g URL opening in
                    League of Legends)
           Product: Wine
           Version: unspecified
          Hardware: x86
                OS: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: programs
          Assignee: wine-bugs at winehq.org
          Reporter: hydratech at gmail.com
      Distribution: ---

Created attachment 57401
  --> https://bugs.winehq.org/attachment.cgi?id=57401
Diff containing the necessary fix for correct multiple quote handling.

Follow up to bug 42508:

On Windows the following two command-lines open a URL in the browser:
C:\>start "" https://winehq.org

C:\>start "" "https://winehq.org"

In other words, the second quoted argument in the second command-line does not
override the console title to be "http://winehq.org".

Now consider the following two wine commands:
$ wine start '""' 'https://winehq.org'

$ wine start '""' '"https://winehq.org"'

The first line opens the website in a browser, as expected. The second line
however opens a console window running cmd. (the title is not actually set, but
that's another issue.)

In start.exe's parsing loop the if statement for detecting the console title
does not stop checking for the title when a title has already been parsed
successfully. Causing 'https://winehq.org' to be interpreted as the console
title, and executing cmd.exe since no executable remains on the command-line.

I have provided a diff containing the very simple fix necessary to correctly
handle this case. As most of start.exe is still hardtabbed I've left the tabs
for now.

League of Legends relies on the correct behaviour of this second quoted
argument to open URLs in the browser; see bug 42508.

-- 
Do not reply to this email, post in Bugzilla using the
above URL to reply.
You are receiving this mail because:
You are watching all bug changes.