* [PATCH] af_key: fix info leaks in notify messages
@ 2013-06-26 21:52 Mathias Krause
2013-06-26 22:16 ` David Miller
0 siblings, 1 reply; 2+ messages in thread
From: Mathias Krause @ 2013-06-26 21:52 UTC (permalink / raw)
To: Steffen Klassert, David S. Miller; +Cc: Mathias Krause, netdev, Herbert Xu
key_notify_sa_flush() and key_notify_policy_flush() miss to initialize
the sadb_msg_reserved member of the broadcasted message and thereby
leak 2 bytes of heap memory to listeners. Fix that.
Signed-off-by: Mathias Krause <minipli@googlemail.com>
Cc: Steffen Klassert <steffen.klassert@secunet.com>
Cc: "David S. Miller" <davem@davemloft.net>
Cc: Herbert Xu <herbert@gondor.apana.org.au>
---
net/key/af_key.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/net/key/af_key.c b/net/key/af_key.c
index c5fbd75..9da8620 100644
--- a/net/key/af_key.c
+++ b/net/key/af_key.c
@@ -1710,6 +1710,7 @@ static int key_notify_sa_flush(const struct km_event *c)
hdr->sadb_msg_version = PF_KEY_V2;
hdr->sadb_msg_errno = (uint8_t) 0;
hdr->sadb_msg_len = (sizeof(struct sadb_msg) / sizeof(uint64_t));
+ hdr->sadb_msg_reserved = 0;
pfkey_broadcast(skb, GFP_ATOMIC, BROADCAST_ALL, NULL, c->net);
@@ -2699,6 +2700,7 @@ static int key_notify_policy_flush(const struct km_event *c)
hdr->sadb_msg_errno = (uint8_t) 0;
hdr->sadb_msg_satype = SADB_SATYPE_UNSPEC;
hdr->sadb_msg_len = (sizeof(struct sadb_msg) / sizeof(uint64_t));
+ hdr->sadb_msg_reserved = 0;
pfkey_broadcast(skb_out, GFP_ATOMIC, BROADCAST_ALL, NULL, c->net);
return 0;
--
1.7.10.4
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] af_key: fix info leaks in notify messages
2013-06-26 21:52 [PATCH] af_key: fix info leaks in notify messages Mathias Krause
@ 2013-06-26 22:16 ` David Miller
0 siblings, 0 replies; 2+ messages in thread
From: David Miller @ 2013-06-26 22:16 UTC (permalink / raw)
To: minipli; +Cc: steffen.klassert, netdev, herbert
From: Mathias Krause <minipli@googlemail.com>
Date: Wed, 26 Jun 2013 23:52:30 +0200
> key_notify_sa_flush() and key_notify_policy_flush() miss to initialize
> the sadb_msg_reserved member of the broadcasted message and thereby
> leak 2 bytes of heap memory to listeners. Fix that.
>
> Signed-off-by: Mathias Krause <minipli@googlemail.com>
Applied and queued up for -stable, thanks!
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2013-06-26 22:16 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-06-26 21:52 [PATCH] af_key: fix info leaks in notify messages Mathias Krause
2013-06-26 22:16 ` David Miller
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.