bugfix: should reset the value_len to 0 when reuse the expired list type key in shared dict. (#263)

Author: ngtee8

src/ngx_stream_lua_shdict.c

@@ -734,7 +734,7 @@ ngx_stream_lua_shdict_push_helper(lua_State *L, int flags)
                        "type matched, reusing it");
 
         sd->expires = 0;
-
+        sd->value_len = 0;
         /* free list nodes */
 
         queue = ngx_stream_lua_shdict_get_list_head(sd, key.len);

t/043-shdict.t

@@ -1962,3 +1962,77 @@ nil
 nil
 --- no_error_log
 [error]
+
+
+
+=== TEST 88: push to an expired list
+--- stream_config
+    lua_shared_dict dogs 1m;
+--- stream_server_config
+    content_by_lua_block {
+        local dogs = ngx.shared.dogs
+        local len, err = dogs:lpush("cc", "1") --add another list to avoid key"aa" be cleaned (run ‘ngx_http_lua_shdict_expire(ctx, 1)’ may clean key ,ensure key'aa' not clean ,just expired))
+        if not len then
+            ngx.say("push cc  err: ", err)
+        end
+        local len, err = dogs:lpush("aa", "1")
+        if not len then
+            ngx.say("push1 err: ", err)
+        end
+        local succ, err = dogs:expire("aa", 0.2)
+        if not succ then
+            ngx.say("expire err: ",err)
+        end
+        ngx.sleep(0.3) -- list aa expired
+        local len, err = dogs:lpush("aa", "2") --push to an expired list may set as a new list
+        if not len then
+            ngx.say("push2 err: ", err)
+        end
+        local len, err = dogs:llen("aa") -- new list len is 1
+        if not len then
+            ngx.say("llen err: ", err)
+        else
+            ngx.say("aa:len :", dogs:llen("aa"))
+        end
+    }
+--- stream_response
+aa:len :1
+--- no_error_log
+[error]
+
+
+
+=== TEST 89: push to an expired list then pop many time (more then list len )
+--- stream_config
+    lua_shared_dict dogs 1m;
+--- stream_server_config
+    content_by_lua_block {
+        local dogs = ngx.shared.dogs
+        local len, err = dogs:lpush("cc", "1") --add another list to avoid key"aa" be cleaned (run ‘ngx_http_lua_shdict_expire(ctx, 1)’ may clean key ,ensure key'aa' not clean ,just expired))
+        if not len then
+            ngx.say("push cc  err: ", err)
+        end
+        local len, err = dogs:lpush("aa", "1")
+        if not len then
+            ngx.say("push1 err: ", err)
+        end
+        local succ, err = dogs:expire("aa", 0.2)
+        if not succ then
+            ngx.say("expire err: ",err)
+        end
+        ngx.sleep(0.3) -- list aa expired
+        local len, err = dogs:lpush("aa", "2") --push to an expired list may set as a new list
+        if not len then
+            ngx.say("push2 err: ", err)
+        end
+        local val, err = dogs:lpop("aa")
+        if not val then
+            ngx.say("llen err: ", err)
+        end
+        local val, err = dogs:lpop("aa")  -- val == nil
+        ngx.say("aa list value: ", val)
+    }
+--- stream_response
+aa list value: nil
+--- no_error_log
+[error]