The branch main has been updated by rscheff:

URL: 
https://cgit.FreeBSD.org/src/commit/?id=893ed42eca32f7b5e3a5b8cef3f364be0d9cf3ef

commit 893ed42eca32f7b5e3a5b8cef3f364be0d9cf3ef
Author:     Richard Scheffenegger <[email protected]>
AuthorDate: 2024-01-06 19:07:39 +0000
Commit:     Richard Scheffenegger <[email protected]>
CommitDate: 2024-01-06 19:23:52 +0000

    tcp: Make use of enum for sack_changed
    
    No functional change.
    
    Reviewed By:           tuexen, #transport
    MFC after:             3 days
    Sponsored by:          NetApp, Inc.
    Differential Revision: https://reviews.freebsd.org/D43346
---
 sys/netinet/tcp_input.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/sys/netinet/tcp_input.c b/sys/netinet/tcp_input.c
index ab8fc4b4a3e4..1d6430f6ef20 100644
--- a/sys/netinet/tcp_input.c
+++ b/sys/netinet/tcp_input.c
@@ -2499,7 +2499,8 @@ tcp_do_segment(struct tcpcb *tp, struct mbuf *m, struct 
tcphdr *th,
                        goto dropafterack;
                }
                if (tcp_is_sack_recovery(tp, &to)) {
-                       if (((sack_changed = tcp_sack_doack(tp, &to, 
th->th_ack)) != 0) &&
+                       sack_changed = tcp_sack_doack(tp, &to, th->th_ack);
+                       if ((sack_changed != SACK_NOCHANGE) &&
                            (tp->t_flags & TF_LRD)) {
                                tcp_sack_lost_retransmission(tp, th);
                        }

Reply via email to