diff options
author | Breno Leitao <leitao@debian.org> | 2023-10-16 06:47:42 -0700 |
---|---|---|
committer | Jens Axboe <axboe@kernel.dk> | 2023-10-19 16:41:37 -0600 |
commit | 0b05b0cd78c92371fdde6333d006f39eaf9e0860 (patch) | |
tree | 47f27adbfa440cc83bdc820a382ff514247966cd /include/linux/bpf-cgroup.h | |
parent | 1406245c29454ff84919736be83e14cdaba7fec1 (diff) | |
download | linux-0b05b0cd78c92371fdde6333d006f39eaf9e0860.tar.gz linux-0b05b0cd78c92371fdde6333d006f39eaf9e0860.tar.bz2 linux-0b05b0cd78c92371fdde6333d006f39eaf9e0860.zip |
net/socket: Break down __sys_getsockopt
Split __sys_getsockopt() into two functions by removing the core
logic into a sub-function (do_sock_getsockopt()). This will avoid
code duplication when doing the same operation in other callers, for
instance.
do_sock_getsockopt() will be called by io_uring getsockopt() command
operation in the following patch.
The same was done for the setsockopt pair.
Suggested-by: Martin KaFai Lau <martin.lau@linux.dev>
Signed-off-by: Breno Leitao <leitao@debian.org>
Acked-by: Jakub Kicinski <kuba@kernel.org>
Acked-by: Martin KaFai Lau <martin.lau@kernel.org>
Link: https://lore.kernel.org/r/20231016134750.1381153-5-leitao@debian.org
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'include/linux/bpf-cgroup.h')
-rw-r--r-- | include/linux/bpf-cgroup.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/linux/bpf-cgroup.h b/include/linux/bpf-cgroup.h index cecfe8c99f28..ffaca1ab5e8d 100644 --- a/include/linux/bpf-cgroup.h +++ b/include/linux/bpf-cgroup.h @@ -378,7 +378,7 @@ static inline bool cgroup_bpf_sock_enabled(struct sock *sk, ({ \ int __ret = 0; \ if (cgroup_bpf_enabled(CGROUP_GETSOCKOPT)) \ - get_user(__ret, optlen); \ + copy_from_sockptr(&__ret, optlen, sizeof(int)); \ __ret; \ }) |