From: goldsimon <goldsimon@gmx.de>
Date: Fri, 28 Oct 2011 11:09:04 +0000 (+0200)
Subject: fixed bug #34638: Dead code in tcp_receive - pcb->dupacks
X-Git-Url: http://rtime.felk.cvut.cz/gitweb/pes-rpp/rpp-lwip.git/commitdiff_plain/edcc859b589618128d67f1dc470562177833d9d6

fixed bug #34638: Dead code in tcp_receive - pcb->dupacks
---

diff --git a/CHANGELOG b/CHANGELOG
index f5f553ce..bbf80928 100644
--- a/CHANGELOG
+++ b/CHANGELOG
@@ -69,6 +69,9 @@ HISTORY
 
  ++ Bugfixes:
 
+  2011-10-28: Simon Goldschmidt
+  * tcp_in.c: fixed bug #34638: Dead code in tcp_receive - pcb->dupacks
+
   2011-10-23: Simon Goldschmidt
   * mem.c: fixed bug #34429: possible memory corruption with
     LWIP_ALLOW_MEM_FREE_FROM_OTHER_CONTEXT set to 1
diff --git a/src/core/tcp_in.c b/src/core/tcp_in.c
index b2d9db2a..0dd2c904 100644
--- a/src/core/tcp_in.c
+++ b/src/core/tcp_in.c
@@ -938,8 +938,9 @@ tcp_receive(struct tcp_pcb *pcb)
             /* Clause 5 */
             if (pcb->lastack == ackno) {
               found_dupack = 1;
-              if (pcb->dupacks + 1 > pcb->dupacks)
+              if ((u8_t)(pcb->dupacks + 1) > pcb->dupacks) {
                 ++pcb->dupacks;
+              }
               if (pcb->dupacks > 3) {
                 /* Inflate the congestion window, but not if it means that
                    the value overflows. */