diff options
| author | Eric Dumazet <eric.dumazet@gmail.com> | 2010-04-10 01:47:31 +0200 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2010-04-13 12:03:14 +0200 |
| commit | 4ffa87012efd7b664762b579213d4663560ef4a3 (patch) | |
| tree | 37e255f778b1bda29341799b6aa209e62ce69f84 /net | |
| parent | e1000e: stop cleaning when we reach tx_ring->next_to_use (diff) | |
| download | linux-4ffa87012efd7b664762b579213d4663560ef4a3.tar.xz linux-4ffa87012efd7b664762b579213d4663560ef4a3.zip | |
can: avoids a false warning
At this point optlen == sizeof(sfilter) but some compilers are dumb.
Reported-by: Németh Márton <nm127@freemail.h
Signed-off-by: Eric Dumazet <eric.dumazet@gmail.com>
Acked-by: Oliver Hartkopp <oliver@hartkopp.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net')
| -rw-r--r-- | net/can/raw.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/can/raw.c b/net/can/raw.c index 3a7dffb6519c..da99cf153b33 100644 --- a/net/can/raw.c +++ b/net/can/raw.c @@ -445,7 +445,7 @@ static int raw_setsockopt(struct socket *sock, int level, int optname, return -EFAULT; } } else if (count == 1) { - if (copy_from_user(&sfilter, optval, optlen)) + if (copy_from_user(&sfilter, optval, sizeof(sfilter))) return -EFAULT; } |