linux-toradex.git/net/can, branch v2.6.25.17

can: add sanity checks

2008-07-24T16:14:06+00:00

commit 7f2d38eb7a42bea1c1df51bbdaa2ca0f0bdda07f upstream

Even though the CAN netlayer only deals with CAN netdevices, the
netlayer interface to the userspace and to the device layer should
perform some sanity checks.

This patch adds several sanity checks that mainly prevent userspace apps
to send broken content into the system that may be misinterpreted by
some other userspace application.

Signed-off-by: Oliver Hartkopp 
Signed-off-by: Urs Thuermann 
Acked-by: Andre Naujoks 
Signed-off-by: David S. Miller 
Signed-off-by: Greg Kroah-Hartman

can: Fix copy_from_user() results interpretation

2008-06-16T20:19:54+00:00

[ Upstream commit: 3f91bd420a955803421f2db17b2e04aacfbb2bb8 ]

Both copy_to_ and _from_user return the number of bytes, that failed to
reach their destination, not the 0/-EXXX values.

Based on patch from Pavel Emelyanov 

Signed-off-by: Sam Ravnborg 
Acked-by: Oliver Hartkopp 
Signed-off-by: David S. Miller 
Signed-off-by: Chris Wright

can: Fix can_send() handling on dev_queue_xmit() failures

2008-05-15T14:50:03+00:00

[ Upstream commit: c2ab7ac225e29006b7117d6a9fe8f3be8d98b0c2 ]

The tx packet counting and the local loopback of CAN frames should
only happen in the case that the CAN frame has been enqueued to the
netdevice tx queue successfully.

Thanks to Andre Naujoks  for reporting this issue.

Signed-off-by: Oliver Hartkopp 
Signed-off-by: Urs Thuermann 
Signed-off-by: David S. Miller 
Signed-off-by: Greg Kroah-Hartman

net: Fix wrong interpretation of some copy_to_user() results.

2008-05-01T21:44:32+00:00

[ Upstream commit: 653252c2302cdf2dfbca66a7e177f7db783f9efa ]

I found some places, that erroneously return the value obtained from
the copy_to_user() call: if some amount of bytes were not able to get
to the user (this is what this one returns) the proper behavior is to
return the -EFAULT error, not that number itself.

Signed-off-by: Pavel Emelyanov 
Signed-off-by: David S. Miller 
Signed-off-by: Greg Kroah-Hartman

[CAN]: Minor clean-ups

2008-02-08T02:05:04+00:00

Remove unneeded variable.
Rename local variable error to err like in all other places.
Some white-space changes.

Signed-off-by: Urs Thuermann 
Signed-off-by: Oliver Hartkopp 
Signed-off-by: David S. Miller

[CAN]: Move proto_{,un}register() out of spin-locked region

2008-02-08T02:04:45+00:00

The implementation of proto_register() has changed so that it can now
sleep.  The call to proto_register() must be moved out of the
spin-locked region.

Signed-off-by: Urs Thuermann 
Signed-off-by: Oliver Hartkopp 
Signed-off-by: David S. Miller

[CAN]: Clean up module auto loading

2008-02-08T02:04:21+00:00

Remove local char array to construct module name.
Don't call request_module() when CONFIG_KMOD is not set.

Signed-off-by: Urs Thuermann 
Signed-off-by: Oliver Hartkopp 
Signed-off-by: David S. Miller

[CAN]: Add virtual CAN netdevice driver

2008-01-28T22:54:12+00:00

This patch adds the virtual CAN bus (vcan) network driver.
The vcan device is just a loopback device for CAN frames, no
real CAN hardware is involved.

Signed-off-by: Oliver Hartkopp 
Signed-off-by: Urs Thuermann 
Signed-off-by: David S. Miller

[CAN]: Add broadcast manager (bcm) protocol

2008-01-28T22:54:11+00:00

This patch adds the CAN broadcast manager (bcm) protocol.

Signed-off-by: Oliver Hartkopp 
Signed-off-by: Urs Thuermann 
Signed-off-by: David S. Miller

[CAN]: Add raw protocol

2008-01-28T22:54:10+00:00

This patch adds the CAN raw protocol.

Signed-off-by: Oliver Hartkopp 
Signed-off-by: Urs Thuermann 
Signed-off-by: David S. Miller