diff --git a/vulnerabilities/snappy-java-integer-overflow-in-compress-leads-to-dos-xray-522075.md b/vulnerabilities/snappy-java-integer-overflow-in-compress-leads-to-dos-xray-522075.md index a8ffe07f28..4bd2c7d5ef 100644 --- a/vulnerabilities/snappy-java-integer-overflow-in-compress-leads-to-dos-xray-522075.md +++ b/vulnerabilities/snappy-java-integer-overflow-in-compress-leads-to-dos-xray-522075.md @@ -32,9 +32,9 @@ The function `compress(char[] input)` in the file `Snappy.java` receives an arra Since the length is not tested, the multiplication by two can cause an integer overflow and become negative. The rawCompress function then uses the received length and passes it to the natively compiled maxCompressedLength function, using the returned value to allocate a byte array. -Since the maxCompressedLength function treats the length as an unsigned integer, it doesn?t care that it is negative, and it returns a valid value, which is casted to a signed integer by the Java engine. If the result is negative, a `java.lang.NegativeArraySizeException` exception will be raised while trying to allocate the array `buf`. On the other side, if the result is positive, the `buf` array will successfully be allocated, but its size might be too small to use for the compression, causing a fatal Access Violation error. +Since the maxCompressedLength function treats the length as an unsigned integer, it doesn't care that it is negative, and it returns a valid value, which is casted to a signed integer by the Java engine. If the result is negative, a `java.lang.NegativeArraySizeException` exception will be raised while trying to allocate the array `buf`. On the other side, if the result is positive, the `buf` array will successfully be allocated, but its size might be too small to use for the compression, causing a fatal Access Violation error. -The same issue exists also when using the `compress` functions that receive double, float, int, long and short, each using a different multiplier that may cause the same issue. The issue most likely won?t occur when using a byte array, since creating a byte array of size 0x80000000 (or any other negative value) is impossible in the first place. +The same issue exists also when using the `compress` functions that receive double, float, int, long and short, each using a different multiplier that may cause the same issue. The issue most likely won't occur when using a byte array, since creating a byte array of size 0x80000000 (or any other negative value) is impossible in the first place. Version 1.1.10.1 contains a patch for this issue. diff --git a/vulnerabilities/snappy-java-unchecked-chunk-length-dos-xray-522074.md b/vulnerabilities/snappy-java-unchecked-chunk-length-dos-xray-522074.md index ae79755cb4..1baac8fc4c 100644 --- a/vulnerabilities/snappy-java-unchecked-chunk-length-dos-xray-522074.md +++ b/vulnerabilities/snappy-java-unchecked-chunk-length-dos-xray-522074.md @@ -28,9 +28,9 @@ org.xerial.snappy:snappy-java snappy-java is a fast compressor/decompressor for Java. Due to use of an unchecked chunk length, an unrecoverable fatal error can occur in versions prior to 1.1.10.1. -The code in the function hasNextChunk in the fileSnappyInputStream.java checks if a given stream has more chunks to read. It does that by attempting to read 4 bytes. If it wasn?t possible to read the 4 bytes, the function returns false. Otherwise, if 4 bytes were available, the code treats them as the length of the next chunk. +The code in the function hasNextChunk in the fileSnappyInputStream.java checks if a given stream has more chunks to read. It does that by attempting to read 4 bytes. If it wasn't possible to read the 4 bytes, the function returns false. Otherwise, if 4 bytes were available, the code treats them as the length of the next chunk. -In the case that the `compressed` variable is null, a byte array is allocated with the size given by the input data. Since the code doesn?t test the legality of the `chunkSize` variable, it is possible to pass a negative number (such as 0xFFFFFFFF which is -1), which will cause the code to raise a `java.lang.NegativeArraySizeException` exception. A worse case would happen when passing a huge positive value (such as 0x7FFFFFFF), which would raise the fatal `java.lang.OutOfMemoryError` error. +In the case that the `compressed` variable is null, a byte array is allocated with the size given by the input data. Since the code doesn't test the legality of the `chunkSize` variable, it is possible to pass a negative number (such as 0xFFFFFFFF which is -1), which will cause the code to raise a `java.lang.NegativeArraySizeException` exception. A worse case would happen when passing a huge positive value (such as 0x7FFFFFFF), which would raise the fatal `java.lang.OutOfMemoryError` error. Version 1.1.10.1 contains a patch for this issue.