Created valgrind suppression file for PHP >= 7.2

Author: sergeyklay

.travis.yml

@@ -58,26 +58,7 @@ install:
     - bash ./install-development --phpize $(phpenv which phpize) --php-config $(phpenv which php-config)
 
 script:
-    - |
-        valgrind \
-            --read-var-info=yes \
-            --error-exitcode=1 \
-            --fullpath-after= \
-            --track-origins=yes \
-            --leak-check=full \
-            --num-callers=20 \
-            --run-libc-freeres=no \
-            --suppressions=$(pwd)/tests/php-leaks.supp \
-            $(phpenv which php) \
-                -d variables_order=EGPCS \
-                run-tests.php \
-                -p $(which php) \
-                -d extension=$(pwd)/modules/zephir_parser.so \
-                -d variables_order=EGPCS \
-                -g "FAIL,XFAIL,BORK,WARN,SKIP" \
-                --offline \
-                --show-diff \
-                --set-timeout 120
+    - bash ${TRAVIS_BUILD_DIR}/tests/ci/run-tests.sh
 
 addons:
     apt:

tests/.gitignore

@@ -1,4 +1,5 @@
 *.sh
+!ci/run-tests.sh
 *.out
 *.exp
 *.diff

tests/ci/install-travis

@@ -2,7 +2,7 @@
 #
 # Zephir Parser
 #
-# Copyright (c) 2013-2017 Zephir Team and contributors
+# Copyright (c) 2013-present Zephir Team and contributors
 #
 # This source file is subject the MIT license, that is bundled with
 # this package in the file LICENSE, and is available through the
@@ -15,12 +15,6 @@
 
 set -e
 
-# Ensure that this is being run inside a CI container
-#if [ "${CI}" != "true" ]; then
-    #echo "This script is designed to run inside a CI container only. Exiting"
-    #exit 1
-#fi
-
 CURRENT_DIR=$( cd "$( dirname "${BASH_SOURCE[0]}" )" && pwd )
 
 CC=${CC:-gcc}

tests/ci/run-tests.sh

@@ -0,0 +1,45 @@
+#!/usr/bin/env bash
+#
+# Zephir Parser
+#
+# Copyright (c) 2013-present Zephir Team and contributors
+#
+# This source file is subject the MIT license, that is bundled with
+# this package in the file LICENSE, and is available through the
+# world-wide-web at the following url:
+# http://zephir-lang.com/license.html
+#
+# If you did not receive a copy of the MIT license and are unable
+# to obtain it through the world-wide-web, please send a note to
+# [email protected] so we can mail you a copy immediately.
+
+CURRENT_DIR=$( cd "$( dirname "${BASH_SOURCE[0]}" )" && pwd )
+
+PHP_MAJOR="$(`phpenv which php` -r 'echo phpversion();' | cut -d '.' -f 1)"
+PHP_MINOR="$(`phpenv which php` -r 'echo phpversion();' | cut -d '.' -f 2)"
+
+if [ "${PHP_MAJOR}.${PHP_MINOR}" = "7.2" ] || [ "${PHP_MAJOR}.${PHP_MINOR}" = "7.3" ]; then
+	export PHP_LEAKS_SUPPRESSION_FILE=${CURRENT_DIR}/../php-travis-leaks.supp
+else
+	export PHP_LEAKS_SUPPRESSION_FILE=${CURRENT_DIR}/../php-leaks.supp
+fi
+
+valgrind \
+	--read-var-info=yes \
+	--error-exitcode=1 \
+	--fullpath-after= \
+	--track-origins=yes \
+	--leak-check=full \
+	--num-callers=20 \
+	--run-libc-freeres=no \
+	--suppressions=${PHP_LEAKS_SUPPRESSION_FILE} \
+	$(phpenv which php) \
+		-d variables_order=EGPCS \
+		run-tests.php \
+		-p $(phpenv which php) \
+		-d extension=${CURRENT_DIR}/../../modules/zephir_parser.so \
+		-d variables_order=EGPCS \
+		-g "FAIL,XFAIL,BORK,WARN,SKIP" \
+		--offline \
+		--show-diff \
+		--set-timeout 120

tests/php-ng-leaks.supp

@@ -0,0 +1,46 @@
+{
+   <insert_a_suppression_name_here>
+   Memcheck:Cond
+   obj:*
+   obj:*
+   obj:*
+   obj:*
+   obj:*
+   obj:*
+}
+{
+   <insert_a_suppression_name_here>
+   Memcheck:Leak
+   match-leak-kinds: definite
+   fun:malloc
+   fun:__zend_malloc
+   fun:zend_string_alloc
+   fun:zend_string_init
+   fun:cli_register_file_handles
+   fun:do_cli
+   fun:main
+}
+{
+   <insert_a_suppression_name_here>
+   Memcheck:Leak
+   match-leak-kinds: definite
+   fun:malloc
+   fun:__zend_malloc
+   fun:zend_string_alloc
+   fun:zend_string_init
+   fun:cli_register_file_handles
+   fun:do_cli
+   fun:main
+}
+{
+   <insert_a_suppression_name_here>
+   Memcheck:Leak
+   match-leak-kinds: definite
+   fun:malloc
+   fun:__zend_malloc
+   fun:zend_string_alloc
+   fun:zend_string_init
+   fun:cli_register_file_handles
+   fun:do_cli
+   fun:main
+}