You need to sign in or sign up before continuing.
Commit f797c7dc authored by antirez's avatar antirez
Browse files

bad data on RESTORE can no longer crash the server but create a memory leak with some input string

parent f304f22b
...@@ -1140,8 +1140,10 @@ void restoreCommand(redisClient *c) { ...@@ -1140,8 +1140,10 @@ void restoreCommand(redisClient *c) {
/* Finally create the object from the serialized dump and /* Finally create the object from the serialized dump and
* store it at the specified key. */ * store it at the specified key. */
o = rdbLoadObject(data[0],fp); if ((data[0] > 4 && data[0] < 9) ||
if (o == NULL) { data[0] > 11 ||
(o = rdbLoadObject(data[0],fp)) == NULL)
{
addReplyError(c,"Bad data format."); addReplyError(c,"Bad data format.");
fclose(fp); fclose(fp);
return; return;
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment