Re: [PATCH v2] platform/surface: aggregator: Fix access of unaligned value

2021-02-11 Thread Hans de Goede
Hi, On 2/11/21 1:41 PM, Maximilian Luz wrote: > The raw message frame length is unaligned and explicitly marked as > little endian. It should not be accessed without the appropriate > accessor functions. Fix this. > > Note that payload.len already contains the correct length after parsing > via

Re: [PATCH v2] platform/surface: aggregator: Fix access of unaligned value

2021-02-11 Thread Andy Shevchenko
On Thu, Feb 11, 2021 at 01:41:49PM +0100, Maximilian Luz wrote: > The raw message frame length is unaligned and explicitly marked as > little endian. It should not be accessed without the appropriate > accessor functions. Fix this. > > Note that payload.len already contains the correct length

[PATCH v2] platform/surface: aggregator: Fix access of unaligned value

2021-02-11 Thread Maximilian Luz
The raw message frame length is unaligned and explicitly marked as little endian. It should not be accessed without the appropriate accessor functions. Fix this. Note that payload.len already contains the correct length after parsing via sshp_parse_frame(), so we can simply use that instead.