* [PATCH linux-next] tools: fix warning comparing pointer to 0
@ 2021-08-20 3:30 CGEL
2021-08-23 15:54 ` Daniel Borkmann
0 siblings, 1 reply; 2+ messages in thread
From: CGEL @ 2021-08-20 3:30 UTC (permalink / raw)
To: Shuah Khan
Cc: Alexei Starovoitov, Daniel Borkmann, Andrii Nakryiko,
Martin KaFai Lau, Song Liu, John Fastabend, KP Singh,
Andrei Matei, linux-kselftest, netdev, bpf, linux-kernel,
jing yangyang, Zeal Robot
From: jing yangyang <jing.yangyang@zte.com.cn>
Fix the following coccicheck warning:
./tools/testing/selftests/bpf/progs/profiler.inc.h:364:18-22:WARNING
comparing pointer to 0
./tools/testing/selftests/bpf/progs/profiler.inc.h:537:23-27:WARNING
comparing pointer to 0
./tools/testing/selftests/bpf/progs/profiler.inc.h:544:21-25:WARNING
comparing pointer to 0
./tools/testing/selftests/bpf/progs/profiler.inc.h:770:13-17:WARNING
comparing pointer to 0
Reported-by: Zeal Robot <zealci@zte.com.cn>
Signed-off-by: jing yangyang <jing.yangyang@zte.com.cn>
---
tools/testing/selftests/bpf/progs/profiler.inc.h | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/tools/testing/selftests/bpf/progs/profiler.inc.h b/tools/testing/selftests/bpf/progs/profiler.inc.h
index 4896fdf..5c0bdab 100644
--- a/tools/testing/selftests/bpf/progs/profiler.inc.h
+++ b/tools/testing/selftests/bpf/progs/profiler.inc.h
@@ -361,7 +361,7 @@ static INLINE void* populate_var_metadata(struct var_metadata_t* metadata,
int zero = 0;
struct var_kill_data_t* kill_data = bpf_map_lookup_elem(&data_heap, &zero);
- if (kill_data == NULL)
+ if (!kill_dat)
return NULL;
struct task_struct* task = (struct task_struct*)bpf_get_current_task();
@@ -534,14 +534,14 @@ static INLINE bool is_dentry_allowed_for_filemod(struct dentry* file_dentry,
*device_id = dev_id;
bool* allowed_device = bpf_map_lookup_elem(&allowed_devices, &dev_id);
- if (allowed_device == NULL)
+ if (!allowed_device)
return false;
u64 ino = BPF_CORE_READ(file_dentry, d_inode, i_ino);
*file_ino = ino;
bool* allowed_file = bpf_map_lookup_elem(&allowed_file_inodes, &ino);
- if (allowed_file == NULL)
+ if (!allowed_fil)
if (!is_ancestor_in_allowed_inodes(BPF_CORE_READ(file_dentry, d_parent)))
return false;
return true;
@@ -689,7 +689,7 @@ int raw_tracepoint__sched_process_exec(struct bpf_raw_tracepoint_args* ctx)
u64 inode = BPF_CORE_READ(bprm, file, f_inode, i_ino);
bool* should_filter_binprm = bpf_map_lookup_elem(&disallowed_exec_inodes, &inode);
- if (should_filter_binprm != NULL)
+ if (should_filter_binprm)
goto out;
int zero = 0;
--
1.8.3.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH linux-next] tools: fix warning comparing pointer to 0
2021-08-20 3:30 [PATCH linux-next] tools: fix warning comparing pointer to 0 CGEL
@ 2021-08-23 15:54 ` Daniel Borkmann
0 siblings, 0 replies; 2+ messages in thread
From: Daniel Borkmann @ 2021-08-23 15:54 UTC (permalink / raw)
To: CGEL, Shuah Khan
Cc: Alexei Starovoitov, Andrii Nakryiko, Martin KaFai Lau, Song Liu,
John Fastabend, KP Singh, Andrei Matei, linux-kselftest, netdev,
bpf, linux-kernel, jing yangyang, Zeal Robot
On 8/20/21 5:30 AM, CGEL wrote:
> From: jing yangyang <jing.yangyang@zte.com.cn>
>
> Fix the following coccicheck warning:
> ./tools/testing/selftests/bpf/progs/profiler.inc.h:364:18-22:WARNING
> comparing pointer to 0
> ./tools/testing/selftests/bpf/progs/profiler.inc.h:537:23-27:WARNING
> comparing pointer to 0
> ./tools/testing/selftests/bpf/progs/profiler.inc.h:544:21-25:WARNING
> comparing pointer to 0
> ./tools/testing/selftests/bpf/progs/profiler.inc.h:770:13-17:WARNING
> comparing pointer to 0
>
> Reported-by: Zeal Robot <zealci@zte.com.cn>
> Signed-off-by: jing yangyang <jing.yangyang@zte.com.cn>
Please properly explain in the commit message what this 'fixes' exactly and
why it is needed.
> ---
> tools/testing/selftests/bpf/progs/profiler.inc.h | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/tools/testing/selftests/bpf/progs/profiler.inc.h b/tools/testing/selftests/bpf/progs/profiler.inc.h
> index 4896fdf..5c0bdab 100644
> --- a/tools/testing/selftests/bpf/progs/profiler.inc.h
> +++ b/tools/testing/selftests/bpf/progs/profiler.inc.h
> @@ -361,7 +361,7 @@ static INLINE void* populate_var_metadata(struct var_metadata_t* metadata,
> int zero = 0;
> struct var_kill_data_t* kill_data = bpf_map_lookup_elem(&data_heap, &zero);
>
> - if (kill_data == NULL)
> + if (!kill_dat)
And please don't send broken stuff like this.
> return NULL;
> struct task_struct* task = (struct task_struct*)bpf_get_current_task();
>
> @@ -534,14 +534,14 @@ static INLINE bool is_dentry_allowed_for_filemod(struct dentry* file_dentry,
> *device_id = dev_id;
> bool* allowed_device = bpf_map_lookup_elem(&allowed_devices, &dev_id);
>
> - if (allowed_device == NULL)
> + if (!allowed_device)
> return false;
>
> u64 ino = BPF_CORE_READ(file_dentry, d_inode, i_ino);
> *file_ino = ino;
> bool* allowed_file = bpf_map_lookup_elem(&allowed_file_inodes, &ino);
>
> - if (allowed_file == NULL)
> + if (!allowed_fil)
... same. You did not bother to compile test even.
> if (!is_ancestor_in_allowed_inodes(BPF_CORE_READ(file_dentry, d_parent)))
> return false;
> return true;
> @@ -689,7 +689,7 @@ int raw_tracepoint__sched_process_exec(struct bpf_raw_tracepoint_args* ctx)
> u64 inode = BPF_CORE_READ(bprm, file, f_inode, i_ino);
>
> bool* should_filter_binprm = bpf_map_lookup_elem(&disallowed_exec_inodes, &inode);
> - if (should_filter_binprm != NULL)
> + if (should_filter_binprm)
> goto out;
>
> int zero = 0;
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2021-08-23 15:54 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-08-20 3:30 [PATCH linux-next] tools: fix warning comparing pointer to 0 CGEL
2021-08-23 15:54 ` Daniel Borkmann
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).