diff --git a/tests/runtests.pl b/tests/runtests.pl index 7af18d6a5..35b4ddd7f 100755 --- a/tests/runtests.pl +++ b/tests/runtests.pl @@ -6,7 +6,7 @@ # | (__| |_| | _ <| |___ # \___|\___/|_| \_\_____| # -# Copyright (C) 1998 - 2004, Daniel Stenberg, , et al. +# Copyright (C) 1998 - 2005, Daniel Stenberg, , et al. # # This software is licensed as described in the file COPYING, which # you should have received as part of this distribution. The terms @@ -29,6 +29,7 @@ use strict; @INC=(@INC, $ENV{'srcdir'}, "."); require "getpart.pm"; # array functions +require "valgrind.pm"; # valgrind report parser my $srcdir = $ENV{'srcdir'} || '.'; my $HOSTIP="127.0.0.1"; @@ -94,6 +95,15 @@ if($valgrind) { if (($? >> 8)==0) { $valgrind_tool="--tool=memcheck "; } + open(C, "<$CURL"); + my $l = ; + if($l =~ /^\#\!/) { + # The first line starts with "#!" which implies a shell-script. + # This means libcurl is built shared and curl is a wrapper-script + # Disable valgrind in this setup + $valgrind=0; + } + close(C); } my $gdb = checkcmd("gdb"); @@ -1434,60 +1444,20 @@ sub singletest { if($f =~ /^valgrind$testnum\.pid/) { $l = $f; last; - } - } - my $leak; - my $invalidread; - my $uninitedvar; - my $error; - my $partial; - - open(VAL, ") { - if($_ =~ /definitely lost: (\d*) bytes/) { - $leak = $1; - if($leak) { - $error++; - } - last; - } - elsif($_ =~ /Invalid read of size (\d+)/) { - $invalidread = $1; - $error++; - last; - } - 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(!$feature{'SSL'}) { - $uninitedvar = 1; - $error++; - last; - } - else { - $partial=1; - } } } - close(VAL); - if($error) { + my $src=$ENV{'srcdir'}; + if(!$src) { + $src="."; + } + my @e = valgrindparse($src, $feature{'SSL'}, "log/$l"); + if($e[0]) { print " valgrind ERROR "; - if($leak) { - print "\n Leaked $leak bytes\n"; - } - if($invalidread) { - print "\n Read $invalidread invalid bytes\n"; - } - if($uninitedvar) { - print "\n Conditional jump or move depends on uninitialised value(s)\n"; - } + print @e; return 1; } elsif(!$short) { - printf " valgrind %s", $partial?"PARTIAL":"OK"; + printf " valgrind OK"; } } else { diff --git a/tests/valgrind.pm b/tests/valgrind.pm new file mode 100644 index 000000000..e2ead03c8 --- /dev/null +++ b/tests/valgrind.pm @@ -0,0 +1,109 @@ +#*************************************************************************** +# _ _ ____ _ +# Project ___| | | | _ \| | +# / __| | | | |_) | | +# | (__| |_| | _ <| |___ +# \___|\___/|_| \_\_____| +# +# Copyright (C) 1998 - 2005, Daniel Stenberg, , et al. +# +# This software is licensed as described in the file COPYING, which +# you should have received as part of this distribution. The terms +# are also available at http://curl.haxx.se/docs/copyright.html. +# +# You may opt to use, copy, modify, merge, publish, distribute and/or sell +# copies of the Software, and permit persons to whom the Software is +# furnished to do so, under the terms of the COPYING file. +# +# This software is distributed on an "AS IS" basis, WITHOUT WARRANTY OF ANY +# KIND, either express or implied. +# +# $Id$ +########################################################################### + +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; + + open(VAL, "<$file"); + while() { + 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); + + if(-f "$srcdir/../src/$source" || + -f "$srcdir/../lib/$source") { + # this is our source + # print "$func() at $source:$line\n"; + $us++; + } + } + } + else { + if($us) { + # 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; + } + } + 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; + } + } + } + } + close(VAL); + return @o; +} + +1;