git: 893ed42eca32 - main - tcp: Make use of enum for sack_changed

From: Richard Scheffenegger <rscheff_at_FreeBSD.org>
Date: Sat, 06 Jan 2024 19:44:05 UTC
The branch main has been updated by rscheff:

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

commit 893ed42eca32f7b5e3a5b8cef3f364be0d9cf3ef
Author:     Richard Scheffenegger <rscheff@FreeBSD.org>
AuthorDate: 2024-01-06 19:07:39 +0000
Commit:     Richard Scheffenegger <rscheff@FreeBSD.org>
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);
 			}