Fixed MONITOR mode and Issue 296

diff --git a/src/db.c b/src/db.c
index 6d287d72c914762eef4ec17eb08a2f324df71e58..f380bf6ec05882c6e79ec3993608e68d8e96c168 100644 (file)
--- a/src/db.c
+++ b/src/db.c
@@ -221,19 +221,19 @@ void keysCommand(redisClient *c) {
     dictIterator *di;
     dictEntry *de;
     sds pattern = c->argv[1]->ptr;
     dictIterator *di;
     dictEntry *de;
     sds pattern = c->argv[1]->ptr;

-    int plen = sdslen(pattern);
+    int plen = sdslen(pattern), allkeys;

     unsigned long numkeys = 0;
     robj *lenobj = createObject(REDIS_STRING,NULL);
 
     di = dictGetIterator(c->db->dict);
     addReply(c,lenobj);
     decrRefCount(lenobj);
     unsigned long numkeys = 0;
     robj *lenobj = createObject(REDIS_STRING,NULL);
 
     di = dictGetIterator(c->db->dict);
     addReply(c,lenobj);
     decrRefCount(lenobj);

+    allkeys = (pattern[0] == '*' && pattern[1] == '\0');

     while((de = dictNext(di)) != NULL) {
         sds key = dictGetEntryKey(de);
         robj *keyobj;
 
     while((de = dictNext(di)) != NULL) {
         sds key = dictGetEntryKey(de);
         robj *keyobj;
 

-        if ((pattern[0] == '*' && pattern[1] == '\0') ||
-            stringmatchlen(pattern,plen,key,sdslen(key),0)) {
+        if (allkeys || stringmatchlen(pattern,plen,key,sdslen(key),0)) {

             keyobj = createStringObject(key,sdslen(key));
             if (expireIfNeeded(c->db,keyobj) == 0) {
                 addReplyBulk(c,keyobj);
             keyobj = createStringObject(key,sdslen(key));
             if (expireIfNeeded(c->db,keyobj) == 0) {
                 addReplyBulk(c,keyobj);

diff --git a/src/redis-cli.c b/src/redis-cli.c
index 8b7d0777dc5f7b4f4c7f8a9819d3d53f632dbf29..3a6b0a9024ee583ea57b087d524095c16286dc5f 100644 (file)
--- a/src/redis-cli.c
+++ b/src/redis-cli.c
@@ -96,7 +96,7 @@ static sds cliReadLine(int fd) {
         ssize_t ret;
 
         ret = read(fd,&c,1);
         ssize_t ret;
 
         ret = read(fd,&c,1);

-        if (ret == -1) {
+        if (ret <= 0) {

             sdsfree(line);
             return NULL;
         } else if ((ret == 0) || (c == '\n')) {
             sdsfree(line);
             return NULL;
         } else if ((ret == 0) || (c == '\n')) {


@@ -282,7 +282,8 @@ static int cliSendCommand(int argc, char **argv, int repeat) {
     while(repeat--) {
         anetWrite(fd,cmd,sdslen(cmd));
         while (config.monitor_mode) {
     while(repeat--) {
         anetWrite(fd,cmd,sdslen(cmd));
         while (config.monitor_mode) {

-            cliReadSingleLineReply(fd,0);
+            if (cliReadSingleLineReply(fd,0)) exit(1);
+            printf("\n");

         }
 
         if (config.pubsub_mode) {
         }
 
         if (config.pubsub_mode) {