Augment a comment.

git-svn-id: svn://svn.valgrind.org/valgrind/trunk@10714

diff --git a/memcheck/mc_main.c b/memcheck/mc_main.c
index 7929d245410bcbaacf863761d09ba2b03bb24452..d83464fcda394863cf53461e255700d1c90e6a30 100644 (file)
--- a/memcheck/mc_main.c
+++ b/memcheck/mc_main.c
@@ -5780,6 +5780,12 @@ static void mc_pre_clo_init(void)
    // - Telying on the zeroed-ness of whole brk'd pages is pretty grotty... I
    //   doubt most programmers know the above information.
    // So I'm not terribly unhappy with marking it as undefined. --njn.
+   //
+   // [More:  I think most of what John said only applies to sbrk().  It seems
+   // that brk() always deals in whole pages.  And since this event deals
+   // directly with brk(), not with sbrk(), perhaps it would be reasonable to
+   // just mark all memory it allocates as defined.]
+   //
    VG_(track_new_mem_brk)         ( make_mem_undefined_w_tid );
    VG_(track_new_mem_mmap)        ( mc_new_mem_mmap );