public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH 2.6.27-rc9] af_key: fix SADB_X_SPDDELETE response
@ 2008-10-09 12:10 Tobias Brunner
  2008-10-09 21:44 ` David Miller
  0 siblings, 1 reply; 4+ messages in thread
From: Tobias Brunner @ 2008-10-09 12:10 UTC (permalink / raw)
  To: David S. Miller; +Cc: linux-kernel

When deleting an SPD entry using SADB_X_SPDDELETE, c.data.byid is not 
initialized to zero in pfkey_spddelete(). Thus, key_notify_policy() 
responds with a PF_KEY message of type SADB_X_SPDDELETE2 instead of 
SADB_X_SPDDELETE.

Signed-off-by: Tobias Brunner <tobias.brunner@strongswan.org>

--- a/net/key/af_key.c	2008-10-09 13:10:04.000000000 +0200
+++ b/net/key/af_key.c	2008-10-09 13:49:11.000000000 +0200
@@ -2317,6 +2317,7 @@
 
 	c.seq = hdr->sadb_msg_seq;
 	c.pid = hdr->sadb_msg_pid;
+	c.data.byid = 0;
 	c.event = XFRM_MSG_DELPOLICY;
 	km_policy_notify(xp, pol->sadb_x_policy_dir-1, &c);
 


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH 2.6.27-rc9] af_key: fix SADB_X_SPDDELETE response
  2008-10-09 12:10 [PATCH 2.6.27-rc9] af_key: fix SADB_X_SPDDELETE response Tobias Brunner
@ 2008-10-09 21:44 ` David Miller
  2008-10-10  8:23   ` Tobias Brunner
  0 siblings, 1 reply; 4+ messages in thread
From: David Miller @ 2008-10-09 21:44 UTC (permalink / raw)
  To: tobias.brunner; +Cc: linux-kernel

From: Tobias Brunner <tobias.brunner@strongswan.org>
Date: Thu, 09 Oct 2008 14:10:21 +0200

> When deleting an SPD entry using SADB_X_SPDDELETE, c.data.byid is not initialized to zero in pfkey_spddelete(). Thus, key_notify_policy() responds with a PF_KEY message of type SADB_X_SPDDELETE2 instead of SADB_X_SPDDELETE.
> 
> Signed-off-by: Tobias Brunner <tobias.brunner@strongswan.org>

This patch does not apply cleanly to net-next-2.6, a lot has
changed in this area over the past several weeks.

Please respin this so that I can add it, thanks.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH 2.6.27-rc9] af_key: fix SADB_X_SPDDELETE response
  2008-10-09 21:44 ` David Miller
@ 2008-10-10  8:23   ` Tobias Brunner
  2008-10-10 16:30     ` David Miller
  0 siblings, 1 reply; 4+ messages in thread
From: Tobias Brunner @ 2008-10-10  8:23 UTC (permalink / raw)
  To: David Miller; +Cc: linux-kernel

David Miller wrote:
> This patch does not apply cleanly to net-next-2.6, a lot has
> changed in this area over the past several weeks.
> 
> Please respin this so that I can add it, thanks.

Please find the patch for net-next-2.6 below.

Thanks,
	Tobias

---

diff --git a/net/key/af_key.c b/net/key/af_key.c
index 362fe31..e55e044 100644
--- a/net/key/af_key.c
+++ b/net/key/af_key.c
@@ -2341,6 +2341,7 @@ static int pfkey_spddelete(struct sock *sk, struct sk_buff *skb, struct sadb_msg
 
 	c.seq = hdr->sadb_msg_seq;
 	c.pid = hdr->sadb_msg_pid;
+	c.data.byid = 0;
 	c.event = XFRM_MSG_DELPOLICY;
 	km_policy_notify(xp, pol->sadb_x_policy_dir-1, &c);
 

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH 2.6.27-rc9] af_key: fix SADB_X_SPDDELETE response
  2008-10-10  8:23   ` Tobias Brunner
@ 2008-10-10 16:30     ` David Miller
  0 siblings, 0 replies; 4+ messages in thread
From: David Miller @ 2008-10-10 16:30 UTC (permalink / raw)
  To: tobias.brunner; +Cc: linux-kernel

From: Tobias Brunner <tobias.brunner@strongswan.org>
Date: Fri, 10 Oct 2008 10:23:36 +0200

> David Miller wrote:
> > This patch does not apply cleanly to net-next-2.6, a lot has
> > changed in this area over the past several weeks.
> > Please respin this so that I can add it, thanks.
> 
> Please find the patch for net-next-2.6 below.

Please resubmit:

1) At least CC:'d to the correct mailing list, netdev@vger.kernel.org

   This is important so that the patch gets tracked at:

	http://patchwork.ozlabs.org/project/netdev/list/

2) With full commit log message and signoffs presented again.

Thanks.

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2008-10-10 16:31 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-10-09 12:10 [PATCH 2.6.27-rc9] af_key: fix SADB_X_SPDDELETE response Tobias Brunner
2008-10-09 21:44 ` David Miller
2008-10-10  8:23   ` Tobias Brunner
2008-10-10 16:30     ` David Miller

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox