Guard aggainst empty list.

fingerkiller · Feb 5, 2013 · 6c16cc8 · 6c16cc8
1 parent 6625620
commit 6c16cc8
Showing 1 changed file with 12 additions and 7 deletions.
diff --git a/Core/HTTPConnection.m b/Core/HTTPConnection.m
@@ -2388,7 +2388,9 @@ - (void)socket:(GCDAsyncSocket *)sock didWriteDataWithTag:(long)tag
 	if (tag == HTTP_PARTIAL_RESPONSE_BODY)
 	{
 		// Update the amount of data we have in asyncSocket's write queue
-		[responseDataSizes removeObjectAtIndex:0];
+        if ([responseDataSizes count] > 0) {
+            [responseDataSizes removeObjectAtIndex:0];
+        }
 
 		// We only wrote a part of the response - there may be more
 		[self continueSendingStandardResponseBody];
@@ -2397,8 +2399,9 @@ - (void)socket:(GCDAsyncSocket *)sock didWriteDataWithTag:(long)tag
 	{
 		// Update the amount of data we have in asyncSocket's write queue.
 		// This will allow asynchronous responses to continue sending more data.
-		[responseDataSizes removeObjectAtIndex:0];
-
+        if ([responseDataSizes count] > 0) {
+            [responseDataSizes removeObjectAtIndex:0];
+        }
 		// Don't continue sending the response yet.
 		// The chunked footer that was sent after the body will tell us if we have more data to send.
 	}
@@ -2410,16 +2413,18 @@ - (void)socket:(GCDAsyncSocket *)sock didWriteDataWithTag:(long)tag
 	else if (tag == HTTP_PARTIAL_RANGE_RESPONSE_BODY)
 	{
 		// Update the amount of data we have in asyncSocket's write queue
-		[responseDataSizes removeObjectAtIndex:0];
-
+        if ([responseDataSizes count] > 0) {
+            [responseDataSizes removeObjectAtIndex:0];
+        }
 		// We only wrote a part of the range - there may be more
 		[self continueSendingSingleRangeResponseBody];
 	}
 	else if (tag == HTTP_PARTIAL_RANGES_RESPONSE_BODY)
 	{
 		// Update the amount of data we have in asyncSocket's write queue
-		[responseDataSizes removeObjectAtIndex:0];
-
+        if ([responseDataSizes count] > 0) {
+            [responseDataSizes removeObjectAtIndex:0];
+        }
 		// We only wrote part of the range - there may be more, or there may be more ranges
 		[self continueSendingMultiRangeResponseBody];
 	}