svn commit: r283718 - stable/10/sys/netinet

Michael Tuexen tuexen at FreeBSD.org
Fri May 29 12:30:20 UTC 2015


Author: tuexen
Date: Fri May 29 12:30:19 2015
New Revision: 283718
URL: https://svnweb.freebsd.org/changeset/base/283718

Log:
  MFC r277348:
  
  Remove an unnecessary check.
  
  Reported by:	Coverity
  CID:		749576

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

Modified: stable/10/sys/netinet/sctp_input.c
==============================================================================
--- stable/10/sys/netinet/sctp_input.c	Fri May 29 12:27:45 2015	(r283717)
+++ stable/10/sys/netinet/sctp_input.c	Fri May 29 12:30:19 2015	(r283718)
@@ -3108,7 +3108,6 @@ sctp_handle_ecn_cwr(struct sctp_cwr_chun
 	uint32_t cwr_tsn;
 
 	cwr_tsn = ntohl(cp->tsn);
-
 	override = cp->ch.chunk_flags & SCTP_CWR_REDUCE_OVERRIDE;
 	TAILQ_FOREACH(chk, &stcb->asoc.control_send_queue, sctp_next) {
 		if (chk->rec.chunk_id.id != SCTP_ECN_ECHO) {
@@ -3124,10 +3123,8 @@ sctp_handle_ecn_cwr(struct sctp_cwr_chun
 			stcb->asoc.ecn_echo_cnt_onq--;
 			TAILQ_REMOVE(&stcb->asoc.control_send_queue, chk,
 			    sctp_next);
-			if (chk->data) {
-				sctp_m_freem(chk->data);
-				chk->data = NULL;
-			}
+			sctp_m_freem(chk->data);
+			chk->data = NULL;
 			stcb->asoc.ctrl_queue_cnt--;
 			sctp_free_a_chunk(stcb, chk, SCTP_SO_NOT_LOCKED);
 			if (override == 0) {


More information about the svn-src-all mailing list