[Qemu-devel] [PATCH V2] net/net: Bugfix for net_fill_rstate()

[Zhang Chen]

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 <address@hidden>
Signed-off-by: Li Zhijian <address@hidden>
---
 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;
 }
-- 
2.7.4