Fix memory issue in parser for long string

Do a check on the cursor before trying to get length, because we could already be at the end. [#5]
rabbitmq · Mar 28, 2016 · 9d64a8e · 9d64a8e
1 parent 5a07e90
commit 9d64a8e
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/RMQClient/AMQParser.m b/RMQClient/AMQParser.m
@@ -107,6 +107,9 @@ - (NSString *)parseShortString {
 }
 
 - (NSString *)parseLongString {
+    if (self.cursor >= self.end) {
+        return @"";
+    }
     UInt32 length = CFSwapInt32BigToHost(*(UInt32 *)self.cursor);
     const char *expectedStringEnd = self.cursor + sizeof(length) + length;