Commit 6c4c17b0 authored by Tyler Hicks's avatar Tyler Hicks Committed by Linus Torvalds

ecryptfs: discard ecryptfsd registration messages in miscdev

The userspace eCryptfs daemon sends HELO and QUIT messages to the kernel
for per-user daemon (un)registration.  These messages are required when
netlink is used as the transport, but (un)registration is handled by
opening and closing the device file when miscdev is the transport.  These
messages should be discarded in the miscdev transport so that a daemon
isn't registered twice.
Signed-off-by: default avatarTyler Hicks <tyhicks@linux.vnet.ibm.com>
Cc: Michael Halcrow <mhalcrow@us.ibm.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 746f1e55
...@@ -357,46 +357,6 @@ ecryptfs_miscdev_read(struct file *file, char __user *buf, size_t count, ...@@ -357,46 +357,6 @@ ecryptfs_miscdev_read(struct file *file, char __user *buf, size_t count,
return rc; return rc;
} }
/**
* ecryptfs_miscdev_helo
* @euid: effective user id of miscdevess sending helo packet
* @user_ns: The namespace in which @euid applies
* @pid: miscdevess id of miscdevess sending helo packet
*
* Returns zero on success; non-zero otherwise
*/
static int ecryptfs_miscdev_helo(uid_t euid, struct user_namespace *user_ns,
struct pid *pid)
{
int rc;
rc = ecryptfs_process_helo(ECRYPTFS_TRANSPORT_MISCDEV, euid, user_ns,
pid);
if (rc)
printk(KERN_WARNING "Error processing HELO; rc = [%d]\n", rc);
return rc;
}
/**
* ecryptfs_miscdev_quit
* @euid: effective user id of miscdevess sending quit packet
* @user_ns: The namespace in which @euid applies
* @pid: miscdevess id of miscdevess sending quit packet
*
* Returns zero on success; non-zero otherwise
*/
static int ecryptfs_miscdev_quit(uid_t euid, struct user_namespace *user_ns,
struct pid *pid)
{
int rc;
rc = ecryptfs_process_quit(euid, user_ns, pid);
if (rc)
printk(KERN_WARNING
"Error processing QUIT message; rc = [%d]\n", rc);
return rc;
}
/** /**
* ecryptfs_miscdev_response - miscdevess response to message previously sent to daemon * ecryptfs_miscdev_response - miscdevess response to message previously sent to daemon
* @data: Bytes comprising struct ecryptfs_message * @data: Bytes comprising struct ecryptfs_message
...@@ -512,26 +472,7 @@ ecryptfs_miscdev_write(struct file *file, const char __user *buf, ...@@ -512,26 +472,7 @@ ecryptfs_miscdev_write(struct file *file, const char __user *buf,
__func__, rc); __func__, rc);
break; break;
case ECRYPTFS_MSG_HELO: case ECRYPTFS_MSG_HELO:
rc = ecryptfs_miscdev_helo(current->euid,
current->nsproxy->user_ns,
task_pid(current));
if (rc) {
printk(KERN_ERR "%s: Error attempting to process "
"helo from pid [0x%p]; rc = [%d]\n", __func__,
task_pid(current), rc);
goto out_free;
}
break;
case ECRYPTFS_MSG_QUIT: case ECRYPTFS_MSG_QUIT:
rc = ecryptfs_miscdev_quit(current->euid,
current->nsproxy->user_ns,
task_pid(current));
if (rc) {
printk(KERN_ERR "%s: Error attempting to process "
"quit from pid [0x%p]; rc = [%d]\n", __func__,
task_pid(current), rc);
goto out_free;
}
break; break;
default: default:
ecryptfs_printk(KERN_WARNING, "Dropping miscdev " ecryptfs_printk(KERN_WARNING, "Dropping miscdev "
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment