[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.019 / U:1.124 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site