caleb-brown/redict
1
0
Fork 0
mirror of https://codeberg.org/redict/redict.git synced 2025-01-23 00:28:26 -05:00
Code Issues Actions Packages Projects Releases Wiki Activity

Fix two minor bugs (MIGRATE key args and getKeysUsingCommandTable) (#9455)

Browse Source 
1. MIGRATE has a potnetial key arg in argv[3]. It should be reflected in the command table.
2. getKeysUsingCommandTable should never free getKeysResult, it is always freed by the caller)
   The reason we never encountered this double-free bug is that almost always getKeysResult
   uses the statis buffer and doesn't allocate a new one.
This commit is contained in:
guybe7 2021-09-02 16:19:27 +02:00 committed by GitHub
parent 306a5ccd2d
commit 6aa2285e32
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 1 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
src/db.c
View File

@ -1614,7 +1614,6 @@ int getKeysUsingCommandTable(struct redisCommand *cmd,robj **argv, int argc, get
* return no keys and expect the command implementation to report * return no keys and expect the command implementation to report
* an arity or syntax error. */ * an arity or syntax error. */
if (cmd->flags & CMD_MODULE || cmd->arity < 0) { if (cmd->flags & CMD_MODULE || cmd->arity < 0) {
getKeysFreeResult(result);
result->numkeys = 0; result->numkeys = 0;
return 0; return 0;
} else { } else {

2
src/server.c
View File

@ -891,7 +891,7 @@ struct redisCommand redisCommandTable[] = {
{"migrate",migrateCommand,-6, {"migrate",migrateCommand,-6,
"write random @keyspace @dangerous", "write random @keyspace @dangerous",
0,migrateGetKeys,0,0,0,0,0,0}, 0,migrateGetKeys,3,3,1,0,0,0},
{"asking",askingCommand,1, {"asking",askingCommand,1,
"fast @connection", "fast @connection",