Re: [PATCH 3/4] Input: zforce - Remove unnecessary payload data checks

2014-01-27 Thread Heiko Stübner
On Monday, 27. January 2014 19:46:12 Luis Ortega wrote: > The function zforce_read_packet() reads 2 values (bytes) of payload > header, validates them and then proceeds to read the payload body. > The function stores all these in a u8 buffer. > > The PAYLOAD_LENGTH check seems to be trying to dete

[PATCH 3/4] Input: zforce - Remove unnecessary payload data checks

2014-01-27 Thread Luis Ortega
The function zforce_read_packet() reads 2 values (bytes) of payload header, validates them and then proceeds to read the payload body. The function stores all these in a u8 buffer. The PAYLOAD_LENGTH check seems to be trying to detect an overflow error. However, since we are just reading a u8 valu