Fix Packet size mismatch! error on collection drop

Author: gedaiu

mongodb/vibe/db/mongo/collection.d

@@ -1313,6 +1313,15 @@ struct MongoCollection {
 	}
 }
 
+/// Drop a collection
+@safe unittest {
+	import vibe.db.mongo.mongo;
+  auto client = connectMongoDB("127.0.0.1");
+  auto chunks = client.getCollection("test.fs.chunks");
+
+  chunks.drop;
+}
+
 /**
   Specifies a level of isolation for read operations. For example, you can use read concern to only read data that has propagated to a majority of nodes in a replica set.
 

mongodb/vibe/db/mongo/connection.d

@@ -714,7 +714,7 @@ final class MongoConnection {
 	{
 		import std.traits;
 
-		auto bytes_read = m_bytesRead;
+		auto packet_start_index = m_bytesRead;
 		int msglen = recvInt();
 		int resid = recvInt();
 		int respto = recvInt();
@@ -731,8 +731,8 @@ final class MongoConnection {
 			sectionLength -= uint.sizeof; // CRC present
 
 		bool gotSec0;
-		while (m_bytesRead - bytes_read < sectionLength) {
-			// TODO: directly deserialize from the wire
+		while (m_bytesRead - packet_start_index < msglen) {
+		// TODO: directly deserialize from the wire
 			static if (!dupBson) {
 				ubyte[256] buf = void;
 				ubyte[] bufsl = buf;
@@ -783,9 +783,9 @@ final class MongoConnection {
 			logDiagnostic("recvData: crc=%s (discarded)", crc);
 		}
 
-		assert(bytes_read + msglen == m_bytesRead,
+		assert(packet_start_index + msglen == m_bytesRead,
 			format!"Packet size mismatch! Expected %s bytes, but read %s."(
-				msglen, m_bytesRead - bytes_read));
+				msglen, m_bytesRead - packet_start_index));
 
 		return resid;
 	}