public inbox for glibc-cvs@sourceware.org
help / color / mirror / Atom feed
* [glibc] malloc debug: fix compile error when enable macro MALLOC_DEBUG > 1
@ 2020-10-30 18:53 DJ Delorie
  0 siblings, 0 replies; only message in thread
From: DJ Delorie @ 2020-10-30 18:53 UTC (permalink / raw)
  To: glibc-cvs

https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=961d12d9471f0178441e67e073ba2f3085cf20f1

commit 961d12d9471f0178441e67e073ba2f3085cf20f1
Author: liqingqing <liqingqing3@huawei.com>
Date:   Thu Oct 22 17:11:44 2020 +0800

    malloc debug: fix compile error when enable macro MALLOC_DEBUG > 1
    
    malloc debug: fix compile error when enable macro MALLOC_DEBUG > 1.
    
    this is because commit e9c4fe93b3855239752819303ca377dff0ed0553 has change the struct malloc_chunk's member "size" to "mchunk_size".
    
    the reproduction is like that:
    setp1: modify related Makefile.
    vim ../glibc/malloc/Makefile
    CPPFLAGS-malloc.o += -DMALLOC_DEBUG=2
    
    step2: ../configure --prefix=/usr
           make -j32
    
    this will cause the compile error:
    /home/liqingqing/glibc_upstream/buildglibc/malloc/malloc.o
    In file included from malloc.c:1899:0:
    arena.c: In function 'dump_heap':
    arena.c:422:58: error: 'struct malloc_chunk' has no member named 'size'
           fprintf (stderr, "chunk %p size %10lx", p, (long) p->size);
                                                              ^~
    arena.c:428:17: error: 'struct malloc_chunk' has no member named 'size'
           else if (p->size == (0 | PREV_INUSE))
    
    Reviewed-by: DJ Delorie <dj@redhat.com>

Diff:
---
 malloc/arena.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/malloc/arena.c b/malloc/arena.c
index cecdb7f4c4..202daf15b0 100644
--- a/malloc/arena.c
+++ b/malloc/arena.c
@@ -419,13 +419,13 @@ dump_heap (heap_info *heap)
                    ~MALLOC_ALIGN_MASK);
   for (;; )
     {
-      fprintf (stderr, "chunk %p size %10lx", p, (long) p->size);
+      fprintf (stderr, "chunk %p size %10lx", p, (long) chunksize_nomask(p));
       if (p == top (heap->ar_ptr))
         {
           fprintf (stderr, " (top)\n");
           break;
         }
-      else if (p->size == (0 | PREV_INUSE))
+      else if (chunksize_nomask(p) == (0 | PREV_INUSE))
         {
           fprintf (stderr, " (fence)\n");
           break;


^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2020-10-30 18:53 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-10-30 18:53 [glibc] malloc debug: fix compile error when enable macro MALLOC_DEBUG > 1 DJ Delorie

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 read-only IMAP folder(s) and NNTP newsgroup(s).