Pavel Emelianov wrote: > Patrick McHardy wrote: > >>>There is a race between netlink_dump_start() and netlink_release() >>>that can lead to the situation when a netlink socket with non-zero >>>callback is freed. >> >> >>Can you describe the race in more detail please? >> > > Here it is: > > [...] > The proposal it to make sock_orphan before detaching the callback > in netlink_release() and to check for the sock to be SOCK_DEAD in > netlink_dump_start() before setting a new callback.
Thanks, good catch. Your patch also looks good. Acked-by: Patrick McHardy <[EMAIL PROTECTED]> - 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