Minor refactor of ignroe numbers option

michael-grunder · michael-grunder · commit 29e5cf0d8c03 · 2025-02-05T14:12:42.000-08:00
* We want to run the logic if either a serializer OR a compression
  option is set.
* IEE754 doubles can theoretically have a huge number of characters.
diff --git a/library.c b/library.c
@@ -3882,10 +3882,10 @@ redis_unpack(RedisSock *redis_sock, const char *src, int srclen, zval *zdst) {
     size_t len;
     char *buf;
 
-    if (UNEXPECTED((redis_sock->serializer != REDIS_SERIALIZER_NONE &&
+    if (UNEXPECTED((redis_sock->serializer != REDIS_SERIALIZER_NONE ||
                     redis_sock->compression != REDIS_COMPRESSION_NONE) &&
                     redis_sock->pack_ignore_numbers) &&
-                    srclen > 0 && srclen < 24)
+                    srclen > 0 && srclen < 512)
     {
         switch (is_numeric_string(src, srclen, &lval, &dval, 0)) {
             case IS_LONG:
