From: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>
Date: Mon, 23 Mar 2015 15:19:49 +0000 (+0100)
Subject: validate: raise an error if valgrind detected issues
X-Git-Tag: 1.19.3~491^2~1322
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=f1985bc73892968af672b61d1131f1c6de53dd0b;p=platform%2Fupstream%2Fgstreamer.git

validate: raise an error if valgrind detected issues

Differential Revision: http://phabricator.freedesktop.org/D53
---

diff --git a/validate/launcher/baseclasses.py b/validate/launcher/baseclasses.py
index 443ad75..73171d3 100644
--- a/validate/launcher/baseclasses.py
+++ b/validate/launcher/baseclasses.py
@@ -42,6 +42,8 @@ from utils import mkdir, Result, Colors, printc, DEFAULT_TIMEOUT, GST_SECOND, \
 # The factor by which we increase the hard timeout when running inside
 # Valgrind
 VALGRIND_TIMEOUT_FACTOR = 5
+# The error reported by valgrind when detecting errors
+VALGRIND_ERROR_CODE = 20
 
 
 class Test(Loggable):
@@ -289,6 +291,7 @@ class Test(Loggable):
             ('leak-resolution', 'high'),
             ('num-callers', '20'),
             ('log-file', vglogsfile),
+            ('error-exitcode', str(VALGRIND_ERROR_CODE)),
         ]
 
         supps = self.get_valgrind_suppressions()
@@ -475,6 +478,8 @@ class GstValidateTest(Test):
             self.set_result(Result.FAILED,
                             "Application segfaulted",
                             "segfault")
+        elif self.process.returncode == VALGRIND_ERROR_CODE:
+            self.set_result(Result.FAILED, "Valgrind reported errors")
         elif criticals or self.process.returncode != 0:
             if criticals is None:
                 criticals = "No criticals"