diff options
Diffstat (limited to 'fs/ncpfs/ioctl.c')
| -rw-r--r-- | fs/ncpfs/ioctl.c | 53 | 
1 files changed, 26 insertions, 27 deletions
diff --git a/fs/ncpfs/ioctl.c b/fs/ncpfs/ioctl.c index c2a1f9a155c..d5659d96ee7 100644 --- a/fs/ncpfs/ioctl.c +++ b/fs/ncpfs/ioctl.c @@ -17,15 +17,12 @@  #include <linux/mount.h>  #include <linux/slab.h>  #include <linux/highuid.h> -#include <linux/smp_lock.h>  #include <linux/vmalloc.h>  #include <linux/sched.h> -#include <linux/ncp_fs.h> -  #include <asm/uaccess.h> -#include "ncplib_kernel.h" +#include "ncp_fs.h"  /* maximum limit for ncp_objectname_ioctl */  #define NCP_OBJECT_NAME_MAX_LEN	4096 @@ -44,11 +41,11 @@ ncp_get_fs_info(struct ncp_server * server, struct inode *inode,  		return -EFAULT;  	if (info.version != NCP_GET_FS_INFO_VERSION) { -		DPRINTK("info.version invalid: %d\n", info.version); +		ncp_dbg(1, "info.version invalid: %d\n", info.version);  		return -EINVAL;  	}  	/* TODO: info.addr = server->m.serv_addr; */ -	SET_UID(info.mounted_uid, server->m.mounted_uid); +	SET_UID(info.mounted_uid, from_kuid_munged(current_user_ns(), server->m.mounted_uid));  	info.connection		= server->connection;  	info.buffer_size	= server->buffer_size;  	info.volume_number	= NCP_FINFO(inode)->volNumber; @@ -69,10 +66,10 @@ ncp_get_fs_info_v2(struct ncp_server * server, struct inode *inode,  		return -EFAULT;  	if (info2.version != NCP_GET_FS_INFO_VERSION_V2) { -		DPRINTK("info.version invalid: %d\n", info2.version); +		ncp_dbg(1, "info.version invalid: %d\n", info2.version);  		return -EINVAL;  	} -	info2.mounted_uid   = server->m.mounted_uid; +	info2.mounted_uid   = from_kuid_munged(current_user_ns(), server->m.mounted_uid);  	info2.connection    = server->connection;  	info2.buffer_size   = server->buffer_size;  	info2.volume_number = NCP_FINFO(inode)->volNumber; @@ -135,10 +132,10 @@ ncp_get_compat_fs_info_v2(struct ncp_server * server, struct inode *inode,  		return -EFAULT;  	if (info2.version != NCP_GET_FS_INFO_VERSION_V2) { -		DPRINTK("info.version invalid: %d\n", info2.version); +		ncp_dbg(1, "info.version invalid: %d\n", info2.version);  		return -EINVAL;  	} -	info2.mounted_uid   = server->m.mounted_uid; +	info2.mounted_uid   = from_kuid_munged(current_user_ns(), server->m.mounted_uid);  	info2.connection    = server->connection;  	info2.buffer_size   = server->buffer_size;  	info2.volume_number = NCP_FINFO(inode)->volNumber; @@ -311,8 +308,7 @@ static long __ncp_ioctl(struct inode *inode, unsigned int cmd, unsigned long arg  		else  			result = server->reply_size;  		ncp_unlock_server(server); -		DPRINTK("ncp_ioctl: copy %d bytes\n", -			result); +		ncp_dbg(1, "copy %d bytes\n", result);  		if (result >= 0)  			if (copy_to_user(request.data, bouncebuffer, result))  				result = -EFAULT; @@ -351,22 +347,25 @@ static long __ncp_ioctl(struct inode *inode, unsigned int cmd, unsigned long arg  		{  			u16 uid; -			SET_UID(uid, server->m.mounted_uid); +			SET_UID(uid, from_kuid_munged(current_user_ns(), server->m.mounted_uid));  			if (put_user(uid, (u16 __user *)argp))  				return -EFAULT;  			return 0;  		}  	case NCP_IOC_GETMOUNTUID32: -		if (put_user(server->m.mounted_uid, -			     (u32 __user *)argp)) +	{ +		uid_t uid = from_kuid_munged(current_user_ns(), server->m.mounted_uid); +		if (put_user(uid, (u32 __user *)argp))  			return -EFAULT;  		return 0; +	}  	case NCP_IOC_GETMOUNTUID64: -		if (put_user(server->m.mounted_uid, -			     (u64 __user *)argp)) +	{ +		uid_t uid = from_kuid_munged(current_user_ns(), server->m.mounted_uid); +		if (put_user(uid, (u64 __user *)argp))  			return -EFAULT;  		return 0; - +	}  	case NCP_IOC_GETROOT:  		{  			struct ncp_setroot_ioctl sr; @@ -385,9 +384,9 @@ static long __ncp_ioctl(struct inode *inode, unsigned int cmd, unsigned long arg  						sr.namespace = server->name_space[sr.volNumber];  						result = 0;  					} else -						DPRINTK("ncpfs: s_root->d_inode==NULL\n"); +						ncp_dbg(1, "s_root->d_inode==NULL\n");  				} else -					DPRINTK("ncpfs: s_root==NULL\n"); +					ncp_dbg(1, "s_root==NULL\n");  			} else {  				sr.volNumber = -1;  				sr.namespace = 0; @@ -440,11 +439,11 @@ static long __ncp_ioctl(struct inode *inode, unsigned int cmd, unsigned long arg  							NCP_FINFO(s_inode)->DosDirNum = dosde;  							server->root_setuped = 1;  						} else { -							DPRINTK("ncpfs: s_root->d_inode==NULL\n"); +							ncp_dbg(1, "s_root->d_inode==NULL\n");  							result = -EIO;  						}  					} else { -						DPRINTK("ncpfs: s_root==NULL\n"); +						ncp_dbg(1, "s_root==NULL\n");  						result = -EIO;  					}  				} @@ -811,9 +810,9 @@ outrel:  long ncp_ioctl(struct file *filp, unsigned int cmd, unsigned long arg)  { -	struct inode *inode = filp->f_dentry->d_inode; +	struct inode *inode = file_inode(filp);  	struct ncp_server *server = NCP_SERVER(inode); -	uid_t uid = current_uid(); +	kuid_t uid = current_uid();  	int need_drop_write = 0;  	long ret; @@ -822,12 +821,12 @@ long ncp_ioctl(struct file *filp, unsigned int cmd, unsigned long arg)  	case NCP_IOC_CONN_LOGGED_IN:  	case NCP_IOC_SETROOT:  		if (!capable(CAP_SYS_ADMIN)) { -			ret = -EACCES; +			ret = -EPERM;  			goto out;  		}  		break;  	} -	if (server->m.mounted_uid != uid) { +	if (!uid_eq(server->m.mounted_uid, uid)) {  		switch (cmd) {  		/*  		 * Only mount owner can issue these ioctls.  Information @@ -904,7 +903,7 @@ long ncp_ioctl(struct file *filp, unsigned int cmd, unsigned long arg)  	ret = __ncp_ioctl(inode, cmd, arg);  outDropWrite:  	if (need_drop_write) -		mnt_drop_write(filp->f_path.mnt); +		mnt_drop_write_file(filp);  out:  	return ret;  }  | 
