]> git.baikalelectronics.ru Git - kernel.git/commit
bpf: Fix wrong reg type conversion in release_reference()
authorYoulin Li <liulin063@gmail.com>
Thu, 3 Nov 2022 09:34:39 +0000 (17:34 +0800)
committerDaniel Borkmann <daniel@iogearbox.net>
Thu, 3 Nov 2022 23:24:12 +0000 (00:24 +0100)
commita1c6aba87306768c024ec90402d903919c06edf7
treefd056852b9da624c5355d39a81032e9410053c36
parent0de014b6f1587e18a606bbfd00c1e6e0ff42b971
bpf: Fix wrong reg type conversion in release_reference()

Some helper functions will allocate memory. To avoid memory leaks, the
verifier requires the eBPF program to release these memories by calling
the corresponding helper functions.

When a resource is released, all pointer registers corresponding to the
resource should be invalidated. The verifier use release_references() to
do this job, by apply  __mark_reg_unknown() to each relevant register.

It will give these registers the type of SCALAR_VALUE. A register that
will contain a pointer value at runtime, but of type SCALAR_VALUE, which
may allow the unprivileged user to get a kernel pointer by storing this
register into a map.

Using __mark_reg_not_init() while NOT allow_ptr_leaks can mitigate this
problem.

Fixes: 8825dfd9e473 ("bpf: Add reference tracking to verifier")
Signed-off-by: Youlin Li <liulin063@gmail.com>
Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
Link: https://lore.kernel.org/bpf/20221103093440.3161-1-liulin063@gmail.com
kernel/bpf/verifier.c