Fix bound check

flanglet · flanglet · commit bb1ced4d6c25 · 2021-12-11T21:23:59.000-08:00
diff --git a/src/transform/LZCodec.cpp b/src/transform/LZCodec.cpp
@@ -121,7 +121,7 @@ bool LZXCodec<T>::forward(SliceArray<byte>& input, SliceArray<byte>& output, int
         int bestLen = 0;
 
         if (memcmp(&src[srcIdx], &src[ref], 4) == 0) {
-            bestLen = 4 + findMatch(src, srcIdx + 4, ref + 4, min(srcEnd - srcIdx, MAX_MATCH));
+            bestLen = 4 + findMatch(src, srcIdx + 4, ref + 4, min(srcEnd - srcIdx - 4, MAX_MATCH));
         }
 
         // No good match ?

Original file line number	Diff line number	Diff line change
`@@ -121,7 +121,7 @@ bool LZXCodec<T>::forward(SliceArray<byte>& input, SliceArray<byte>& output, int`
`121`	`121`	`int bestLen = 0;`
`122`	`122`
`123`	`123`	`if (memcmp(&src[srcIdx], &src[ref], 4) == 0) {`
`124`		`- bestLen = 4 + findMatch(src, srcIdx + 4, ref + 4, min(srcEnd - srcIdx, MAX_MATCH));`
	`124`	`+ bestLen = 4 + findMatch(src, srcIdx + 4, ref + 4, min(srcEnd - srcIdx - 4, MAX_MATCH));`
`125`	`125`	`}`
`126`	`126`
`127`	`127`	`// No good match ?`