https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=195883

            Bug ID: 195883
           Summary: Incorrect handling of errors in tcp_output() function
           Product: Base System
           Version: 11.0-CURRENT
          Hardware: Any
                OS: Any
            Status: New
          Severity: Affects Many People
          Priority: ---
         Component: kern
          Assignee: [email protected]
          Reporter: [email protected]

Created attachment 150466
  --> https://bugs.freebsd.org/bugzilla/attachment.cgi?id=150466&action=edit
Suggested patch

Hi,

In the tcp_output() function, nothing is protecting TCP timers from elapsing
during its execution. Under certain conditions a TCP timeout can overlap
if_transmit() and iff the if_transmit() function returns non-zero in that case,
the following repeated check will give a different outcome than previously:

((tp->t_flags & TF_FORCEDATA) == 0 ||
!tcp_timer_active(tp, TT_PERSIST))

This leads up to a panic where the inpcb transmit state gets corrupted.

Suggested solution:
Store outcome from previous check and use that result later on.

--HPS

-- 
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
[email protected] mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-bugs
To unsubscribe, send any mail to "[email protected]"

Reply via email to