kernel32: Tiny improvement to the GetVolumePathNameW stub

Michael Karcher wine at
Sat Aug 16 12:33:55 CDT 2008

Am Samstag, den 16.08.2008, 17:12 +0000 schrieb Louis. Lenders:
> Hi,  this fixes Adobe Lightroom 2.0 start up bug, mentioned in 
> as long wine doesn't handle with volume mount points, it's probably
> safe to satisfy most apps.

The idea of your patch looks right to me. But (1) please don't use
memcmp to compare wide characters, especially, don't use memcmp with a
count of one, as it only compares one half the the wide character. use
  filename[1] == ':' && filename[2] == '\\'
instead. (2) Please check that filename really has at least 3 characters
before accessing it. (3)Please check the size of the output buffer
before copying anything into it.

Finally, the chance of getting this patch accepted raises if you
accompany the implementation with an API test. Another point to increase
the chances of getting that patch committed is to implement the corner
cases (at least the empty string) explained on MSDN, also backed up by
an API test. Please also note the documented behaviour for the case of
the output buffer being short by one byte.

  Michael Karcher

More information about the wine-patches mailing list