xfrm_user: fix info leak in copy_to_user_state()
authorMathias Krause <minipli@googlemail.com>
Wed, 19 Sep 2012 11:33:39 +0000 (11:33 +0000)
committerBen Hutchings <ben@decadent.org.uk>
Wed, 10 Oct 2012 02:31:23 +0000 (03:31 +0100)
commite470a5bc5c80cad5d8877701784351c2d8cdb6bc
treee07ae09411d8d2e0ac72493d0161895b9dfdc0fc
parent744e0a9c51333d712e76850bf58b4aeb277016fe
xfrm_user: fix info leak in copy_to_user_state()

[ Upstream commit f778a636713a435d3a922c60b1622a91136560c1 ]

The memory reserved to dump the xfrm state includes the padding bytes of
struct xfrm_usersa_info added by the compiler for alignment (7 for
amd64, 3 for i386). Add an explicit memset(0) before filling the buffer
to avoid the info leak.

Signed-off-by: Mathias Krause <minipli@googlemail.com>
Acked-by: Steffen Klassert <steffen.klassert@secunet.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
net/xfrm/xfrm_user.c