| Message ID | YtZzjvHXVXMXxpXO@kili (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | selftest/vm: uninitialized variable in main() | expand |
On Tue, Jul 19, 2022 at 2:04 AM Dan Carpenter <dan.carpenter@oracle.com> wrote: > > Initialize "length" to zero by default. > > Fixes: ff712a627f72 ("selftests/vm: cleanup hugetlb file after mremap test") > Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Thanks, Reviewed-by: Mina Almasry <almasrymina@google.com> > --- > tools/testing/selftests/vm/hugepage-mremap.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/tools/testing/selftests/vm/hugepage-mremap.c b/tools/testing/selftests/vm/hugepage-mremap.c > index 585978f181ed..e63a0214f639 100644 > --- a/tools/testing/selftests/vm/hugepage-mremap.c > +++ b/tools/testing/selftests/vm/hugepage-mremap.c > @@ -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 >
On Tue, Jul 19, 2022 at 5:04 PM Dan Carpenter <dan.carpenter@oracle.com> wrote: > > Initialize "length" to zero by default. > > Fixes: ff712a627f72 ("selftests/vm: cleanup hugetlb file after mremap test") > Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Reviewed-by: Muchun Song <songmuchun@bytedance.com> Thanks.
diff --git a/tools/testing/selftests/vm/hugepage-mremap.c b/tools/testing/selftests/vm/hugepage-mremap.c index 585978f181ed..e63a0214f639 100644 --- a/tools/testing/selftests/vm/hugepage-mremap.c +++ b/tools/testing/selftests/vm/hugepage-mremap.c @@ -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]);
Initialize "length" to zero by default. Fixes: ff712a627f72 ("selftests/vm: cleanup hugetlb file after mremap test") Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> --- tools/testing/selftests/vm/hugepage-mremap.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)