diff options
author | Joe Perches <joe@perches.com> | 2015-03-23 18:01:35 -0700 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2015-04-19 10:10:46 +0200 |
commit | 9dece36273f91a8e7467b7eadbe319fa924eec83 (patch) | |
tree | 1f09bfad6053b6c699187a04f5419a44e300af30 | |
parent | a13a3624298bbcb80c6ba46fbf7e2213a3d41013 (diff) | |
download | lwn-9dece36273f91a8e7467b7eadbe319fa924eec83.tar.gz lwn-9dece36273f91a8e7467b7eadbe319fa924eec83.zip |
selinux: fix sel_write_enforce broken return value
commit 6436a123a147db51a0b06024a8350f4c230e73ff upstream.
Return a negative error value like the rest of the entries in this function.
Signed-off-by: Joe Perches <joe@perches.com>
Acked-by: Stephen Smalley <sds@tycho.nsa.gov>
[PM: tweaked subject line]
Signed-off-by: Paul Moore <pmoore@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r-- | security/selinux/selinuxfs.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/security/selinux/selinuxfs.c b/security/selinux/selinuxfs.c index ff427733c290..464be51025f6 100644 --- a/security/selinux/selinuxfs.c +++ b/security/selinux/selinuxfs.c @@ -150,7 +150,7 @@ static ssize_t sel_write_enforce(struct file *file, const char __user *buf, goto out; /* No partial writes. */ - length = EINVAL; + length = -EINVAL; if (*ppos != 0) goto out; |