On Fri, Mar 5, 2021 at 3:05 AM Sergey Nazarov <[email protected]> wrote: > > We need to use put_unaligned when writing 32-bit DOI value > in cipso_v4_gentag_hdr to avoid unaligned memory access. > > v2: unneeded type cast removed as Ondrej Mosnacek suggested. > > Signed-off-by: Sergey Nazarov <[email protected]> > --- > net/ipv4/cipso_ipv4.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-)
Acked-by: Paul Moore <[email protected]> > diff --git a/net/ipv4/cipso_ipv4.c b/net/ipv4/cipso_ipv4.c > index 471d33a..6e59902 100644 > --- a/net/ipv4/cipso_ipv4.c > +++ b/net/ipv4/cipso_ipv4.c > @@ -1162,7 +1162,7 @@ static void cipso_v4_gentag_hdr(const struct > cipso_v4_doi *doi_def, > { > buf[0] = IPOPT_CIPSO; > buf[1] = CIPSO_V4_HDR_LEN + len; > - *(__be32 *)&buf[2] = htonl(doi_def->doi); > + put_unaligned_be32(doi_def->doi, &buf[2]); > } > > /** > -- > 1.8.3.1 -- paul moore www.paul-moore.com
