Skip to contentSkip to conversation

selftest/vm: uninitialized variable in main()

PATCH Applied Dan Carpenter 3 participants 3 messages Jul 19, 2022 – Jul 20, 2022

Subsystems (3)
Memory management Kselftest Kernel janitors
lore
Share & tools
lore

Dan Carpenter

author
#0 lore raw

PATCH 1/1 selftest/vm: uninitialized variable in main()

Initialize "length" to zero by default.

Fixes ff712a627f72 ("selftests/vm: cleanup hugetlb file after mremap test")
Signed-off-by Dan Carpenter <[email protected]>
---
 tools/testing/selftests/vm/hugepage-mremap.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
tools/testing/selftests/vm/hugepage-mremap.c +1 -1
index 585978f181ed..e63a0214f639 100644
@@ -107,7 +107,7 @@ static void register_region_with_uffd(char *addr, size_t len)
 
 int main(int argc, char *argv[])
 {
-	size_t length;
+	size_t length = 0;
 
 	if (argc != 2 && argc != 3) {
 		printf("Usage: %s [length_in_MB] <hugetlb_file>\n", argv[0]);
Signature · 1 line
2.35.1
Reviewed-by on v1 1/1
#1 lore raw
Quoted 5 lines · Dan Carpenter

Initialize "length" to zero by default.

Fixes ff712a627f72 ("selftests/vm: cleanup hugetlb file after mremap test")
Signed-off-by Dan Carpenter <[email protected]>

Thanks,

Reviewed-by Mina Almasry <[email protected]>
Quoted 20 lines
---
 tools/testing/selftests/vm/hugepage-mremap.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
tools/testing/selftests/vm/hugepage-mremap.c +1 -2
index 585978f181ed..e63a0214f639 100644
@@ -107,7 +107,7 @@ static void register_region_with_uffd(char *addr, size_t len)
 
 int main(int argc, char *argv[])
 {
-       size_t length;
+       size_t length = 0;
 
        if (argc != 2 && argc != 3) {
                printf("Usage: %s [length_in_MB] <hugetlb_file>\n", argv[0]);
--

2.35.1