插件发送消息可能泄露 #998

kbengine/kbengine#998

Author: jay602

Bundle.cs

@@ -25,8 +25,20 @@ public Bundle()
 
 		public void clear()
 		{
-			stream = MemoryStream.createObject();
-			streamList = new List<MemoryStream>();
+            // 把不用的MemoryStream放回缓冲池，以减少垃圾回收的消耗
+            for (int i = 0; i < streamList.Count; ++i)
+            {
+                if (stream != streamList[i])
+                    streamList[i].reclaimObject();
+            }
+
+            streamList.Clear();
+
+            if(stream != null)
+                stream.clear();
+            else
+                stream = MemoryStream.createObject();
+
 			numMessage = 0;
 			messageLength = 0;
 			msgtype = null;
@@ -110,21 +122,12 @@ public void send(NetworkInterface networkInterface)
 				Dbg.ERROR_MSG("Bundle::send: networkInterface invalid!");  
 			}
 
-			// 把不用的MemoryStream放回缓冲池，以减少垃圾回收的消耗
-			for (int i = 0; i < streamList.Count; ++i)
-			{
-				streamList[i].reclaimObject();
-			}
-
-			streamList.Clear();
-			stream.clear();
-
 			// 我们认为，发送完成，就视为这个bundle不再使用了，
 			// 所以我们会把它放回对象池，以减少垃圾回收带来的消耗，
 			// 如果需要继续使用，应该重新Bundle.createObject()，
 			// 如果外面不重新createObject()而直接使用，就可能会出现莫名的问题，
 			// 仅以此备注，警示使用者。
-			Bundle.reclaimObject(this);
+			reclaimObject();
 		}
 
 		public void checkStream(int v)