diff options
| author | Jarkko Sakkinen <jarkko.sakkinen@intel.com> | 2011-10-10 14:29:28 +0300 |
|---|---|---|
| committer | Casey Schaufler <cschaufler@cschaufler-intel.(none)> | 2011-10-12 14:30:07 -0700 |
| commit | f8859d98c1d1e73393285fb9dd57007839956247 (patch) | |
| tree | a6937380935074702febe48239bb891b4242752d | |
| parent | 84088ba239293abb24260c6c36d86e8775b6707f (diff) | |
| download | linux-3.10-f8859d98c1d1e73393285fb9dd57007839956247.tar.gz linux-3.10-f8859d98c1d1e73393285fb9dd57007839956247.tar.bz2 linux-3.10-f8859d98c1d1e73393285fb9dd57007839956247.zip | |
Smack: fix for /smack/access output, use string instead of byte
Small fix for the output of access SmackFS file. Use string
is instead of byte. Makes it easier to extend API if it is
needed.
Signed-off-by: Jarkko Sakkinen <jarkko.sakkinen@intel.com>
| -rw-r--r-- | security/smack/smackfs.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/security/smack/smackfs.c b/security/smack/smackfs.c index 54f6e18dea2..5498c4a2d1a 100644 --- a/security/smack/smackfs.c +++ b/security/smack/smackfs.c @@ -1497,6 +1497,7 @@ static ssize_t smk_write_access(struct file *file, const char __user *buf, { struct smack_rule rule; char *data; + int res; if (!capable(CAP_MAC_ADMIN)) return -EPERM; @@ -1508,8 +1509,10 @@ static ssize_t smk_write_access(struct file *file, const char __user *buf, if (count < SMK_LOADLEN || smk_parse_rule(data, &rule)) return -EINVAL; - data[0] = smk_access(rule.smk_subject, rule.smk_object, - rule.smk_access, NULL) == 0; + res = smk_access(rule.smk_subject, rule.smk_object, rule.smk_access, + NULL); + data[0] = res == 0 ? '1' : '0'; + data[1] = '\0'; simple_transaction_set(file, 1); return SMK_LOADLEN; |