xfrm: Fix unaligned access to stats in copy_to_user_state()
authorSowmini Varadhan <sowmini.varadhan@oracle.com>
Wed, 21 Oct 2015 15:48:25 +0000 (11:48 -0400)
committerSteffen Klassert <steffen.klassert@secunet.com>
Fri, 23 Oct 2015 04:49:29 +0000 (06:49 +0200)
On sparc, deleting established SAs (e.g., by restarting ipsec)
results in unaligned access messages via xfrm_del_sa ->
km_state_notify -> xfrm_send_state_notify().

Even though struct xfrm_usersa_info is aligned on 8-byte boundaries,
netlink attributes are fundamentally only 4 byte aligned, and this
cannot be changed for nla_data() that is passed up to userspace.
As a result, the put_unaligned() macro needs to be used to
set up potentially unaligned fields such as the xfrm_stats in
copy_to_user_state()

Signed-off-by: Sowmini Varadhan <sowmini.varadhan@oracle.com>
Signed-off-by: Steffen Klassert <steffen.klassert@secunet.com>
net/xfrm/xfrm_user.c

index a8de9e3002000d7eaa76f6764797e5b231d187ff..639e0d51b31ad804cf72a0fd3caa43a55426c433 100644 (file)
@@ -31,6 +31,7 @@
 #if IS_ENABLED(CONFIG_IPV6)
 #include <linux/in6.h>
 #endif
+#include <asm/unaligned.h>
 
 static int verify_one_alg(struct nlattr **attrs, enum xfrm_attr_type_t type)
 {
@@ -728,7 +729,9 @@ static void copy_to_user_state(struct xfrm_state *x, struct xfrm_usersa_info *p)
        memcpy(&p->sel, &x->sel, sizeof(p->sel));
        memcpy(&p->lft, &x->lft, sizeof(p->lft));
        memcpy(&p->curlft, &x->curlft, sizeof(p->curlft));
-       memcpy(&p->stats, &x->stats, sizeof(p->stats));
+       put_unaligned(x->stats.replay_window, &p->stats.replay_window);
+       put_unaligned(x->stats.replay, &p->stats.replay);
+       put_unaligned(x->stats.integrity_failed, &p->stats.integrity_failed);
        memcpy(&p->saddr, &x->props.saddr, sizeof(p->saddr));
        p->mode = x->props.mode;
        p->replay_window = x->props.replay_window;