Re: [PATCH bpf 5/9] bpf: Check the validity of max_rdwr_access for sk storage map iterator

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Subject: Re: [PATCH bpf 5/9] bpf: Check the validity of max_rdwr_access for sk storage map iterator
From: Yonghong Song <yhs@xxxxxx>
Date: Mon, 8 Aug 2022 07:56:17 -0700
Cc: Andrii Nakryiko <andrii@xxxxxxxxxx>, Alexei Starovoitov <ast@xxxxxxxxxx>, Daniel Borkmann <daniel@xxxxxxxxxxxxx>, Martin KaFai Lau <kafai@xxxxxx>, Song Liu <songliubraving@xxxxxx>, KP Singh <kpsingh@xxxxxxxxxx>, "David S . Miller" <davem@xxxxxxxxxxxxx>, Jakub Kicinski <kuba@xxxxxxxxxx>, Stanislav Fomichev <sdf@xxxxxxxxxx>, Hao Luo <haoluo@xxxxxxxxxx>, Jiri Olsa <jolsa@xxxxxxxxxx>, John Fastabend <john.fastabend@xxxxxxxxx>, Lorenz Bauer <lmb@xxxxxxxxxxxxxx>, houtao1@xxxxxxxxxx
In-reply-to: <20220806074019.2756957-6-houtao@huaweicloud.com>
User-agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:91.0) Gecko/20100101 Thunderbird/91.12.0



On 8/6/22 12:40 AM, Hou Tao wrote:

From: Hou Tao <houtao1@xxxxxxxxxx>

The value of sock map is writable in map iterator, so check
max_rdwr_access instead of max_rdonly_access.

Fixes: 5ce6e77c7edf ("bpf: Implement bpf iterator for sock local storage map")
Signed-off-by: Hou Tao <houtao1@xxxxxxxxxx>


Acked-by: Yonghong Song <yhs@xxxxxx>

References:
- [PATCH bpf 0/9] fixes for bpf map iterator
  - From: Hou Tao
- [PATCH bpf 5/9] bpf: Check the validity of max_rdwr_access for sk storage map iterator
  - From: Hou Tao

Prev by Date: Re: [PATCH bpf 4/9] bpf: Acquire map uref in .init_seq_private for sock{map,hash} iterator
Next by Date: Re: [PATCH bpf 6/9] bpf: Only allow sleepable program for resched-able iterator
Previous by thread: [PATCH bpf 5/9] bpf: Check the validity of max_rdwr_access for sk storage map iterator
Next by thread: Re: [PATCH bpf 5/9] bpf: Check the validity of max_rdwr_access for sk storage map iterator
Index(es):
- Date
- Thread

[Index of Archives] [Linux Samsung SoC] [Linux Rockchip SoC] [Linux Actions SoC] [Linux for Synopsys ARC Processors] [Linux NFS] [Linux NILFS] [Linux USB Devel] [Video for Linux] [Linux Audio Users] [Yosemite News] [Linux Kernel] [Linux SCSI]