Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 18 Mar 2014 04:44:00 -0700
From:      Adrian Chadd <adrian@freebsd.org>
To:        Andrew Rybchenko <Andrew.Rybchenko@oktetlabs.ru>
Cc:        "net@freebsd.org" <net@freebsd.org>
Subject:   Re: [PATCH 1/6] sfxge: fix mbuf leak if it does not fit in software queue
Message-ID:  <CAJ-VmomkRXKauiDm4OKuiYbQYH8GdnJb8xHNfRzzNeJ7NpoDKQ@mail.gmail.com>
In-Reply-To: <53280DB3.4080900@oktetlabs.ru>
References:  <53280DB3.4080900@oktetlabs.ru>

next in thread | previous in thread | raw e-mail | index | archive | help
Hi!

Who's the solarflare driver maintainer?

Ah, there isn't one. The closest is Ben Hutchings <bwh@solarflare.com>.

I can commit these if no-one else is willing but I don't have any
solarflare hardware to test it on.


-a


On 18 March 2014 02:11, Andrew Rybchenko <Andrew.Rybchenko@oktetlabs.ru> wrote:
>
> sfxge: 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@oktetlabs.ru>
> Sponsored by:   Solarflare Communications, Inc.
>
> diff -r e2bc8f64f1b2 -r ff9f5d3dbafe src/driver/freebsd/sfxge_tx.c
> --- a/head/sys/dev/sfxge/sfxge_tx.c    Tue Mar 04 13:13:05 2014 +0400
> +++ b/head/sys/dev/sfxge/sfxge_tx.c    Tue Mar 04 13:15:13 2014 +0400
> @@ -536,6 +536,7 @@
>      return (0);
>
>  fail:
> +    m_freem(m);
>      return (rc);
>
>  }
>
> _______________________________________________
> freebsd-net@freebsd.org mailing list
> http://lists.freebsd.org/mailman/listinfo/freebsd-net
> To unsubscribe, send any mail to "freebsd-net-unsubscribe@freebsd.org"



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