Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 18 Mar 2014 15:01:32 +0000 (UTC)
From:      "George V. Neville-Neil" <gnn@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r263302 - head/sys/dev/sfxge
Message-ID:  <201403181501.s2IF1Wld098850@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: gnn
Date: Tue Mar 18 15:01:32 2014
New Revision: 263302
URL: http://svnweb.freebsd.org/changeset/base/263302

Log:
  fix mbuf leak if it does not fit in software queue
  
  mbuf should be owned by if_transmit function in any case.
  
  Submitted-by:   Andrew Rybchenko <Andrew.Rybchenko at oktetlabs.ru>
  Sponsored by:   Solarflare Communications, Inc.

Modified:
  head/sys/dev/sfxge/sfxge_tx.c

Modified: head/sys/dev/sfxge/sfxge_tx.c
==============================================================================
--- head/sys/dev/sfxge/sfxge_tx.c	Tue Mar 18 14:41:18 2014	(r263301)
+++ head/sys/dev/sfxge/sfxge_tx.c	Tue Mar 18 15:01:32 2014	(r263302)
@@ -536,6 +536,7 @@ sfxge_tx_packet_add(struct sfxge_txq *tx
 	return (0);
 
 fail:
+	m_freem(m);
 	return (rc);
 	
 }



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201403181501.s2IF1Wld098850>