lsm: twek apparmor_process_label_get()

Fixes: Coverity 1473189 Fixes: Coverity 1473190 Fixes: 47f4914d ("apparmor: prefer /proc/.../attr/apparmor/current over legacy interface") Signed-off-by: 's avatarChristian Brauner <christian.brauner@ubuntu.com>
parent 580253dc
...@@ -430,15 +430,21 @@ error: ...@@ -430,15 +430,21 @@ error:
static char *apparmor_process_label_get(struct lsm_ops *ops, pid_t pid) static char *apparmor_process_label_get(struct lsm_ops *ops, pid_t pid)
{ {
int label_fd; __do_close int fd_label = -EBADF;
__do_free char *label = NULL; __do_free char *label = NULL;
int ret;
size_t len; size_t len;
label_fd = __apparmor_process_label_open(ops, pid, O_RDONLY, false); fd_label = __apparmor_process_label_open(ops, pid, O_RDONLY, false);
if (label_fd < 0) if (fd_label < 0)
return NULL;
ret = fd_to_buf(fd_label, &label, &len);
if (ret < 0)
return NULL; return NULL;
fd_to_buf(label_fd, &label, &len); if (len == 0)
return NULL;
len = strcspn(label, "\n \t"); len = strcspn(label, "\n \t");
if (len) if (len)
......
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