Skip to content

Commit

Permalink
[CAN]: Fix plain integer definitions in userspace header.
Browse files Browse the repository at this point in the history
This patch fixes the use of plain integers instead of __u32 in a struct
that is visible from kernel space and user space.

Thanks to Sam Ravnborg for pointing out the wrong plain int usage.

Signed-off-by: Oliver Hartkopp <oliver@hartkopp.net>
Acked-by: Sam Ravnborg <sam@ravnborg.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
  • Loading branch information
Oliver Hartkopp authored and davem330 committed Jan 28, 2008
1 parent ccb2963 commit 4195e31
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions include/linux/can/bcm.h
Original file line number Diff line number Diff line change
Expand Up @@ -26,12 +26,12 @@
* @frames: array of CAN frames.
*/
struct bcm_msg_head {
int opcode;
int flags;
int count;
__u32 opcode;
__u32 flags;
__u32 count;
struct timeval ival1, ival2;
canid_t can_id;
int nframes;
__u32 nframes;
struct can_frame frames[0];
};

Expand Down

0 comments on commit 4195e31

Please sign in to comment.