From: Roel Kluin <[EMAIL PROTECTED]>
Date: Wed, 24 Oct 2007 12:19:52 +0200
> Several unlocking issues
> Signed-off-by: Roel Kluin <[EMAIL PROTECTED]>
Some of these are wrong, for example:
> diff --git a/net/bridge/netfilter/ebtables.c b/net/bridge/netfilter/ebtables.c
> index 817169e..b09c499 100644
> --- a/net/bridge/netfilter/ebtables.c
> +++ b/net/bridge/netfilter/ebtables.c
> @@ -282,8 +282,10 @@ find_inlist_lock_noload(struct list_head *head, const
> char *name, int *error,
> return NULL;
>
> list_for_each_entry(e, head, list) {
> - if (strcmp(e->name, name) == 0)
> + if (strcmp(e->name, name) == 0) {
> + mutex_unlock(mutex);
> return e;
> + }
> }
> *error = -ENOENT;
> mutex_unlock(mutex);
Please look at the comment right about this function, it clearly
explains that if the named object is found, the function returns
with the mutex locked.
The rest of the networking cases look OK, please just resubmit
the correct networking cases to [EMAIL PROTECTED]
Thanks.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
