bpf: Check the helper function is valid in get_helper_proto

JIRA: https://issues.redhat.com/browse/RHEL-78204

commit e4414b0
Author: Jiri Olsa <[email protected]>
Date:   Thu Aug 14 22:06:55 2025 +0200

    bpf: Check the helper function is valid in get_helper_proto

    kernel test robot reported verifier bug [1] where the helper func
    pointer could be NULL due to disabled config option.

    As Alexei suggested we could check on that in get_helper_proto
    directly. Marking tail_call helper func with BPF_PTR_POISON,
    because it is unused by design.

      [1] https://lore.kernel.org/oe-lkp/[email protected]

    Reported-by: kernel test robot <[email protected]>
    Reported-by: [email protected]
    Suggested-by: Alexei Starovoitov <[email protected]>
    Signed-off-by: Jiri Olsa <[email protected]>
    Signed-off-by: Daniel Borkmann <[email protected]>
    Acked-by: Paul Chaignon <[email protected]>
    Acked-by: Daniel Borkmann <[email protected]>
    Link: https://lore.kernel.org/bpf/[email protected]
    Closes: https://lore.kernel.org/oe-lkp/[email protected]

Signed-off-by: Jerome Marchand <[email protected]>

Author: jeromemarchand

kernel/bpf/core.c

@@ -3029,7 +3029,10 @@ EXPORT_SYMBOL_GPL(bpf_event_output);
 
 /* Always built-in helper functions. */
 const struct bpf_func_proto bpf_tail_call_proto = {
-	.func		= NULL,
+	/* func is unused for tail_call, we set it to pass the
+	 * get_helper_proto check
+	 */
+	.func		= BPF_PTR_POISON,
 	.gpl_only	= false,
 	.ret_type	= RET_VOID,
 	.arg1_type	= ARG_PTR_TO_CTX,

kernel/bpf/verifier.c

@@ -11358,7 +11358,7 @@ static int get_helper_proto(struct bpf_verifier_env *env, int func_id,
 		return -EINVAL;
 
 	*ptr = env->ops->get_func_proto(func_id, env->prog);
-	return *ptr ? 0 : -EINVAL;
+	return *ptr && (*ptr)->func ? 0 : -EINVAL;
 }
 
 static int check_helper_call(struct bpf_verifier_env *env, struct bpf_insn *insn,