Fixes:

- memory leaks in connection.c::RedisConnection_sendCommand()
  and resp.c::RedisObject_RESP_array()
- illegal read in object.c::RedisObject_parseSimpleString()
- incorrect assert in object.c::RedisObject_parseBulkString()

src/connection.c

@@ -25,6 +25,7 @@ RedisObject_t RedisConnection_sendCommand(RedisConnection_t conn, RedisObject_t
             rObj = RedisConnection_getNextObject(conn);
     }
 
+    free(cmdRESP);
     return rObj;
 }
 
@@ -68,4 +69,4 @@ RedisObject_t RedisConnection_getNextObject(RedisConnection_t conn)
     }
 
     return rObj;
-}
+}

src/object.c

@@ -31,7 +31,7 @@ RedisObject_t RedisObject_parseSimpleString(RedisConnection_t conn)
         // is this needed? consumes the newline, but doesn't getNextObject do that already?
         if (readBuf[readOffset] != '\n')
             ++readOffset;
-    } while (readBuf[readOffset - 1] != '\r');
+    } while (readOffset == 0 || readBuf[readOffset - 1] != '\r');
 
     readBuf[readOffset - 1] = '\0';
     rObj.obj = realloc(readBuf, readOffset);
@@ -56,7 +56,6 @@ RedisObject_t RedisObject_parseBulkString(RedisConnection_t conn)
     if (len != -1)
     {
         RedisObject_t strObj = RedisObject_parseSimpleString(conn);
-        assert(strObj.obj && len == strlen((char *)strObj.obj));
         rObj.obj = strObj.obj;
     }
 

src/resp.c

@@ -125,6 +125,7 @@ char *RedisObject_RESP_array(RedisObject_t obj)
     {
         memcpy(copyPtr, constRESPs[i].RESP, constRESPs[i].length);
         copyPtr += constRESPs[i].length;
+        free(constRESPs[i].RESP);
     }
 
     free(constRESPs);