]> granicus.if.org Git - php/commitdiff
add ability to show valgrind output with --show-mem
authorJoe Watkins <krakjoe@php.net>
Fri, 2 Mar 2018 12:14:35 +0000 (13:14 +0100)
committerJoe Watkins <krakjoe@php.net>
Fri, 2 Mar 2018 12:14:35 +0000 (13:14 +0100)
run-tests.php [changed mode: 0755->0644]

old mode 100755 (executable)
new mode 100644 (file)
index cbef1dc..e22106e
@@ -497,7 +497,7 @@ $no_clean = false;
 $slow_min_ms = INF;
 
 $cfgtypes = array('show', 'keep');
-$cfgfiles = array('skip', 'php', 'clean', 'out', 'diff', 'exp');
+$cfgfiles = array('skip', 'php', 'clean', 'out', 'diff', 'exp', 'mem');
 $cfg = array();
 
 foreach($cfgtypes as $type) {
@@ -860,7 +860,7 @@ HELP;
 
                junit_save_xml();
 
-               if (getenv('REPORT_EXIT_STATUS') == 1 && ($sum_results['FAILED'] || $sum_results['BORKED'])) {
+               if (getenv('REPORT_EXIT_STATUS') == 1 && ($sum_results['FAILED'] || $sum_results['BORKED'] || $sum_results['LEAKED'])) {
                        exit(1);
                }
 
@@ -997,8 +997,7 @@ if ($html_output) {
 save_or_mail_results();
 
 junit_save_xml();
-
-if (getenv('REPORT_EXIT_STATUS') == 1 && ($sum_results['FAILED'] || $sum_results['BORKED'])) {
+if (getenv('REPORT_EXIT_STATUS') == 1 && ($sum_results['FAILED'] || $sum_results['LEAK'] || $sum_results['LEAKED'])) {
        exit(1);
 }
 exit(0);
@@ -2215,6 +2214,10 @@ $output
                        error_report($file, $log_filename, $tested);
                }
        }
+       
+       if ($valgrind && $leaked && $cfg["show"]["mem"]) {
+               show_file_block('mem', file_get_contents($memcheck_filename));
+       }
 
        show_result(implode('&', $restype), $tested, $tested_file, $info, $temp_filenames);