Add --track-origins=yes to valgrind

mattsta · antirez · commit 491d57abaae7 · 2015-01-21T15:48:19.000+01:00
diff --git a/tests/instances.tcl b/tests/instances.tcl
@@ -68,7 +68,7 @@ proc spawn_instance {type base_port count {conf {}}} {
         }
 
         if {$::valgrind} {
-            set pid [exec valgrind --suppressions=../../../src/valgrind.sup --show-reachable=no --show-possibly-lost=no --leak-check=full ../../../src/${prgname} $cfgfile &]
+            set pid [exec valgrind --track-origins=yes --suppressions=../../../src/valgrind.sup --show-reachable=no --show-possibly-lost=no --leak-check=full ../../../src/${prgname} $cfgfile &]
         } else {
             set pid [exec ../../../src/${prgname} $cfgfile &]
         }
@@ -401,7 +401,7 @@ proc restart_instance {type id} {
     }
 
     if {$::valgrind} {
-        set pid [exec valgrind --suppressions=../../../src/valgrind.sup --show-reachable=no --show-possibly-lost=no --leak-check=full ../../../src/${prgname} $cfgfile &]
+        set pid [exec valgrind --track-origins=yes --suppressions=../../../src/valgrind.sup --show-reachable=no --show-possibly-lost=no --leak-check=full ../../../src/${prgname} $cfgfile &]
     } else {
         set pid [exec ../../../src/${prgname} $cfgfile &]
     }
diff --git a/tests/support/server.tcl b/tests/support/server.tcl
@@ -207,7 +207,7 @@ proc start_server {options {code undefined}} {
     set stderr [format "%s/%s" [dict get $config "dir"] "stderr"]
 
     if {$::valgrind} {
-        set pid [exec valgrind --suppressions=src/valgrind.sup --show-reachable=no --show-possibly-lost=no --leak-check=full src/redis-server $config_file > $stdout 2> $stderr &]
+        set pid [exec valgrind --track-origins=yes --suppressions=src/valgrind.sup --show-reachable=no --show-possibly-lost=no --leak-check=full src/redis-server $config_file > $stdout 2> $stderr &]
     } else {
         set pid [exec src/redis-server $config_file > $stdout 2> $stderr &]
     }

Original file line number	Diff line number	Diff line change
`@@ -68,7 +68,7 @@ proc spawn_instance {type base_port count {conf {}}} {`
`68`	`68`	`}`
`69`	`69`
`70`	`70`	`if {$::valgrind} {`
`71`		`- set pid [exec valgrind --suppressions=../../../src/valgrind.sup --show-reachable=no --show-possibly-lost=no --leak-check=full ../../../src/${prgname} $cfgfile &]`
	`71`	`+ set pid [exec valgrind --track-origins=yes --suppressions=../../../src/valgrind.sup --show-reachable=no --show-possibly-lost=no --leak-check=full ../../../src/${prgname} $cfgfile &]`
`72`	`72`	`} else {`
`73`	`73`	`set pid [exec ../../../src/${prgname} $cfgfile &]`
`74`	`74`	`}`
`@@ -401,7 +401,7 @@ proc restart_instance {type id} {`
`401`	`401`	`}`
`402`	`402`
`403`	`403`	`if {$::valgrind} {`
`404`		`- set pid [exec valgrind --suppressions=../../../src/valgrind.sup --show-reachable=no --show-possibly-lost=no --leak-check=full ../../../src/${prgname} $cfgfile &]`
	`404`	`+ set pid [exec valgrind --track-origins=yes --suppressions=../../../src/valgrind.sup --show-reachable=no --show-possibly-lost=no --leak-check=full ../../../src/${prgname} $cfgfile &]`
`405`	`405`	`} else {`
`406`	`406`	`set pid [exec ../../../src/${prgname} $cfgfile &]`
`407`	`407`	`}`
Original file line number	Diff line number	Diff line change
`@@ -207,7 +207,7 @@ proc start_server {options {code undefined}} {`
`207`	`207`	`set stderr [format "%s/%s" [dict get $config "dir"] "stderr"]`
`208`	`208`
`209`	`209`	`if {$::valgrind} {`
`210`		`- set pid [exec valgrind --suppressions=src/valgrind.sup --show-reachable=no --show-possibly-lost=no --leak-check=full src/redis-server $config_file > $stdout 2> $stderr &]`
	`210`	`+ set pid [exec valgrind --track-origins=yes --suppressions=src/valgrind.sup --show-reachable=no --show-possibly-lost=no --leak-check=full src/redis-server $config_file > $stdout 2> $stderr &]`
`211`	`211`	`} else {`
`212`	`212`	`set pid [exec src/redis-server $config_file > $stdout 2> $stderr &]`
`213`	`213`	`}`