argv mem leak during multi command execution. (#9598)

Changes in #9528 lead to memory leak if the command implementation
used rewriteClientCommandArgument inside MULTI-EXEC.

Adding an explicit test for that case since the test that uncovered it
didn't specifically target this scenario

1 files changed, 17 insertions, 0 deletions

diff --git a/tests/unit/protocol.tcl b/tests/unit/protocol.tcl
index 4f185d9bd..ec08e2d36 100644
--- a/tests/unit/protocol.tcl
+++ b/tests/unit/protocol.tcl
@@ -203,6 +203,23 @@ start_server {tags {"protocol network"}} {
         r mset {*}$args
         assert_equal [r get "{k}2"] v2
     }
+    
+    test "test argument rewriting - issue 9598" {
+        # INCRBYFLOAT uses argument rewriting for correct float value propagation.
+        # We use it to make sure argument rewriting works properly. It's important 
+        # this test is run under valgrind to verify there are no memory leaks in 
+        # arg buffer handling.
+        r flushdb
+
+        # Test normal argument handling
+        r set k 0
+        assert_equal [r incrbyfloat k 1.0] 1
+        
+        # Test argument handing in multi-state buffers
+        r multi
+        r incrbyfloat k 1.0
+        assert_equal [r exec] 2
+    }
 
 }