From: Steve French <sfrench@samba.org>
Date: Sat, 9 Apr 2005 05:01:00 +0000 (+0000)
Subject: r6262: Minor updates to cifs umount helper
X-Git-Url: http://git.samba.org/?a=commitdiff_plain;h=16167208cfb5d8f995126a4af98911240d58e007;p=metze%2Fsamba%2Fwip.git

r6262: Minor updates to cifs umount helper
(This used to be commit cd8b93edc6d488ab6f6652ef533ce13e885b265b)
---

diff --git a/source3/client/umount.cifs.c b/source3/client/umount.cifs.c
index 0210de85a0c3..18dbc3b1cf66 100644
--- a/source3/client/umount.cifs.c
+++ b/source3/client/umount.cifs.c
@@ -49,7 +49,7 @@
 #define MNT_EXPIRE 0x04
 #endif
 
-#define CIFS_IOC_CHECKUMOUNT _IOR('u', 2, int)
+#define CIFS_IOC_CHECKUMOUNT _IO('c', 2)
    
 static struct option longopts[] = {
 	{ "all", 0, NULL, 'a' },
@@ -93,14 +93,21 @@ static int umount_check_perm(char * dir)
 	/* presumably can not chdir into the target as we do on mount */
 
 	fileid = open(dir, O_RDONLY | O_DIRECTORY | O_NOFOLLOW, 0);
-
-	/* check if fileid valid if fileid == -1 BB FIXME */
+	if(fileid == -1) {
+		if(verboseflg)
+			printf("error opening mountpoint %d %s",errno,strerror(errno));
+		return errno;
+	}
 
 	rc = ioctl(fileid, CIFS_IOC_CHECKUMOUNT, NULL);
 
 	if(verboseflg)
-		printf("ioctl returned %d with errno %d\n",rc,errno);
+		printf("ioctl returned %d with errno %d %s\n",rc,errno,strerror(errno));
 
+	if(rc == ENOTTY)
+		printf("user unmounting via %s is an optional feature of the cifs filesystem driver (cifs.ko)\n\tand requires cifs.ko version 1.32 or later\n",thisprogram);
+	else if (rc > 0)
+		printf("user unmount of %s failed with %d %s",dir,errno,strerror(errno));
 	close(fileid);
 
 	return rc;