|
From: Bert W. <ber...@go...> - 2010-07-13 11:59:31
|
The loop checks whether the next chunk starts before the current
chunk, but the chunk number printed was that of the current chunk.
Fix this.
Regards,
Bert
Index: memcheck/mc_malloc_wrappers.c
===================================================================
--- memcheck/mc_malloc_wrappers.c (revision 11212)
+++ memcheck/mc_malloc_wrappers.c (working copy)
@@ -618,7 +618,7 @@
if (chunks[i]->data + chunks[i]->szB > chunks[i+1]->data ) {
VG_(message)(Vg_UserMsg,
"Mempool chunk %d / %d overlaps with its successor\n",
- i+1, n_chunks);
+ i+2, n_chunks);
bad = 1;
}
}
|
|
From: Bert W. <ber...@go...> - 2010-07-13 12:41:09
|
On Tue, Jul 13, 2010 at 13:59, Bert Wesarg <ber...@go...> wrote:
> The loop checks whether the next chunk starts before the current
> chunk, but the chunk number printed was that of the current chunk.
Sorry, I have misread the message, I read "predecessor" not
"successor". Please ignore this patch.
>
> Fix this.
>
> Regards,
> Bert
>
> Index: memcheck/mc_malloc_wrappers.c
> ===================================================================
> --- memcheck/mc_malloc_wrappers.c (revision 11212)
> +++ memcheck/mc_malloc_wrappers.c (working copy)
> @@ -618,7 +618,7 @@
> if (chunks[i]->data + chunks[i]->szB > chunks[i+1]->data ) {
> VG_(message)(Vg_UserMsg,
> "Mempool chunk %d / %d overlaps with its successor\n",
> - i+1, n_chunks);
> + i+2, n_chunks);
> bad = 1;
> }
> }
>
|