http://bugs.winehq.org/show_bug.cgi?id=10974
Summary: wine prefers loopback devices over regular files Product: Wine Version: CVS/GIT Platform: Other OS/Version: other Status: UNCONFIRMED Severity: normal Priority: P2 Component: wine-kernel AssignedTo: wine-bugs@winehq.org ReportedBy: rmh@aybabtu.com
Created an attachment (id=9920) --> (http://bugs.winehq.org/attachment.cgi?id=9920) fix loopback handling
When using a loopback device as a wine drive, wine will prefer to use the loopback device (e.g. /dev/loop0) for volume access rather than the regular file it's based on (e.g. ~/foo.iso).
From dlls/ntdll/directory.c:
else if (!stat( entry->mnt_fsname, &st ) && S_ISREG(st.st_mode)) { /* if device is a regular file check for a loop mount */ if ((device = strstr( entry->mnt_opts, "loop=" ))) { char *p = strchr( device + 5, ',' ); if (p) *p = 0; return device + 5; } }
I'm not sure if this has any advantage, but it has a big inconvenient. It's almost certain that /dev/loop0 is owned by root and not world-readable, and it's also mostly certain that wine isn't being run as root. If the image is user-owned, by using the loop device we lose the ability to read it.
This typicaly causes breakage that is difficult to figure out. For example, see http://appdb.winehq.org/objectManager.php?sClass=version&iId=727&iTe...
Note: I'm filing this separate from bug 10957 because it's an unrelated issue. The patch I'm attaching here collides with the one I sent for bug 10957, but takes both situations into account.
http://bugs.winehq.org/show_bug.cgi?id=10974
Robert Millan rmh@aybabtu.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Keywords| |patch
http://bugs.winehq.org/show_bug.cgi?id=10974
--- Comment #1 from Austin English austinenglish@gmail.com 2008-06-05 10:33:33 --- Still present in 1.0-rc3
http://bugs.winehq.org/show_bug.cgi?id=10974
Austin English austinenglish@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Keywords| |source
--- Comment #2 from Austin English austinenglish@gmail.com 2008-12-03 16:33:51 --- Still present in 1.1.9.
http://bugs.winehq.org/show_bug.cgi?id=10974
Austin English austinenglish@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Version|CVS/GIT |unspecified
--- Comment #3 from Austin English austinenglish@gmail.com 2009-01-18 03:47:31 --- Removing deprecated CVS/GIT version tag. Please retest in current git. If the bug is still present in today's wine, but was not present in some earlier version of wine, please update version field to earliest known version of wine that had the bug. Thanks!
http://bugs.winehq.org/show_bug.cgi?id=10974
--- Comment #4 from Austin English austinenglish@gmail.com 2009-07-21 15:28:05 --- I sent the patch for you: http://www.winehq.org/pipermail/wine-patches/2009-July/076226.html
http://bugs.winehq.org/show_bug.cgi?id=10974
--- Comment #5 from Alexandre Julliard julliard@winehq.org 2009-07-21 16:32:43 --- It very much needs to be a device. We can't use a regular file. The real bug is in mount, it should fix the permissions of the device when mounting it.
http://bugs.winehq.org/show_bug.cgi?id=10974
Saulius K. saulius2@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |saulius2@gmail.com
http://bugs.winehq.org/show_bug.cgi?id=10974
Vitaliy Margolen vitaliy@kievinfo.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |RESOLVED Resolution| |WONTFIX
--- Comment #6 from Vitaliy Margolen vitaliy@kievinfo.com 2009-07-22 07:24:55 --- As Alexandre said, Wine needs a device, not a file.
http://bugs.winehq.org/show_bug.cgi?id=10974
Vitaliy Margolen vitaliy@kievinfo.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED
--- Comment #7 from Vitaliy Margolen vitaliy@kievinfo.com 2009-07-22 07:25:53 --- Closing