Arnaldo Carvalho de Melo wrote:
Em Tue, Oct 02, 2007 at 03:10:09PM +0200, Urs Thuermann escreveu:
+
+ if (ro->bound && ro->ifindex)
+ dev = dev_get_by_index(&init_net, ro->ifindex);
dev_get_by_index can fail, are you sure that raw_enable_filters can cope
with this possibility?
When dev_get_by_index() fails, the netdev is not there anymore.
In this case the netdev notifier handler already set ro->bound to 0.
Additionally raw_enable_filters() can handle *dev even when it's NULL.
Oliver
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
