[LTP] [PATCH v2] syscalls/bpf: zero-initialize bpf_attr including padding bits
Jan Stancek
jstancek@redhat.com
Thu Feb 6 22:23:44 CET 2025
gcc 15 stopped zero-initializing padding bits:
https://gcc.gnu.org/gcc-15/changes.html
However kernel bpf syscall checks that all unused fields for a command
are set to zero in CHECK_ATTR() macro, which causes tests to fail with
EINVAL.
Signed-off-by: Jan Stancek <jstancek@redhat.com>
---
v2 changes:
- Use slightly better version suggested by Cyril
testcases/kernel/syscalls/bpf/bpf_common.c | 14 +++++++++++---
1 file changed, 11 insertions(+), 3 deletions(-)
diff --git a/testcases/kernel/syscalls/bpf/bpf_common.c b/testcases/kernel/syscalls/bpf/bpf_common.c
index 95b5bc12eaa4..9148b0437279 100644
--- a/testcases/kernel/syscalls/bpf/bpf_common.c
+++ b/testcases/kernel/syscalls/bpf/bpf_common.c
@@ -49,7 +49,10 @@ int bpf_map_create(union bpf_attr *const attr)
int bpf_map_array_create(const uint32_t max_entries)
{
- union bpf_attr map_attr = {
+ /* zero-initialize entire struct including padding bits */
+ union bpf_attr map_attr = {};
+
+ map_attr = (union bpf_attr) {
.map_type = BPF_MAP_TYPE_ARRAY,
.key_size = 4,
.value_size = 8,
@@ -64,13 +67,18 @@ void bpf_map_array_get(const int map_fd,
const uint32_t *const array_indx,
uint64_t *const array_val)
{
- union bpf_attr elem_attr = {
+ /* zero-initialize entire struct including padding bits */
+ union bpf_attr elem_attr = {};
+ int ret;
+
+ elem_attr = (union bpf_attr) {
.map_fd = map_fd,
.key = ptr_to_u64(array_indx),
.value = ptr_to_u64(array_val),
.flags = 0
};
- const int ret = bpf(BPF_MAP_LOOKUP_ELEM, &elem_attr, sizeof(elem_attr));
+
+ ret = bpf(BPF_MAP_LOOKUP_ELEM, &elem_attr, sizeof(elem_attr));
if (ret) {
tst_brk(TBROK | TTERRNO,
--
2.43.0
More information about the ltp
mailing list