net: remove redundant check for timer pending state before del_timer

As in del_timer() there has already placed a timer_pending() function
to check whether the timer to be deleted is pending or not, it's
unnecessary to check timer pending state again before del_timer() is
called.

Signed-off-by: Ying Xue <ying.xue@windriver.com>
Cc: Eric Dumazet <edumazet@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

1 files changed, 1 insertions, 3 deletions

diff --git a/net/sctp/outqueue.c b/net/sctp/outqueue.c
index 9bcdbd02d777..01dca753db16 100644
--- a/net/sctp/outqueue.c
+++ b/net/sctp/outqueue.c
@@ -1700,10 +1700,8 @@ static void sctp_check_transmitted(struct sctp_outq *q,
                  * address.
                  */
                 if (!transport->flight_size) {
-                        if (timer_pending(&transport->T3_rtx_timer) &&
-                            del_timer(&transport->T3_rtx_timer)) {
+                        if (del_timer(&transport->T3_rtx_timer))
                                 sctp_transport_put(transport);
-                        }
                 } else if (restart_timer) {
                         if (!mod_timer(&transport->T3_rtx_timer,
                                        jiffies + transport->rto))