diff options
| author | Stuart Haslam <stuart.haslam@linaro.org> | 2015-03-05 12:10:03 +0000 |
|---|---|---|
| committer | Maxim Uvarov <maxim.uvarov@linaro.org> | 2015-03-06 13:43:10 +0300 |
| commit | 301e45bb1faef9b0a306bc5c23f92371b1d3add9 (patch) | |
| tree | dcac95339feda4fdaff19e707515728074f0a895 /test | |
| parent | a7bda31a17a7eae12b15776d9b6220a658a1b4a3 (diff) | |
validation: common: prevent masking test failures
The addition of tests_global_term() inadvertently masked test failures
by overwritting the value of ret.
Signed-off-by: Stuart Haslam <stuart.haslam@linaro.org>
Reviewed-by: Bill Fischofer <bill.fischofer@linaro.org>
Signed-off-by: Maxim Uvarov <maxim.uvarov@linaro.org>
Diffstat (limited to 'test')
| -rw-r--r-- | test/validation/common/odp_cunit_common.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/test/validation/common/odp_cunit_common.c b/test/validation/common/odp_cunit_common.c index 655d1578c..83e502618 100644 --- a/test/validation/common/odp_cunit_common.c +++ b/test/validation/common/odp_cunit_common.c @@ -80,9 +80,8 @@ int main(void) CU_cleanup_registry(); - ret = tests_global_term(); - if (ret) - return ret; + if (0 != tests_global_term()) + return -1; odp_term_local(); odp_term_global(); |