diff options
| author | Julian Wiedmann <jwi@linux.ibm.com> | 2020-10-01 19:21:27 +0200 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2020-10-04 01:51:07 +0200 |
| commit | a29f245ec97f83f697c7442e0f1222f2ac7c9507 (patch) | |
| tree | 01878b642fd4ff59a6e865238b1bb4d934e34bbe | |
| parent | net/af_iucv: right-size the uid variable in iucv_sock_bind() (diff) | |
| download | linux-a29f245ec97f83f697c7442e0f1222f2ac7c9507.tar.xz linux-a29f245ec97f83f697c7442e0f1222f2ac7c9507.zip | |
net/iucv: fix indentation in __iucv_message_receive()
smatch complains about
net/iucv/iucv.c:1119 __iucv_message_receive() warn: inconsistent indenting
While touching this line, also make the return logic consistent and thus
get rid of a goto label.
Signed-off-by: Julian Wiedmann <jwi@linux.ibm.com>
Reviewed-by: Karsten Graul <kgraul@linux.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
| -rw-r--r-- | net/iucv/iucv.c | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/net/iucv/iucv.c b/net/iucv/iucv.c index cd2e468852e7..349c6ac3313f 100644 --- a/net/iucv/iucv.c +++ b/net/iucv/iucv.c @@ -1116,10 +1116,9 @@ int __iucv_message_receive(struct iucv_path *path, struct iucv_message *msg, if (msg->flags & IUCV_IPRMDATA) return iucv_message_receive_iprmdata(path, msg, flags, buffer, size, residual); - if (cpumask_empty(&iucv_buffer_cpumask)) { - rc = -EIO; - goto out; - } + if (cpumask_empty(&iucv_buffer_cpumask)) + return -EIO; + parm = iucv_param[smp_processor_id()]; memset(parm, 0, sizeof(union iucv_param)); parm->db.ipbfadr1 = (u32)(addr_t) buffer; @@ -1135,7 +1134,6 @@ int __iucv_message_receive(struct iucv_path *path, struct iucv_message *msg, if (residual) *residual = parm->db.ipbfln1f; } -out: return rc; } EXPORT_SYMBOL(__iucv_message_receive); |