[PATCH 1/6] sfxge: fix mbuf leak if it does not fit in software queue

[Adrian Chadd]

Hi!

Who's the solarflare driver maintainer?

Ah, there isn't one. The closest is Ben Hutchings <bwh at 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 at 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 at 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 at freebsd.org mailing list
> http://lists.freebsd.org/mailman/listinfo/freebsd-net
> To unsubscribe, send any mail to "freebsd-net-unsubscribe at freebsd.org"