[Bug 39688] New: Crash in file browser dialog (probably because of > MAX_PATH)

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

Bug ID: 39688
           Summary: Crash in file browser dialog (probably because of >
                    MAX_PATH)
           Product: Wine
           Version: 1.6.2
          Hardware: x86
                OS: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: comdlg32
          Assignee: [e-mail]
          Reporter: [e-mail]
      Distribution: ---

Wine's GetOpenFileNameW/GetSaveFileNameW seems to crash when encountering a
very long path name (i.e. longer than MAX_PATH characters).

[Bug 39688] Crash in file browser dialog (probably because of > MAX_PATH)

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

--- Comment #1 from whatbug  ---
Created attachment 52921
  --> https://bugs.winehq.org/attachment.cgi?id=52921
Backtrace of the crash

[Bug 39688] Crash in file browser dialog (probably because of > MAX_PATH)

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

Bruno Jesus  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|comdlg32                    |-unknown

--- Comment #2 from Bruno Jesus  ---
What version of OpenMPT are you using? It crashes when you select a file in the
dialog and press OK or crashes when you browse to a too deep folder?

Anyway, wine 1.6.2 is very old. Please update to wine 1.8-rc2 and try again.

[Bug 39688] Crash in file browser dialog (probably because of > MAX_PATH)

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

--- Comment #3 from whatbug  ---
The version of OpenMPT does not matter - the application crashes *in the file
browser* before it passes back control to the program.
1.6.2 is still the default on Ubuntu LTS so I'm not really able to upgrade at
the moment.

[Bug 39688] OpenMPT 1.25.04.00 crashes after selecting a file in file browser dialog from a very long path

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

Bruno Jesus  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |download
                URL|                            |https://openmpt.org/downloa
                   |                            |d
            Summary|Crash in file browser       |OpenMPT 1.25.04.00 crashes
                   |dialog (probably because of |after selecting a file in
                   |> MAX_PATH)                 |file browser dialog from a
                   |                            |very long path

--- Comment #4 from Bruno Jesus  ---
(In reply to whatbug from comment #3)
> The version of OpenMPT does not matter - the application crashes *in the
> file browser* before it passes back control to the program.

This was the file I used to test:
35125d37407edfeb24fd5db47124972324f9266a OpenMPT-1.25.04.00-Setup.exe

I installed the program then launched it and opened one of the songs from the
examples folder, this works.

Then I created a very long (> 255 characters) sequence of directories with an
example file inside and the application closed.

Does it work in Windows in the same conditions?
> 1.6.2 is still the default on Ubuntu LTS so I'm not really able to upgrade
> at the moment.

Please take a look at: http://wiki.winehq.org/Ubuntu

[Bug 39688] OpenMPT 1.25.04.00 crashes after selecting a file in file browser dialog from a very long path

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

--- Comment #5 from whatbug  ---
It still happens with v1.9.0, but there might be more to the issue. It does
indeed not appear if you just create an overly long filename. The issue appears
to happen if the overly long filename is found in a shared virtualbox folder.

I've created a virtualbox shared folder and then mounted it like this in a
Xubuntu VM:
mkdir ~/music
sudo mount -t vboxsf -o uid=$UID,gid=$(id -g) E_DRIVE ~/music
In this folder, I have placed an overly long directory name
("1xxxxxxxxx2xxxxxxxxx3xxxxxxxxx4xxxxxxxxx5xxxxxxxxx6xxxxxxxxx7xxxxxxxxx8xxxxxxxxx9xxxxxxxxx0xxxxxxxxx1xxxxxxxxx2xxxxxxxxx3xxxxxxxxx4xxxxxxxxx5xxxxxxxxx6xxxxxxxxx7xxxxxxxxx8xxxxxxxxx9xxxxxxxxx0xxxxxxxxx1xxxxxxxxx2xxxxxxxxx3xxxxxxxxx4xxxxxxxxx5xxx"
for testing) which then crashes the Wine explorer.

[Bug 39688] OpenMPT 1.25.04.00 crashes after selecting a file in file browser dialog from a very long path

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

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #6 from [e-mail] ---
Could be dupe or related to bug 39867.

[Bug 39688] OpenMPT 1.25.04.00 crashes after selecting a file in file browser dialog from a very long path

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

--- Comment #7 from [e-mail] ---
(In reply to Bruno Jesus from comment #4)
> Then I created a very long (> 255 characters) sequence of directories with
> an example file inside and the application closed.

If wine uses some point of MAX_PATH I think it has max value of 255 or 256 +1.

There are 2-3 similar bugs to this currently.

[Bug 39688] OpenMPT 1.25.04.00 crashes after selecting a file in file browser dialog from a very long path

[WineHQ Bugzilla]

http://bugs.winehq.org/show_bug.cgi?id=39688

Ken Sharp  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|---                         |DUPLICATE

--- Comment #8 from Ken Sharp  ---
Would be odd if these weren't duplicate.

*** This bug has been marked as a duplicate of bug 39867 ***

[Bug 39688] OpenMPT 1.25.04.00 crashes after selecting a file in file browser dialog from a very long path

[WineHQ Bugzilla]

http://bugs.winehq.org/show_bug.cgi?id=39688

Alistair Leslie-Hughes  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |CLOSED

--- Comment #9 from Alistair Leslie-Hughes  ---
Closing duplicate.