fixed checks

Author: iliaal

library.c

@@ -2823,6 +2823,9 @@ static uint8_t crc8(unsigned char *input, size_t len) {
 }
 #endif
 
+#define PHP_REDIS_COMPRESSION_RATIO_CHECK(__x) \
+    (redis_sock->compression_min_ratio > 0 && (((double) __x / (double) len) >= redis_sock->compression_min_ratio))
+
 PHP_REDIS_API int
 redis_pack(RedisSock *redis_sock, zval *z, char **val, size_t *val_len)
 {
@@ -2849,7 +2852,7 @@ redis_pack(RedisSock *redis_sock, zval *z, char **val, size_t *val_len)
                 size = len + MIN(UINT_MAX - len, MAX(LZF_MARGIN, len / 25));
                 data = emalloc(size);
                 if ((res = lzf_compress(buf, len, data, size)) > 0) {
-                    if (((double) res / (double) len) >= redis_sock->compression_min_ratio) {
+                    if (PHP_REDIS_COMPRESSION_RATIO_CHECK(res)) {
                         efree(data);
                         break;
                     }
@@ -2884,7 +2887,7 @@ redis_pack(RedisSock *redis_sock, zval *z, char **val, size_t *val_len)
                 size = ZSTD_compressBound(len);
                 data = emalloc(size);
                 size = ZSTD_compress(data, size, buf, len, level);
-                if (!ZSTD_isError(size) && ((double) size / (double) len) < redis_sock->compression_min_ratio) {
+                if (!ZSTD_isError(size) && !PHP_REDIS_COMPRESSION_RATIO_CHECK(size)) {
                     if (valfree) efree(buf);
                     data = erealloc(data, size);
                     *val = data;