Re: [PATCH 3/5] s390/pci: Move three assignments for the variable "rc" in clp_normal_command()

2017-01-21 Thread Christophe JAILLET
Le 21/01/2017 à 19:12, SF Markus Elfring a écrit : @@ -541,9 +543,8 @@ static int clp_normal_command(struct clp_req *req) if (rc) goto out_free; - rc = -EFAULT; if (copy_to_user(uptr, lpcb, PAGE_SIZE) != 0) - goto out_free; + rc = -E

Re: [PATCH 3/5] s390/pci: Move three assignments for the variable "rc" in clp_normal_command()

2017-01-21 Thread kbuild test robot
Hi Markus, [auto build test WARNING on s390/features] [also build test WARNING on v4.10-rc4 next-20170120] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/SF-Markus-Elfring/s390-pci-Fine-tuning-f

[PATCH 3/5] s390/pci: Move three assignments for the variable "rc" in clp_normal_command()

2017-01-21 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 21 Jan 2017 18:10:57 +0100 A local variable was set to an error code before a concrete error situation was detected. Thus move the corresponding assignments into if branches to indicate a software failure there. This issue was detected by using the Coccinelle soft