All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] netlabel: correctly list all the static label mappings
@ 2013-03-06 21:45 Paul Moore
  2013-03-07 21:21 ` David Miller
  0 siblings, 1 reply; 3+ messages in thread
From: Paul Moore @ 2013-03-06 21:45 UTC (permalink / raw)
  To: netdev

When we have a large number of static label mappings that spill across
the netlink message boundary we fail to properly save our state in the
netlink_callback struct which causes us to repeat the same listings.
This patch fixes this problem by saving the state correctly between
calls to the NetLabel static label netlink "dumpit" routines.

Signed-off-by: Paul Moore <pmoore@redhat.com>
Cc: stable@vger.kernel.org
---
 net/netlabel/netlabel_unlabeled.c |   12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/net/netlabel/netlabel_unlabeled.c b/net/netlabel/netlabel_unlabeled.c
index 847d495..85742b7 100644
--- a/net/netlabel/netlabel_unlabeled.c
+++ b/net/netlabel/netlabel_unlabeled.c
@@ -1250,10 +1250,10 @@ static int netlbl_unlabel_staticlist(struct sk_buff *skb,
 
 unlabel_staticlist_return:
 	rcu_read_unlock();
-	cb->args[0] = skip_bkt;
-	cb->args[1] = skip_chain;
-	cb->args[2] = skip_addr4;
-	cb->args[3] = skip_addr6;
+	cb->args[0] = iter_bkt;
+	cb->args[1] = iter_chain;
+	cb->args[2] = iter_addr4;
+	cb->args[3] = iter_addr6;
 	return skb->len;
 }
 
@@ -1320,8 +1320,8 @@ static int netlbl_unlabel_staticlistdef(struct sk_buff *skb,
 
 unlabel_staticlistdef_return:
 	rcu_read_unlock();
-	cb->args[0] = skip_addr4;
-	cb->args[1] = skip_addr6;
+	cb->args[0] = iter_addr4;
+	cb->args[1] = iter_addr6;
 	return skb->len;
 }
 

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

* Re: [PATCH] netlabel: correctly list all the static label mappings
  2013-03-06 21:45 [PATCH] netlabel: correctly list all the static label mappings Paul Moore
@ 2013-03-07 21:21 ` David Miller
  2013-03-07 21:26   ` Paul Moore
  0 siblings, 1 reply; 3+ messages in thread
From: David Miller @ 2013-03-07 21:21 UTC (permalink / raw)
  To: pmoore; +Cc: netdev

From: Paul Moore <pmoore@redhat.com>
Date: Wed, 06 Mar 2013 16:45:24 -0500

> When we have a large number of static label mappings that spill across
> the netlink message boundary we fail to properly save our state in the
> netlink_callback struct which causes us to repeat the same listings.
> This patch fixes this problem by saving the state correctly between
> calls to the NetLabel static label netlink "dumpit" routines.
> 
> Signed-off-by: Paul Moore <pmoore@redhat.com>

Applied and queued up for -stable, thanks.

> Cc: stable@vger.kernel.org

Please just ask me to queue up networking fixes for stable, rather
than CC:'ing it to stable.

Thanks.

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

* Re: [PATCH] netlabel: correctly list all the static label mappings
  2013-03-07 21:21 ` David Miller
@ 2013-03-07 21:26   ` Paul Moore
  0 siblings, 0 replies; 3+ messages in thread
From: Paul Moore @ 2013-03-07 21:26 UTC (permalink / raw)
  To: David Miller; +Cc: netdev

On Thursday, March 07, 2013 04:21:19 PM David Miller wrote:
> From: Paul Moore <pmoore@redhat.com>
> Date: Wed, 06 Mar 2013 16:45:24 -0500
> 
> > When we have a large number of static label mappings that spill across
> > the netlink message boundary we fail to properly save our state in the
> > netlink_callback struct which causes us to repeat the same listings.
> > This patch fixes this problem by saving the state correctly between
> > calls to the NetLabel static label netlink "dumpit" routines.
> > 
> > Signed-off-by: Paul Moore <pmoore@redhat.com>
> 
> Applied and queued up for -stable, thanks.
> 
> > Cc: stable@vger.kernel.org
> 
> Please just ask me to queue up networking fixes for stable, rather
> than CC:'ing it to stable.

No problem, I'll do that in the future.

Thanks.

-- 
paul moore
security and virtualization @ redhat

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

end of thread, other threads:[~2013-03-07 21:26 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-03-06 21:45 [PATCH] netlabel: correctly list all the static label mappings Paul Moore
2013-03-07 21:21 ` David Miller
2013-03-07 21:26   ` Paul Moore

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.