From: Julia Lawall Date: Wed, 26 May 2010 21:43:44 +0000 (-0700) Subject: ipc/sem.c: use ERR_CAST X-Git-Url: https://git.stricted.de/?a=commitdiff_plain;h=4de85cd6d6018825e19f76f1208775f23ecc393f;p=GitHub%2Fmt8127%2Fandroid_kernel_alcatel_ttab.git ipc/sem.c: use ERR_CAST Use ERR_CAST(x) rather than ERR_PTR(PTR_ERR(x)). The former makes more clear what is the purpose of the operation, which otherwise looks like a no-op. The semantic patch that makes this change is as follows: (http://coccinelle.lip6.fr/) // @@ type T; T x; identifier f; @@ T f (...) { <+... - ERR_PTR(PTR_ERR(x)) + x ...+> } @@ expression x; @@ - ERR_PTR(PTR_ERR(x)) + ERR_CAST(x) // Signed-off-by: Julia Lawall Cc: Manfred Spraul Signed-off-by: Andrew Morton Signed-off-by: Linus Torvalds --- diff --git a/ipc/sem.c b/ipc/sem.c index 5b33228db798..506c8491a8d1 100644 --- a/ipc/sem.c +++ b/ipc/sem.c @@ -1208,7 +1208,7 @@ static struct sem_undo *find_alloc_undo(struct ipc_namespace *ns, int semid) /* step 1: figure out the size of the semaphore array */ sma = sem_lock_check(ns, semid); if (IS_ERR(sma)) - return ERR_PTR(PTR_ERR(sma)); + return ERR_CAST(sma); nsems = sma->sem_nsems; sem_getref_and_unlock(sma);