X-Git-Url: http://git.annexia.org/?a=blobdiff_plain;f=daemon%2Fguestfsd.c;h=2e83b9fa96db06590b8f1f788343fb295a06273a;hb=b7e094fa14b306fe776b9b2695cf82fa32d4923f;hp=dc839ceac87e7e873669cc73767dfee7a17daf58;hpb=1e97e406c36031617a86a4fa6bb78a112848ee87;p=libguestfs.git diff --git a/daemon/guestfsd.c b/daemon/guestfsd.c index dc839ce..2e83b9f 100644 --- a/daemon/guestfsd.c +++ b/daemon/guestfsd.c @@ -33,6 +33,7 @@ #include #include #include +#include #include #include @@ -691,30 +692,45 @@ shell_quote (char *out, int len, const char *in) * use the IS_DEVICE macro. * * See guestfs(3) for the algorithm. + * + * We have to open the device and test for ENXIO, because + * the device nodes themselves will exist in the appliance. */ int device_name_translation (char *device, const char *func) { - struct stat statbuf; + int fd; - if (stat (device, &statbuf) == -1) { - /* If the name begins with "/dev/sd" then try the alternatives. */ - if (strncmp (device, "/dev/sd", 7) != 0) - goto error; + fd = open (device, O_RDONLY); + if (fd >= 0) { + close (fd); + return 0; + } - device[5] = 'h'; /* /dev/hd (old IDE driver) */ - if (stat (device, &statbuf) == 0) - return 0; + if (errno != ENXIO) { + error: + reply_with_perror ("%s: %s", func, device); + return -1; + } - device[5] = 'v'; /* /dev/vd (for virtio devices) */ - if (stat (device, &statbuf) == 0) - return 0; + /* If the name begins with "/dev/sd" then try the alternatives. */ + if (strncmp (device, "/dev/sd", 7) != 0) + goto error; - device[5] = 's'; /* Restore original device name. */ + device[5] = 'h'; /* /dev/hd (old IDE driver) */ + fd = open (device, O_RDONLY); + if (fd >= 0) { + close (fd); + return 0; + } - error: - reply_with_perror ("%s: %s", func, device); - return -1; + device[5] = 'v'; /* /dev/vd (for virtio devices) */ + fd = open (device, O_RDONLY); + if (fd >= 0) { + close (fd); + return 0; } - return 0; + + device[5] = 's'; /* Restore original device name. */ + goto error; }