bugfix: double values larger than int32_t were incorrectly printed out as 64-bit integers.

Author: zhuizhuhaomeng

src/ngx_http_lua_output.h

@@ -30,8 +30,8 @@ ngx_http_lua_get_num_len(lua_State *L, int idx)
     double     num;
 
     num = (double) lua_tonumber(L, idx);
-    if (num == (double) (long) num) {
-        return NGX_INT64_LEN;
+    if (num == (double) (int32_t) num) {
+        return NGX_INT32_LEN;
     }
 
     return NGX_DOUBLE_LEN;
@@ -45,11 +45,16 @@ ngx_http_lua_write_num(lua_State *L, int idx, u_char *dst)
     int        n;
 
     num = (double) lua_tonumber(L, idx);
-    if (num == (double) (long) num) {
-        dst = ngx_snprintf(dst, NGX_INT64_LEN, "%l", (long) num);
+    /*
+     * luajit format number with only 14 significant digits.
+     * To be consistent with lujit, don't use (double) (long) below
+     * or integer greater than 99,999,999,999,999 will different from luajit.
+     */
+    if (num == (double) (int32_t) num) {
+        dst = ngx_snprintf(dst, NGX_INT64_LEN, "%D", (int32_t) num);
 
     } else {
-        /**
+        /*
          * The maximum number of significant digits is 14 in lua.
          * Please refer to lj_strfmt.c for more details.
          */

t/164-say.t

@@ -53,3 +53,59 @@ GET /lua
 GET /lua
 --- response_body
 123 3.1415926
+
+
+
+=== TEST 4: ngx.say min int32 -2147483648
+--- config
+    location /lua {
+        content_by_lua_block {
+            ngx.say(-2147483648)
+        }
+    }
+--- request
+GET /lua
+--- response_body
+-2147483648
+
+
+
+=== TEST 5: ngx.say big integer 2147483647
+--- config
+    location /lua {
+        content_by_lua_block {
+            ngx.say(2147483647)
+        }
+    }
+--- request
+GET /lua
+--- response_body
+2147483647
+
+
+
+=== TEST 6: ngx.say big integer -9223372036854775808
+--- config
+    location /lua {
+        content_by_lua_block {
+            ngx.say(-9223372036854775808)
+        }
+    }
+--- request
+GET /lua
+--- response_body
+-9.2233720368548e+18
+
+
+
+=== TEST 7: ngx.say big integer 18446744073709551615
+--- config
+    location /lua {
+        content_by_lua_block {
+            ngx.say(18446744073709551615)
+        }
+    }
+--- request
+GET /lua
+--- response_body
+1.844674407371e+19