Use redisFreeCommand instead of free() function (#2351)

an-tao · web-flow · commit 1a92fa77ee50 · 2025-07-15T16:38:37.000+08:00
diff --git a/nosql_lib/redis/src/RedisConnection.h b/nosql_lib/redis/src/RedisConnection.h
@@ -74,7 +74,7 @@ class RedisConnection : public trantor::NonCopyable,
     static std::string getFormattedCommand(const std::string_view &command,
                                            va_list ap) noexcept(false)
     {
-        char *cmd;
+        char *cmd{nullptr};
         auto len = redisvFormatCommand(&cmd, command.data(), ap);
         if (len == -1)
         {
@@ -92,7 +92,7 @@ class RedisConnection : public trantor::NonCopyable,
                                  "Unknown format error");
         }
         std::string fullCommand{cmd, static_cast<size_t>(len)};
-        free(cmd);
+        redisFreeCommand(cmd);
         return fullCommand;
     }
 

Original file line number	Diff line number	Diff line change
`@@ -74,7 +74,7 @@ class RedisConnection : public trantor::NonCopyable,`
`74`	`74`	`static std::string getFormattedCommand(const std::string_view &command,`
`75`	`75`	`va_list ap) noexcept(false)`
`76`	`76`	`{`
`77`		`- char *cmd;`
	`77`	`+ char *cmd{nullptr};`
`78`	`78`	`auto len = redisvFormatCommand(&cmd, command.data(), ap);`
`79`	`79`	`if (len == -1)`
`80`	`80`	`{`
`@@ -92,7 +92,7 @@ class RedisConnection : public trantor::NonCopyable,`
`92`	`92`	`"Unknown format error");`
`93`	`93`	`}`
`94`	`94`	`std::string fullCommand{cmd, static_cast<size_t>(len)};`
`95`		`- free(cmd);`
	`95`	`+ redisFreeCommand(cmd);`
`96`	`96`	`return fullCommand;`
`97`	`97`	`}`
`98`	`98`