On 2016年08月18日 11:23, Zhang Chen wrote:
When network is busy, we will receive multiple packets
at one time. In that situation, this bug will lost all
packets except the first one.

Signed-off-by: Zhang Chen <zhangchen.f...@cn.fujitsu.com>
Signed-off-by: Li Zhijian <lizhij...@cn.fujitsu.com>
---
  net/net.c | 8 ++++----
  1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/net/net.c b/net/net.c
index a8e2e6b..0b11f10 100644
--- a/net/net.c
+++ b/net/net.c
@@ -1586,9 +1586,8 @@ void net_socket_rs_init(SocketReadState *rs,
/*
   * Returns
- * 0: SocketReadState is not ready
- * 1: SocketReadState is ready
- * otherwise error occurs
+ * 0: success
+ * -1: error occurs
   */
  int net_fill_rstate(SocketReadState *rs, const uint8_t *buf, int size)
  {
@@ -1636,10 +1635,11 @@ int net_fill_rstate(SocketReadState *rs, const uint8_t 
*buf, int size)
                  if (rs->finalize) {
                      rs->finalize(rs);
                  }
-                return 1;
              }
              break;
          }
      }
+
+    assert(size == 0);
      return 0;
  }

Applied with minor tweaks on commit log.

Thanks

Reply via email to