igo_nr: compute circular buffer ends in samples not bytes

The processing buffers' end pointers were computed by adding a byte
count to sample-typed pointers, placing the end past the real buffer and
letting the wrap check pass too late. Convert the byte sizes to element
counts when computing the ends.

Signed-off-by: Liam Girdwood <liam.r.girdwood@linux.intel.com>

Author: lrgirdwo

src/audio/igo_nr/igo_nr.c

@@ -105,8 +105,12 @@ static int igo_nr_capture_s16(struct comp_data *cd,
 	}
 
 	y2 = y1;
-	x_end = x_start + x_size;
-	y_end = y_start + y_size;
+	/* x_size/y_size are byte counts but the pointers are sample-typed, so
+	 * convert to element counts when computing the buffer ends; otherwise
+	 * the wrap check below allows reading/writing past the buffer
+	 */
+	x_end = x_start + x_size / sizeof(*x_start);
+	y_end = y_start + y_size / sizeof(*y_start);
 
 	/* Deinterleave the source buffer and keeps the active channel data as input. */
 	i = 0;
@@ -207,8 +211,12 @@ static int igo_nr_capture_s24(struct comp_data *cd,
 	}
 
 	y2 = y1;
-	x_end = x_start + x_size;
-	y_end = y_start + y_size;
+	/* x_size/y_size are byte counts but the pointers are sample-typed, so
+	 * convert to element counts when computing the buffer ends; otherwise
+	 * the wrap check below allows reading/writing past the buffer
+	 */
+	x_end = x_start + x_size / sizeof(*x_start);
+	y_end = y_start + y_size / sizeof(*y_start);
 
 	/* Deinterleave the source buffer and keeps the active channel data as input. */
 	i = 0;
@@ -308,8 +316,12 @@ static int igo_nr_capture_s32(struct comp_data *cd,
 	}
 
 	y2 = y1;
-	x_end = x_start + x_size;
-	y_end = y_start + y_size;
+	/* x_size/y_size are byte counts but the pointers are sample-typed, so
+	 * convert to element counts when computing the buffer ends; otherwise
+	 * the wrap check below allows reading/writing past the buffer
+	 */
+	x_end = x_start + x_size / sizeof(*x_start);
+	y_end = y_start + y_size / sizeof(*y_start);
 
 	/* Deinterleave the source buffer and keeps the active channel data as input. */
 	i = 0;