2022-04-22 21:30:33

by Jiapeng Chong

Subject: [PATCH] x86/kaslr: Return true/false (not 1/0) from bool functions

Fix the following coccicheck warning:

./arch/x86/boot/compressed/kaslr.c:670:8-9: WARNING: return of 0/1 in
function 'process_mem_region' with return type bool.

Reported-by: Abaci Robot <[email protected]>
Signed-off-by: Jiapeng Chong <[email protected]>
---
arch/x86/boot/compressed/kaslr.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/x86/boot/compressed/kaslr.c b/arch/x86/boot/compressed/kaslr.c
index 4a3f223973f4..43995c59e652 100644
--- a/arch/x86/boot/compressed/kaslr.c
+++ b/arch/x86/boot/compressed/kaslr.c
@@ -667,7 +667,7 @@ static bool process_mem_region(struct mem_vector *region,
}
}
#endif
- return 0;
+ return false;
}

#ifdef CONFIG_EFI
--
2.20.1.7.g153144c

2022-11-19 17:26:46

by tip-bot2 for Haifeng Xu

Subject: [tip: x86/cleanups] x86/kaslr: Fix process_mem_region()'s return value

The following commit has been merged into the x86/cleanups branch of tip:

Commit-ID: ee92fa03918d114d3ac9c36a8bf2c032ede75a3b
Gitweb: https://git.kernel.org/tip/ee92fa03918d114d3ac9c36a8bf2c032ede75a3b
Author: Jiapeng Chong <[email protected]>
AuthorDate: Fri, 22 Apr 2022 04:25:56 +08:00
Committer: Borislav Petkov <[email protected]>
CommitterDate: Sat, 19 Nov 2022 17:35:10 +01:00

x86/kaslr: Fix process_mem_region()'s return value

Fix the following coccicheck warning:

./arch/x86/boot/compressed/kaslr.c:670:8-9: WARNING: return of 0/1 in
function 'process_mem_region' with return type bool.

Reported-by: Abaci Robot <[email protected]>
Signed-off-by: Jiapeng Chong <[email protected]>
Signed-off-by: Borislav Petkov <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
---
arch/x86/boot/compressed/kaslr.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/x86/boot/compressed/kaslr.c b/arch/x86/boot/compressed/kaslr.c
index e476bcb..454757f 100644
--- a/arch/x86/boot/compressed/kaslr.c
+++ b/arch/x86/boot/compressed/kaslr.c
@@ -668,7 +668,7 @@ static bool process_mem_region(struct mem_vector *region,
}
}
#endif
- return 0;
+ return false;
}

#ifdef CONFIG_EFI