[v2] tst-mallinfo2.c: Use correct multiple for total variable

Message ID 1613616993-2623-1-git-send-email-xuyang2018.jy@cn.fujitsu.com
State New
Headers show
Series
  • [v2] tst-mallinfo2.c: Use correct multiple for total variable
Related show

Commit Message

Yang Xu Feb. 18, 2021, 2:56 a.m.
Since test uses 160 multiple for malloc size, we should also use 160 multiple
for total variable instead of 16, then comparison is meaningful. So fix it.

Also change the ">" to ">=" so that the test is technically valid.
---
 malloc/tst-mallinfo2.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

-- 
2.23.0

Comments

Florian Weimer via Libc-alpha Feb. 23, 2021, 2:29 a.m. | #1
On 2/17/21 9:56 PM, Yang Xu wrote:
> Since test uses 160 multiple for malloc size, we should also use 160 multiple

> for total variable instead of 16, then comparison is meaningful. So fix it.

 
> Also change the ">" to ">=" so that the test is technically valid.


Yang,

Thank you very much for your great work with this patch.

I need your help in order to complete the FSF copyright assignment process
with Fujitsu.

I've sent you an Naohiro and email to start the process.

Thank you for your help.

> ---

>  malloc/tst-mallinfo2.c | 4 ++--

>  1 file changed, 2 insertions(+), 2 deletions(-)

> 

> diff --git a/malloc/tst-mallinfo2.c b/malloc/tst-mallinfo2.c

> index 59a15cf7a8..7d00b65f6e 100644

> --- a/malloc/tst-mallinfo2.c

> +++ b/malloc/tst-mallinfo2.c

> @@ -68,14 +68,14 @@ do_test (void)

>    for (i = 1; i < 20; ++i)

>      {

>        ptr = malloc (160 * i);

> -      total += 16 * i;

> +      total += 160 * i;

>      }

>  

>    mi2 = mallinfo2 ();

>    print_mi ("after", &mi2);

>  

>    /* Check at least something changed.  */

> -  TEST_VERIFY (mi2.uordblks > mi1.uordblks + total);

> +  TEST_VERIFY (mi2.uordblks >= mi1.uordblks + total);

>  

>    return 0;

>  }

> 



-- 
Cheers,
Carlos.

Patch

diff --git a/malloc/tst-mallinfo2.c b/malloc/tst-mallinfo2.c
index 59a15cf7a8..7d00b65f6e 100644
--- a/malloc/tst-mallinfo2.c
+++ b/malloc/tst-mallinfo2.c
@@ -68,14 +68,14 @@  do_test (void)
   for (i = 1; i < 20; ++i)
     {
       ptr = malloc (160 * i);
-      total += 16 * i;
+      total += 160 * i;
     }
 
   mi2 = mallinfo2 ();
   print_mi ("after", &mi2);
 
   /* Check at least something changed.  */
-  TEST_VERIFY (mi2.uordblks > mi1.uordblks + total);
+  TEST_VERIFY (mi2.uordblks >= mi1.uordblks + total);
 
   return 0;
 }