public inbox for b.a.t.m.a.n@lists.open-mesh.org
 help / color / mirror / Atom feed
* [B.A.T.M.A.N.] [PATCH] batman-adv: orig_node must be freed after in case of timeout
@ 2011-03-04  1:13 Antonio Quartulli
  2011-03-04 21:33 ` Marek Lindner
  0 siblings, 1 reply; 2+ messages in thread
From: Antonio Quartulli @ 2011-03-04  1:13 UTC (permalink / raw)
  To: b.a.t.m.a.n; +Cc: Marek Lindner, Antonio Quartulli

This patch corrects commit
4c4236064501e85915087f00a370df8d9f43028d in which
bucket_free_orig_rcu() has been removed but without
being replaced.
In _push_orig(), the orig_node was not freed anymore after
a timeout.

Signed-off-by: Sven Eckelmann <sven@narfation.org>
Signed-off-by: Marek Lindner <lindner_marek@yahoo.de>
---
 originator.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/originator.c b/originator.c
index 53753d3..0b91330 100644
--- a/originator.c
+++ b/originator.c
@@ -349,6 +349,7 @@ static void _purge_orig(struct bat_priv *bat_priv)
 				if (orig_node->gw_flags)
 					gw_node_delete(bat_priv, orig_node);
 				hlist_del_rcu(node);
+				orig_node_free_ref(orig_node);
 				continue;
 			}
 
-- 
1.7.3.4


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

* Re: [B.A.T.M.A.N.] [PATCH] batman-adv: orig_node must be freed after in case of timeout
  2011-03-04  1:13 [B.A.T.M.A.N.] [PATCH] batman-adv: orig_node must be freed after in case of timeout Antonio Quartulli
@ 2011-03-04 21:33 ` Marek Lindner
  0 siblings, 0 replies; 2+ messages in thread
From: Marek Lindner @ 2011-03-04 21:33 UTC (permalink / raw)
  To: The list for a Better Approach To Mobile Ad-hoc Networking

On Friday 04 March 2011 02:13:16 Antonio Quartulli wrote:
> This patch corrects commit
> 4c4236064501e85915087f00a370df8d9f43028d in which
> bucket_free_orig_rcu() has been removed but without
> being replaced.
> In _push_orig(), the orig_node was not freed anymore after
> a timeout.

Applied in revision 1954.

Thanks,
Marek

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

end of thread, other threads:[~2011-03-04 21:33 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-03-04  1:13 [B.A.T.M.A.N.] [PATCH] batman-adv: orig_node must be freed after in case of timeout Antonio Quartulli
2011-03-04 21:33 ` Marek Lindner

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