diff options
author | Daniel Stenberg <daniel@haxx.se> | 2016-12-21 15:10:14 +0100 |
---|---|---|
committer | Daniel Stenberg <daniel@haxx.se> | 2016-12-21 23:42:43 +0100 |
commit | 5c823f51dbbd15193350ae9efa94175abee92894 (patch) | |
tree | 1c09a627079ffefb178864ad5816cb47425e2160 /tests/valgrind.pm | |
parent | f81b2277a8e7e9ce8809ccd30c25b8aa72101215 (diff) | |
download | curl-5c823f51dbbd15193350ae9efa94175abee92894.tar.gz |
runtests: remove the valgrind parser
Old legacy parsing that 1) hid problems for us and 2) probably isn't
needed anymore.
Diffstat (limited to 'tests/valgrind.pm')
-rw-r--r-- | tests/valgrind.pm | 85 |
1 files changed, 2 insertions, 83 deletions
diff --git a/tests/valgrind.pm b/tests/valgrind.pm index 838183b44..e6f66fd50 100644 --- a/tests/valgrind.pm +++ b/tests/valgrind.pm @@ -5,7 +5,7 @@ # | (__| |_| | _ <| |___ # \___|\___/|_| \_\_____| # -# Copyright (C) 1998 - 2010, Daniel Stenberg, <daniel@haxx.se>, et al. +# Copyright (C) 1998 - 2016, Daniel Stenberg, <daniel@haxx.se>, et al. # # This software is licensed as described in the file COPYING, which # you should have received as part of this distribution. The terms @@ -26,90 +26,9 @@ sub valgrindparse { my ($srcdir, # the dir in which the runtests script resides $sslenabled, $file) = @_; - my $leak; - my $invalidread; - my $uninitedvar; - my $error; - my $partial; - my $us; - my @o; - - my $bt=0; - my $nssinit=0; - open(VAL, "<$file"); - while(<VAL>) { - if($bt) { - # back trace parsing - if($_ =~ /^==(\d+)== *(at|by) 0x([0-9A-F]+): (.*)/) { - my $w = $4; - if($w =~ /(.*) \(([^:]*):(\d+)/) { - my ($func, $source, $line)=($1, $2, $3); - my $sourcename = basename($source); - if(-f "$srcdir/../src/$sourcename" || - -f "$srcdir/../lib/$sourcename") { - # this is our source - # print "$func() at $source:$line\n"; - $us++; - } #else {print "Not our source: $func, $source, $line\n";} - } - - # the memory leakage within NSS_InitContext is not a bug of curl - if($w =~ /NSS_InitContext/) { - $nssinit++; - } - } - else { - if($us and not $nssinit) { - # the stack trace included source details about us - - $error++; - if($leak) { - push @o, "\n Leaked $leak bytes\n"; - } - if($invalidread) { - push @o, "\n Read $invalidread invalid bytes\n"; - } - if($uninitedvar) { - push @o, "\n Conditional jump or move depends on uninitialised value(s)\n"; - } - } - $bt = 0; # no more backtrace - $us = 0; - $nssinit = 0; - } - } - else { - if($_ =~ /(\d+) bytes in (\d+) blocks are definitely lost/) { - $leak = $1; - if($leak) { - $error++; - } - $bt = 1; - } - elsif($_ =~ /Invalid read of size (\d+)/) { - $invalidread = $1; - $error++; - $bt = 1; - } - elsif($_ =~ /Conditional jump or move/) { - # If we require SSL, this test case most probaly makes - # us use OpenSSL. OpenSSL produces numerous valgrind - # errors of this kind, rendering it impossible for us to - # detect (valid) reports on actual curl or libcurl code. - - if(!$sslenabled) { - $uninitedvar = 1; - $error++; - $bt = 1; - } - else { - $partial=1; - } - } - } - } + @o = <VAL>; close(VAL); return @o; } |