[lkml]   [1997]   [Apr]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: Diff against updated 3c59x.c for Boomerang boards
  Nathan Bryant <> writes:
> David Miller recently discovered a memory leak that exists in most of the
> Linux network drivers. I believe the updated 3c59x.c (for Boomerang
> cards) on suffers from this problem;
> here's a diff which I think should correct it.
It seems that the drivers on cesdis, written for a particular kernel,
aren't updated along with the kernel. :-(

> +
> + if(skb == NULL) {
> + dev_tint(dev);
> + return NULL;
> }
> Can anybody explain the (skb == NULL) check and the call to dev_tint()?
> There is already a similar section in the updated 3c59x.c, but I'm not
> really sure what it does.

As far as I could tell from looking at the kernel code right now, the
start_xmit code is no longer called with a NULL skb.

I also wondered whether the code that immediately follows in most drivers,

if (skb->len <= 0)
return 0;

has any legal right to existence these days.

The author should gaze at Noah, and ... learn, as they did in the Ark, to crowd
a great deal of matter into a very small compass.
- Sydney, Smith, Edinburgh Review
Matthias Urlichs \ noris network GmbH / Xlink-POP Nürnberg
Schleiermacherstraße 12 \ Linux+Internet / EMail:
90491 Nürnberg (Germany) \ Consulting+Programming+Networking+etc'ing
PGP: 1024/4F578875 1B 89 E2 1C 43 EA 80 44 15 D2 29 CF C6 C7 E0 DE
Click <A HREF="">here</A>. 42

 \ /
  Last update: 2005-03-22 13:39    [W:0.042 / U:1.540 seconds]
©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site