Author: tuexen
Date: Fri May 29 11:52:32 2015
New Revision: 283707
URL: https://svnweb.freebsd.org/changeset/base/283707

Log:
  MFC r276009:
  
  Don't check twice that inp is not NULL.
  
  Reported by:  Coverity
  CID:          748671

Modified:
  stable/10/sys/netinet/sctputil.c
Directory Properties:
  stable/10/   (props changed)

Modified: stable/10/sys/netinet/sctputil.c
==============================================================================
--- stable/10/sys/netinet/sctputil.c    Fri May 29 11:47:54 2015        
(r283706)
+++ stable/10/sys/netinet/sctputil.c    Fri May 29 11:52:32 2015        
(r283707)
@@ -1949,7 +1949,7 @@ sctp_timer_start(int t_type, struct sctp
                 * though we use a different timer. We also add the HB timer
                 * PLUS a random jitter.
                 */
-               if ((inp == NULL) || (stcb == NULL) || (net == NULL)) {
+               if ((stcb == NULL) || (net == NULL)) {
                        return;
                } else {
                        uint32_t rndval;
@@ -2004,9 +2004,6 @@ sctp_timer_start(int t_type, struct sctp
                 * nothing needed but the endpoint here ususually about 60
                 * minutes.
                 */
-               if (inp == NULL) {
-                       return;
-               }
                tmr = &inp->sctp_ep.signature_change;
                to_ticks = inp->sctp_ep.sctp_timeoutticks[SCTP_TIMER_SIGNATURE];
                break;
@@ -2023,9 +2020,6 @@ sctp_timer_start(int t_type, struct sctp
                 * timer since that has stopped and we are in the GONE
                 * state.
                 */
-               if (inp == NULL) {
-                       return;
-               }
                tmr = &inp->sctp_ep.signature_change;
                to_ticks = MSEC_TO_TICKS(SCTP_INP_KILL_TIMEOUT);
                break;
@@ -2034,10 +2028,7 @@ sctp_timer_start(int t_type, struct sctp
                 * Here we use the value found in the EP for PMTU ususually
                 * about 10 minutes.
                 */
-               if ((stcb == NULL) || (inp == NULL)) {
-                       return;
-               }
-               if (net == NULL) {
+               if ((stcb == NULL) || (net == NULL)) {
                        return;
                }
                if (net->dest_state & SCTP_ADDR_NO_PMTUD) {
@@ -2063,7 +2054,7 @@ sctp_timer_start(int t_type, struct sctp
                 * Here we use the endpoints shutdown guard timer usually
                 * about 3 minutes.
                 */
-               if ((inp == NULL) || (stcb == NULL)) {
+               if (stcb == NULL) {
                        return;
                }
                to_ticks = 
inp->sctp_ep.sctp_timeoutticks[SCTP_TIMER_MAXSHUTDOWN];
_______________________________________________
svn-src-stable-10@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-stable-10
To unsubscribe, send any mail to "svn-src-stable-10-unsubscr...@freebsd.org"

Reply via email to