[PATCH 2.4] PKT_SCHED: break is not enough to stop walking

STALE7953d

2 messages, 2 authors, 2004-11-02 · open the first message on its own page

[PATCH 2.4] PKT_SCHED: break is not enough to stop walking

From: Thomas Graf <tgraf@suug.ch>
Date: 2004-10-29 00:29:58

break is not enough to escape from the walking loops, since
multiple encapsulated loops are used to traverse the hash tables.

Signed-off-by: Thomas Graf <tgraf@suug.ch>
--- linux-2.4.28-rc1-bk1.orig/net/sched/cls_fw.c	2004-10-29 01:34:19.000000000 +0200
+++ linux-2.4.28-rc1-bk1/net/sched/cls_fw.c	2004-10-29 01:36:19.000000000 +0200
@@ -292,7 +292,7 @@
 			}
 			if (arg->fn(tp, (unsigned long)f, arg) < 0) {
 				arg->stop = 1;
-				break;
+				return;
 			}
 			arg->count++;
 		}
--- linux-2.4.28-rc1-bk1.orig/net/sched/cls_route.c	2004-10-29 01:34:19.000000000 +0200
+++ linux-2.4.28-rc1-bk1/net/sched/cls_route.c	2004-10-29 01:36:31.000000000 +0200
@@ -541,7 +541,7 @@
 					}
 					if (arg->fn(tp, (unsigned long)f, arg) < 0) {
 						arg->stop = 1;
-						break;
+						return;
 					}
 					arg->count++;
 				}
--- linux-2.4.28-rc1-bk1.orig/net/sched/cls_rsvp.h	2004-10-29 01:34:19.000000000 +0200
+++ linux-2.4.28-rc1-bk1/net/sched/cls_rsvp.h	2004-10-29 01:55:09.000000000 +0200
@@ -604,7 +604,7 @@
 					}
 					if (arg->fn(tp, (unsigned long)f, arg) < 0) {
 						arg->stop = 1;
-						break;
+						return;
 					}
 					arg->count++;
 				}

Re: [PATCH 2.4] PKT_SCHED: break is not enough to stop walking

From: "David S. Miller" <davem@davemloft.net>
Date: 2004-11-02 00:58:20

On Fri, 29 Oct 2004 02:29:58 +0200
Thomas Graf [off-list ref] wrote:
break is not enough to escape from the walking loops, since
multiple encapsulated loops are used to traverse the hash tables.

Signed-off-by: Thomas Graf <tgraf@suug.ch>
Applied, thanks for doing this backport Thomas.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help