diff options
| author | Ulrich Drepper <drepper@redhat.com> | 2009-06-18 22:37:31 -0700 |
|---|---|---|
| committer | Ulrich Drepper <drepper@redhat.com> | 2009-06-18 22:37:31 -0700 |
| commit | f6887a0d9a55f5c80c567d9cb153c1c6582410f9 (patch) | |
| tree | 1136d80d15117cb3acee0bee871635ba647668b1 /malloc | |
| parent | d0a2af710654a038903dd4a300030670bfbeaa2d (diff) | |
| download | glibc-f6887a0d9a55f5c80c567d9cb153c1c6582410f9.tar glibc-f6887a0d9a55f5c80c567d9cb153c1c6582410f9.tar.gz glibc-f6887a0d9a55f5c80c567d9cb153c1c6582410f9.tar.bz2 glibc-f6887a0d9a55f5c80c567d9cb153c1c6582410f9.zip | |
Add some more tests to malloc to detect corruptions.
Diffstat (limited to 'malloc')
| -rw-r--r-- | malloc/malloc.c | 29 |
1 files changed, 27 insertions, 2 deletions
diff --git a/malloc/malloc.c b/malloc/malloc.c index 0b9facefd4..516d401991 100644 --- a/malloc/malloc.c +++ b/malloc/malloc.c @@ -4241,6 +4241,8 @@ _int_malloc(mstate av, size_t bytes) mchunkptr fwd; /* misc temp for linking */ mchunkptr bck; /* misc temp for linking */ + const char *errstr = NULL; + /* Convert request size to internal form by adding SIZE_SZ bytes overhead plus possibly more to obtain necessary alignment and/or @@ -4276,8 +4278,11 @@ _int_malloc(mstate av, size_t bytes) #endif if (victim != 0) { if (__builtin_expect (fastbin_index (chunksize (victim)) != idx, 0)) - malloc_printerr (check_action, "malloc(): memory corruption (fast)", - chunk2mem (victim)); + { + errstr = "malloc(): memory corruption (fast)"; + errout: + malloc_printerr (check_action, errstr, chunk2mem (victim)); + } #ifndef ATOMIC_FASTBINS *fb = victim->fd; #endif @@ -4306,6 +4311,11 @@ _int_malloc(mstate av, size_t bytes) malloc_consolidate(av); else { bck = victim->bk; + if (__builtin_expect (bck->fd != victim, 0)) + { + errstr = "malloc(): smallbin double linked list corrupted"; + goto errout; + } set_inuse_bit_at_offset(victim, nb); bin->bk = bck; bck->fd = bin; @@ -4515,6 +4525,11 @@ _int_malloc(mstate av, size_t bytes) have to perform a complete insert here. */ bck = unsorted_chunks(av); fwd = bck->fd; + if (__builtin_expect (fwd->bk != bck, 0)) + { + errstr = "malloc(): corrupted unsorted chunks"; + goto errout; + } remainder->bk = bck; remainder->fd = fwd; bck->fd = remainder; @@ -4610,6 +4625,11 @@ _int_malloc(mstate av, size_t bytes) have to perform a complete insert here. */ bck = unsorted_chunks(av); fwd = bck->fd; + if (__builtin_expect (fwd->bk != bck, 0)) + { + errstr = "malloc(): corrupted unsorted chunks 2"; + goto errout; + } remainder->bk = bck; remainder->fd = fwd; bck->fd = remainder; @@ -4901,6 +4921,11 @@ _int_free(mstate av, mchunkptr p) bck = unsorted_chunks(av); fwd = bck->fd; + if (__builtin_expect (fwd->bk != bck, 0)) + { + errstr = "free(): corrupted unsorted chunks"; + goto errout; + } p->fd = fwd; p->bk = bck; if (!in_smallbin_range(size)) |