Author: tuexen
Date: Wed Jul 31 21:29:17 2019
New Revision: 350487
URL: https://svnweb.freebsd.org/changeset/base/350487

Log:
  Consistently cleanup mbufs in case of other memory errors.
  
  MFC after:            3 days

Modified:
  head/sys/netinet/sctp_output.c

Modified: head/sys/netinet/sctp_output.c
==============================================================================
--- head/sys/netinet/sctp_output.c      Wed Jul 31 21:21:34 2019        
(r350486)
+++ head/sys/netinet/sctp_output.c      Wed Jul 31 21:29:17 2019        
(r350487)
@@ -5586,8 +5586,7 @@ do_a_abort:
        m = sctp_get_mbuf_for_msg(MCLBYTES, 0, M_NOWAIT, 1, MT_DATA);
        if (m == NULL) {
                /* No memory, INIT timer will re-attempt. */
-               if (op_err)
-                       sctp_m_freem(op_err);
+               sctp_m_freem(op_err);
                return;
        }
        chunk_len = (uint16_t)sizeof(struct sctp_init_ack_chunk);
@@ -5776,8 +5775,11 @@ do_a_abort:
                                net->ro._s_addr = 
sctp_source_address_selection(inp,
                                    stcb, (sctp_route_t *)&net->ro,
                                    net, 0, vrf_id);
-                               if (net->ro._s_addr == NULL)
+                               if (net->ro._s_addr == NULL) {
+                                       sctp_m_freem(op_err);
+                                       sctp_m_freem(m);
                                        return;
+                               }
 
                                net->src_addr_selected = 1;
 
@@ -5806,8 +5808,11 @@ do_a_abort:
                                net->ro._s_addr = 
sctp_source_address_selection(inp,
                                    stcb, (sctp_route_t *)&net->ro,
                                    net, 0, vrf_id);
-                               if (net->ro._s_addr == NULL)
+                               if (net->ro._s_addr == NULL) {
+                                       sctp_m_freem(op_err);
+                                       sctp_m_freem(m);
                                        return;
+                               }
 
                                net->src_addr_selected = 1;
                        }
@@ -5878,6 +5883,7 @@ do_a_abort:
        so = inp->sctp_socket;
        if (so == NULL) {
                /* memory problem */
+               sctp_m_freem(op_err);
                sctp_m_freem(m);
                return;
        } else {
_______________________________________________
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

Reply via email to