From b5580c0d4d6b7d73c1b75cbc3bb4502cc0e1abe6 Mon Sep 17 00:00:00 2001
From: Georgios Pinitas <georgios.pinitas@arm.com>
Date: Mon, 24 Dec 2018 16:34:18 +0000
Subject: COMPMID-1710: Fix memory corruption in GEMMReshapeRHSMatrix CL
 validation

Incorrect data-type was passed in the fixture causing out-of-bounds
writes

Change-Id: I046bf5aa012e11a3fab33682191f4c7bed338745
Reviewed-on: https://review.mlplatform.org/444
Tested-by: Arm Jenkins <bsgcomp@arm.com>
Reviewed-by: Anthony Barbier <Anthony.barbier@arm.com>
---
 tests/validation/CL/GEMMReshapeRHSMatrix.cpp | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

(limited to 'tests/validation/CL/GEMMReshapeRHSMatrix.cpp')

diff --git a/tests/validation/CL/GEMMReshapeRHSMatrix.cpp b/tests/validation/CL/GEMMReshapeRHSMatrix.cpp
index f2627ac644..e101cd2899 100644
--- a/tests/validation/CL/GEMMReshapeRHSMatrix.cpp
+++ b/tests/validation/CL/GEMMReshapeRHSMatrix.cpp
@@ -220,7 +220,7 @@ TEST_SUITE(S16)
 FIXTURE_DATA_TEST_CASE(RunSmall0, CLGEMMReshapeRHSMatrixFixture<short>, framework::DatasetMode::PRECOMMIT,
                 combine(combine(combine(combine(combine(combine(combine(datasets::SmallGEMMReshape2DShapes(),
                                                                    b_values),
-                                                                   framework::dataset::make("DataType", DataType::S32)),
+                                                                   framework::dataset::make("DataType", DataType::S16)),
                                                                    n0_values_precommit),
                                                                    k0_nt_values_precommit),
                                                                    h0_values),
@@ -235,7 +235,7 @@ FIXTURE_DATA_TEST_CASE(RunSmall0, CLGEMMReshapeRHSMatrixFixture<short>, framewor
 FIXTURE_DATA_TEST_CASE(RunSmall1, CLGEMMReshapeRHSMatrixFixture<short>, framework::DatasetMode::PRECOMMIT,
                 combine(combine(combine(combine(combine(combine(combine(datasets::SmallGEMMReshape2DShapes(),
                                                                    b_values),
-                                                                   framework::dataset::make("DataType", DataType::S32)),
+                                                                   framework::dataset::make("DataType", DataType::S16)),
                                                                    n0_values_precommit),
                                                                    k0_t_values_precommit),
                                                                    h0_values),
@@ -250,7 +250,7 @@ FIXTURE_DATA_TEST_CASE(RunSmall1, CLGEMMReshapeRHSMatrixFixture<short>, framewor
 FIXTURE_DATA_TEST_CASE(RunLarge0, CLGEMMReshapeRHSMatrixFixture<short>, framework::DatasetMode::NIGHTLY,
                 combine(combine(combine(combine(combine(combine(combine(datasets::LargeGEMMReshape2DShapes(),
                                                                    b_values),
-                                                                   framework::dataset::make("DataType", DataType::S32)),
+                                                                   framework::dataset::make("DataType", DataType::S16)),
                                                                    n0_values_nightly),
                                                                    k0_nt_values_nightly),
                                                                    h0_values),
@@ -265,7 +265,7 @@ FIXTURE_DATA_TEST_CASE(RunLarge0, CLGEMMReshapeRHSMatrixFixture<short>, framewor
 FIXTURE_DATA_TEST_CASE(RunLarge1, CLGEMMReshapeRHSMatrixFixture<short>, framework::DatasetMode::NIGHTLY,
                 combine(combine(combine(combine(combine(combine(combine(datasets::LargeGEMMReshape2DShapes(),
                                                                    b_values),
-                                                                   framework::dataset::make("DataType", DataType::S32)),
+                                                                   framework::dataset::make("DataType", DataType::S16)),
                                                                    n0_values_nightly),
                                                                    k0_t_values_nightly),
                                                                    h0_values),
@@ -282,7 +282,7 @@ TEST_SUITE(S8)
 FIXTURE_DATA_TEST_CASE(RunSmall0, CLGEMMReshapeRHSMatrixFixture<char>, framework::DatasetMode::PRECOMMIT,
                 combine(combine(combine(combine(combine(combine(combine(datasets::SmallGEMMReshape2DShapes(),
                                                                    b_values),
-                                                                   framework::dataset::make("DataType", DataType::S32)),
+                                                                   framework::dataset::make("DataType", DataType::S8)),
                                                                    n0_values_precommit),
                                                                    k0_nt_values_precommit),
                                                                    h0_values),
@@ -297,7 +297,7 @@ FIXTURE_DATA_TEST_CASE(RunSmall0, CLGEMMReshapeRHSMatrixFixture<char>, framework
 FIXTURE_DATA_TEST_CASE(RunSmall1, CLGEMMReshapeRHSMatrixFixture<char>, framework::DatasetMode::PRECOMMIT,
                 combine(combine(combine(combine(combine(combine(combine(datasets::SmallGEMMReshape2DShapes(),
                                                                    b_values),
-                                                                   framework::dataset::make("DataType", DataType::S32)),
+                                                                   framework::dataset::make("DataType", DataType::S8)),
                                                                    n0_values_precommit),
                                                                    k0_t_values_precommit),
                                                                    h0_values),
@@ -312,7 +312,7 @@ FIXTURE_DATA_TEST_CASE(RunSmall1, CLGEMMReshapeRHSMatrixFixture<char>, framework
 FIXTURE_DATA_TEST_CASE(RunLarge0, CLGEMMReshapeRHSMatrixFixture<char>, framework::DatasetMode::NIGHTLY,
                 combine(combine(combine(combine(combine(combine(combine(datasets::LargeGEMMReshape2DShapes(),
                                                                    b_values),
-                                                                   framework::dataset::make("DataType", DataType::S32)),
+                                                                   framework::dataset::make("DataType", DataType::S8)),
                                                                    n0_values_nightly),
                                                                    k0_nt_values_nightly),
                                                                    h0_values),
@@ -327,7 +327,7 @@ FIXTURE_DATA_TEST_CASE(RunLarge0, CLGEMMReshapeRHSMatrixFixture<char>, framework
 FIXTURE_DATA_TEST_CASE(RunLarge1, CLGEMMReshapeRHSMatrixFixture<char>, framework::DatasetMode::NIGHTLY,
                 combine(combine(combine(combine(combine(combine(combine(datasets::LargeGEMMReshape2DShapes(),
                                                                    b_values),
-                                                                   framework::dataset::make("DataType", DataType::S32)),
+                                                                   framework::dataset::make("DataType", DataType::S8)),
                                                                    n0_values_nightly),
                                                                    k0_t_values_nightly),
                                                                    h0_values),
-- 
cgit v1.2.1