diff options
| author | Patrick McHardy <[email protected]> | 2007-07-11 06:06:43 +0000 |
|---|---|---|
| committer | David S. Miller <[email protected]> | 2007-07-11 06:06:43 +0000 |
| commit | 3be550f34b03e5eb762f74d447ebbeba97efbd6d (patch) | |
| tree | 07881880089af5f9d8dbea1444f09e34a0100ef1 /net/unix/af_unix.c | |
| parent | [IPV6]: Remove unneeded pointer idev from addrconf_cleanup(). (diff) | |
| download | kernel-3be550f34b03e5eb762f74d447ebbeba97efbd6d.tar.gz kernel-3be550f34b03e5eb762f74d447ebbeba97efbd6d.zip | |
[UDP]: Fix length check.
Rémi Denis-Courmont wrote:
> Right. By the way, shouldn't "len" rather be signed in there?
>
> unsigned int len;
>
> /* if we're overly short, let UDP handle it */
> len = skb->len - sizeof(struct udphdr);
> if (len <= 0)
> goto udp;
It should, but the < 0 case can't happen since __udp4_lib_rcv
already makes sure that we have at least a complete UDP header.
Anyways, this patch fixes it.
Signed-off-by: Patrick McHardy <[email protected]>
Signed-off-by: David S. Miller <[email protected]>
Diffstat (limited to 'net/unix/af_unix.c')
0 files changed, 0 insertions, 0 deletions