diff options
author | Felipe Contreras <felipe.contreras@gmail.com> | 2009-10-19 01:54:29 +0300 |
---|---|---|
committer | Jiri Kosina <jkosina@suse.cz> | 2009-12-04 15:39:49 +0100 |
commit | f1970c48ef06ece4e23765501976507ab52b0edd (patch) | |
tree | b8db16b9b697a3cc33f69ceaf7998c4ad3576e9c /ipc | |
parent | e9de25dda359c9272998daddeae7788376a44e41 (diff) | |
download | lwn-f1970c48ef06ece4e23765501976507ab52b0edd.tar.gz lwn-f1970c48ef06ece4e23765501976507ab52b0edd.zip |
ipc: fix unused variable warning
Commit a0d092f introduced the following warning:
ipc/msg.c: In function ?msgctl_down?:
ipc/msg.c:415: warning: ?msqid64? may be used uninitialized in this function
The gcc warning in this case is actually bogus, as msqid64 is touched only
iff cmd == IPC_SET, and in such case, copy_msqid_from_user() initializes
it properly.
Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
Diffstat (limited to 'ipc')
-rw-r--r-- | ipc/msg.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ipc/msg.c b/ipc/msg.c index 2ceab7f12fcb..085bd58f2f07 100644 --- a/ipc/msg.c +++ b/ipc/msg.c @@ -412,7 +412,7 @@ static int msgctl_down(struct ipc_namespace *ns, int msqid, int cmd, struct msqid_ds __user *buf, int version) { struct kern_ipc_perm *ipcp; - struct msqid64_ds msqid64; + struct msqid64_ds uninitialized_var(msqid64); struct msg_queue *msq; int err; |