[PATCH 1/3] bpf: Don't check for current being NULL

From: Richard Weinberger
Date: Mon Oct 16 2017 - 14:18:48 EST


current is never NULL.

Signed-off-by: Richard Weinberger <richard@xxxxxx>
---
kernel/bpf/helpers.c | 12 ------------
1 file changed, 12 deletions(-)

diff --git a/kernel/bpf/helpers.c b/kernel/bpf/helpers.c
index 3d24e238221e..e8845adcd15e 100644
--- a/kernel/bpf/helpers.c
+++ b/kernel/bpf/helpers.c
@@ -120,9 +120,6 @@ BPF_CALL_0(bpf_get_current_pid_tgid)
{
struct task_struct *task = current;

- if (unlikely(!task))
- return -EINVAL;
-
return (u64) task->tgid << 32 | task->pid;
}

@@ -138,9 +135,6 @@ BPF_CALL_0(bpf_get_current_uid_gid)
kuid_t uid;
kgid_t gid;

- if (unlikely(!task))
- return -EINVAL;
-
current_uid_gid(&uid, &gid);
return (u64) from_kgid(&init_user_ns, gid) << 32 |
from_kuid(&init_user_ns, uid);
@@ -156,9 +150,6 @@ BPF_CALL_2(bpf_get_current_comm, char *, buf, u32, size)
{
struct task_struct *task = current;

- if (unlikely(!task))
- goto err_clear;
-
strncpy(buf, task->comm, size);

/* Verifier guarantees that size > 0. For task->comm exceeding
@@ -167,9 +158,6 @@ BPF_CALL_2(bpf_get_current_comm, char *, buf, u32, size)
*/
buf[size - 1] = 0;
return 0;
-err_clear:
- memset(buf, 0, size);
- return -EINVAL;
}

const struct bpf_func_proto bpf_get_current_comm_proto = {
--
2.13.6