tree 99fbb8dd4af0cd1f02543fe6c83ac419717e99b3
parent 97773ef80fa9721d6885ad1de7e5c045d7d52de2
author Christian Brauner <christian.brauner@ubuntu.com> 1570635625 +0200
committer Christian Brauner <christian.brauner@ubuntu.com> 1571186111 +0200
gpgsig -----BEGIN PGP SIGNATURE-----
 
 iHUEABYKAB0WIQRAhzRXHqcMeLMyaSiRxhvAZXjcogUCXaZlvwAKCRCRxhvAZXjc
 oiGpAP419GupYzNReGbyK4d4I7F2CK9mda8LhCqfs1vOBth0UQEA7vVQVkfe+enX
 twpdhd1LkBKOjRgqibc3orcTab8YRgg=
 =7bDB
 -----END PGP SIGNATURE-----

bpf: use copy_struct_from_user() in bpf() syscall

In v5.4-rc2 we added a new helper (cf. [1]) copy_struct_from_user().
This helper is intended for all codepaths that copy structs from
userspace that are versioned by size. The bpf() syscall does exactly
what copy_struct_from_user() is doing.
Note that copy_struct_from_user() is calling min() already. So
technically, the min_t() call could go. But the size is used further
below so leave it.

[1]: f5a1a536fa14 ("lib: introduce copy_struct_from_user() helper")
Cc: Alexei Starovoitov <ast@kernel.org>
Cc: Daniel Borkmann <daniel@iogearbox.net>
Cc: bpf@vger.kernel.org
Acked-by: Aleksa Sarai <cyphar@cyphar.com>
Signed-off-by: Christian Brauner <christian.brauner@ubuntu.com>