diff options
| author | Davidlohr Bueso <davidlohr@hp.com> | 2014-01-27 17:07:06 -0800 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2014-01-27 21:02:39 -0800 |
| commit | 3ab08fe20475658bab65118d599d03cd8ca44dd1 (patch) | |
| tree | 25f4f43e58aad6a4c286a098f1f1f9d69452c30d /ipc/sem.c | |
| parent | 8001c85810dd2277d75ae60376e840456afa9b7e (diff) | |
| download | lwn-3ab08fe20475658bab65118d599d03cd8ca44dd1.tar.gz lwn-3ab08fe20475658bab65118d599d03cd8ca44dd1.zip | |
ipc: remove braces for single statements
Deal with checkpatch messages:
WARNING: braces {} are not necessary for single statement blocks
Signed-off-by: Davidlohr Bueso <davidlohr@hp.com>
Cc: Aswin Chandramouleeswaran <aswin@hp.com>
Cc: Rik van Riel <riel@redhat.com>
Acked-by: Manfred Spraul <manfred@colorfullife.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'ipc/sem.c')
| -rw-r--r-- | ipc/sem.c | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/ipc/sem.c b/ipc/sem.c index 1b1acdb0aee8..bee555417312 100644 --- a/ipc/sem.c +++ b/ipc/sem.c @@ -492,9 +492,9 @@ static int newary(struct ipc_namespace *ns, struct ipc_params *params) size = sizeof(*sma) + nsems * sizeof(struct sem); sma = ipc_rcu_alloc(size); - if (!sma) { + if (!sma) return -ENOMEM; - } + memset(sma, 0, size); sma->sem_perm.mode = (semflg & S_IRWXUGO); @@ -1967,10 +1967,8 @@ sleep_again: * If queue.status != -EINTR we are woken up by another process. * Leave without unlink_queue(), but with sem_unlock(). */ - - if (error != -EINTR) { + if (error != -EINTR) goto out_unlock_free; - } /* * If an interrupt occurred we have to clean up the queue |