commit d6253602ec7ebcbe6adb0acc51a97a5d8c167495
author Moritz Pflanzer <moritz.pflanzer@arm.com> Mon Aug 07 16:49:49 2017 +0100
committer Anthony Barbier <anthony.barbier@arm.com> Mon Sep 17 14:16:42 2018 +0100
tree 7031598ee38ca2fe0e61b499c0e5371e17166311
parent d9769583c44d8ad79e7fcf662715b2460e96084b

COMPMID-417: Fix invalid read in GEMM

Change-Id: Ia817dc4415197c310e7e9f9717ce9679294ddfcf
Reviewed-on: http://mpd-gerrit.cambridge.arm.com/83109
Reviewed-by: Georgios Pinitas <georgios.pinitas@arm.com>
Reviewed-by: Anthony Barbier <anthony.barbier@arm.com>
Reviewed-by: Gian Marco Iodice <gianmarco.iodice@arm.com>
Tested-by: Kaizen <jeremy.johnson+kaizengerrit@arm.com>

src/core/NEON/kernels/NEGEMMMatrixAccumulateBiasesKernel.cpp

diff --git a/src/core/NEON/kernels/NEGEMMMatrixAccumulateBiasesKernel.cpp b/src/core/NEON/kernels/NEGEMMMatrixAccumulateBiasesKernel.cpp
index f3d06ed..a4fc494 100644
--- a/src/core/NEON/kernels/NEGEMMMatrixAccumulateBiasesKernel.cpp
+++ b/src/core/NEON/kernels/NEGEMMMatrixAccumulateBiasesKernel.cpp
@@ -58,11 +58,9 @@
     // Configure kernel window
     Window win = calculate_max_window(*accum->info(), Steps(num_elems_processed_per_iteration));
 
-    AccessWindowStatic biases_access(biases->info(), 0, 0, biases->info()->dimension(0), biases->info()->dimension(1));
-
     update_window_and_padding(win,
                               AccessWindowHorizontal(accum->info(), 0, num_elems_processed_per_iteration),
-                              biases_access);
+                              AccessWindowStatic(biases->info(), 0, 0, win.x().end(), biases->info()->tensor_shape().y()));
 
     AccessWindowHorizontal output_access(accum->info(), 0, num_elems_processed_per_iteration);