From: Dennis Aberilla <dennis.aberilla@mimomax.com>
To: David Miller <davem@davemloft.net>
Cc: info@mocean-labs.com, netdev@vger.kernel.org
Subject: Re: [PATCH] drivers/net: ks8842 Fix crash on received packet when in PIO mode.
Date: Fri, 20 May 2011 11:30:16 +1200 [thread overview]
Message-ID: <20110519233014.GA7329@dens-work> (raw)
In-Reply-To: <20110519.161228.1163475688237712245.davem@davemloft.net>
On Thu, May 19, 2011 at 04:12:28PM -0400, David Miller wrote:
> From: Dennis Aberilla <dennis.aberilla@mimomax.com>
> Date: Thu, 19 May 2011 10:59:47 +1200
>
> > This patch fixes a kernel crash during packet reception due to not
> > enough allocated bytes for the skb. This applies to the driver when
> > running in PIO mode in an ISA bus setup.
> >
> > Signed-off-by: Dennis Aberilla <denzzzhome@yahoo.com>
>
> If you're trying to accomodate the fact that the loops iterate
> always by 2 or 4 bytes at a time, then you will need to allocate
> up to "3" bytes of slack space, not just "2".
>
> You need to describe exactly what the precise problem is in your
> commit message, or else people might find it difficult to figure
> out exactly what the problem is.
Ah right, it should actually be 3. Yes, it accounts for the fact that the
loops are reading the Rx buffer 4 bytes at a time. Sorry it wasn't too
clear previously.
|Dennis
=======================================================================
This email, including any attachments, is only for the intended
addressee. It is subject to copyright, is confidential and may be
the subject of legal or other privilege, none of which is waived or
lost by reason of this transmission.
If the receiver is not the intended addressee, please accept our
apologies, notify us by return, delete all copies and perform no
other act on the email.
Unfortunately, we cannot warrant that the email has not been
altered or corrupted during transmission.
=======================================================================
next prev parent reply other threads:[~2011-05-19 23:30 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-05-18 22:59 [PATCH] drivers/net: ks8842 Fix crash on received packet when in PIO mode Dennis Aberilla
2011-05-19 20:12 ` David Miller
2011-05-19 23:30 ` Dennis Aberilla [this message]
-- strict thread matches above, loose matches on Subject: below --
2011-05-29 21:46 Dennis Aberilla
2011-05-31 22:15 ` David Miller
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20110519233014.GA7329@dens-work \
--to=dennis.aberilla@mimomax.com \
--cc=davem@davemloft.net \
--cc=info@mocean-labs.com \
--cc=netdev@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).