netdev
[Top] [All Lists]

Re: [RFT][PATCH] cleanup net/atm/br2684.c

To: Stephen Hemminger <shemminger@xxxxxxxx>
Subject: Re: [RFT][PATCH] cleanup net/atm/br2684.c
From: Mitchell Blank Jr <mitch@xxxxxxxxxx>
Date: Mon, 11 Aug 2003 14:58:57 -0700
Cc: chas williams <chas@xxxxxxxxxxxxxxxx>, netdev@xxxxxxxxxxx
In-reply-to: <20030811114823.5b81474c.shemminger@xxxxxxxx>
References: <20030811114823.5b81474c.shemminger@xxxxxxxx>
Sender: netdev-bounce@xxxxxxxxxxx
User-agent: Mutt/1.4.1i
You might also want to send this patch to the linux-atm mailing list:
  linux-atm-general@xxxxxxxxxxxxxxxxxxxxx
Unfortunately you need to subscribe to post.
  http://lists.sourceforge.net/lists/listinfo/linux-atm-general

Not sure how many ATM users are following 2.6 yet though.  I might be
able to give it a spin soon.

> -                     unregister_netdev(&brdev->net_dev);
> -                     kfree(brdev);
> +                     unregister_netdev(brdev->net_dev);
> +                     kfree(brdev->net_dev);

Mmmm.. I see this snippet in your diff in multiplce places and it looks
wrong.  Where does the "brdev" structure get freed now?

-Mitch

<Prev in Thread] Current Thread [Next in Thread>