From f1985bc73892968af672b61d1131f1c6de53dd0b Mon Sep 17 00:00:00 2001
From: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>
Date: Mon, 23 Mar 2015 16:19:49 +0100
Subject: [PATCH] validate: raise an error if valgrind detected issues

Differential Revision: http://phabricator.freedesktop.org/D53
---
 validate/launcher/baseclasses.py | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/validate/launcher/baseclasses.py b/validate/launcher/baseclasses.py
index 443ad75449..73171d3d50 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"