All of lore.kernel.org
 help / color / mirror / Atom feed
* Patch "rtnl: fix the loop index update error in rtnl_dump_ifinfo()" has been added to the 4.8-stable tree
@ 2016-12-08  6:21 gregkh
       [not found] ` <17a917ea-b053-b01e-7b66-7698aea58856@cumulusnetworks.com>
  0 siblings, 1 reply; 2+ messages in thread
From: gregkh @ 2016-12-08  6:21 UTC (permalink / raw)
  To: zhangshengju, davem, dsa, gregkh; +Cc: stable, stable-commits


This is a note to let you know that I've just added the patch titled

    rtnl: fix the loop index update error in rtnl_dump_ifinfo()

to the 4.8-stable tree which can be found at:
    http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary

The filename of the patch is:
     rtnl-fix-the-loop-index-update-error-in-rtnl_dump_ifinfo.patch
and it can be found in the queue-4.8 subdirectory.

If you, or anyone else, feels it should not be added to the stable tree,
please let <stable@vger.kernel.org> know about it.


>From foo@baz Thu Dec  8 07:19:12 CET 2016
From: Zhang Shengju <zhangshengju@cmss.chinamobile.com>
Date: Sat, 19 Nov 2016 23:28:32 +0800
Subject: rtnl: fix the loop index update error in rtnl_dump_ifinfo()

From: Zhang Shengju <zhangshengju@cmss.chinamobile.com>


[ Upstream commit 3f0ae05d6fea0ed5b19efdbc9c9f8e02685a3af3 ]

If the link is filtered out, loop index should also be updated. If not,
loop index will not be correct.

Fixes: dc599f76c22b0 ("net: Add support for filtering link dump by master device and kind")
Signed-off-by: Zhang Shengju <zhangshengju@cmss.chinamobile.com>
Acked-by: David Ahern <dsa@cumulusnetworks.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
---
 net/core/rtnetlink.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

--- a/net/core/rtnetlink.c
+++ b/net/core/rtnetlink.c
@@ -1578,7 +1578,7 @@ static int rtnl_dump_ifinfo(struct sk_bu
 		head = &net->dev_index_head[h];
 		hlist_for_each_entry(dev, head, index_hlist) {
 			if (link_dump_filtered(dev, master_idx, kind_ops))
-				continue;
+				goto cont;
 			if (idx < s_idx)
 				goto cont;
 			err = rtnl_fill_ifinfo(skb, dev, RTM_NEWLINK,


Patches currently in stable-queue which might be from zhangshengju@cmss.chinamobile.com are

queue-4.8/rtnl-fix-the-loop-index-update-error-in-rtnl_dump_ifinfo.patch

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

* Re: Patch "rtnl: fix the loop index update error in rtnl_dump_ifinfo()" has been added to the 4.8-stable tree
       [not found] ` <17a917ea-b053-b01e-7b66-7698aea58856@cumulusnetworks.com>
@ 2016-12-08 10:41   ` Greg KH
  0 siblings, 0 replies; 2+ messages in thread
From: Greg KH @ 2016-12-08 10:41 UTC (permalink / raw)
  To: David Ahern; +Cc: zhangshengju, davem, stable, stable-commits

On Wed, Dec 07, 2016 at 10:27:51PM -0800, David Ahern wrote:
> On 12/7/16 10:21 PM, gregkh@linuxfoundation.org wrote:
> > 
> > This is a note to let you know that I've just added the patch titled
> > 
> >     rtnl: fix the loop index update error in rtnl_dump_ifinfo()
> > 
> > to the 4.8-stable tree which can be found at:
> >     http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary
> > 
> > The filename of the patch is:
> >      rtnl-fix-the-loop-index-update-error-in-rtnl_dump_ifinfo.patch
> > and it can be found in the queue-4.8 subdirectory.
> > 
> > If you, or anyone else, feels it should not be added to the stable tree,
> > please let <stable@vger.kernel.org> know about it.
> 
> please drop this patch for 4.8 and all other stable backports. It is a no-op patch.

If it is a no-op, why is it in Linus's tree now too?  Should it be
reverted there?

confused,

greg k-h

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

end of thread, other threads:[~2016-12-08 10:41 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-12-08  6:21 Patch "rtnl: fix the loop index update error in rtnl_dump_ifinfo()" has been added to the 4.8-stable tree gregkh
     [not found] ` <17a917ea-b053-b01e-7b66-7698aea58856@cumulusnetworks.com>
2016-12-08 10:41   ` Greg KH

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.