Quentin LAMBERT
Author's posts
Nov 22 2014
HSI: nokia-modem: fix error return code
Author: Julia Lawall <Julia.Lawall@lip6.fr> Return a negative error code on failure. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // @@ identifier ret; expression e1,e2; @@ ( if (\(ret < 0\|ret != 0\)) { ... return ret; } | ret = 0 ) ... when != ret = …
Nov 22 2014
s390/pci: Deletion of unnecessary checks before the function call “debug_unregister”
Author: Markus Elfring <elfring@users.sourceforge.net> The debug_unregister() function performs also input parameter validation. Thus the test around the call is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring Signed-off-by: Sebastian Ott Signed-off-by: Martin Schwidefsky — arch/s390/pci/pci_debug.c | 7 ++—– 1 file changed, 2 insertions(+), 5 deletions(-) diff –git a/arch/s390/pci/pci_debug.c …
Nov 21 2014
IDE: Deletion of an unnecessary check before the function call “module_put”
Author: Markus Elfring <elfring@users.sourceforge.net> The module_put() function tests whether its argument is NULL and then returns immediately. Thus the test around the call is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring Signed-off-by: David S. Miller — drivers/ide/ide.c | 3 +– 1 file changed, 1 insertion(+), 2 deletions(-) …
Nov 21 2014
ALSA: sb: Deletion of unnecessary checks before two function calls
Author: Markus Elfring <elfring@users.sourceforge.net> The functions release_and_free_resource() and snd_util_memhdr_free() test whether their argument is NULL and then return immediately. Thus the test around the call is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring Signed-off-by: Takashi Iwai — sound/isa/sb/emu8000_synth.c | 3 +– sound/isa/sb/sb_common.c | 3 +– 2 files changed, …
Nov 21 2014
ALSA: es1688_lib: Deletion of an unnecessary check before the function call “release_and_free_resource”
Author: Markus Elfring <elfring@users.sourceforge.net> The release_and_free_resource() function tests whether its argument is NULL and then returns immediately. Thus the test around the call is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring Signed-off-by: Takashi Iwai — sound/isa/es1688/es1688_lib.c | 3 +– 1 file changed, 1 insertion(+), 2 deletions(-) diff …
Nov 21 2014
ALSA: core: Deletion of unnecessary checks before two function calls
Author: Markus Elfring <elfring@users.sourceforge.net> The functions snd_seq_oss_timer_delete() and vunmap() perform also input parameter validation. Thus the test around the call is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring Signed-off-by: Takashi Iwai — sound/core/seq/oss/seq_oss_init.c | 9 +++—— sound/core/sgbuf.c | 3 +– 2 files changed, 4 insertions(+), 8 deletions(-) …
Nov 21 2014
thermal: Exynos: Deletion of unnecessary checks before two function calls
Author: Markus Elfring <elfring@users.sourceforge.net> The functions cpufreq_cooling_unregister() and thermal_zone_device_unregister() test whether their argument is NULL and then return immediately. Thus the test around the call is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring Signed-off-by: Eduardo Valentin — drivers/thermal/samsung/exynos_thermal_common.c | 9 +++—— 1 file changed, 3 insertions(+), 6 deletions(-) …
Nov 21 2014
USB-IP: Deletion of unnecessary checks before the function call “usb_put_dev”
Author: Markus Elfring <elfring@users.sourceforge.net> The usb_put_dev() function tests whether its argument is NULL and then returns immediately. Thus the test around the call is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring Acked-by: Valentina Manea Signed-off-by: Greg Kroah-Hartman — drivers/usb/usbip/vhci_hcd.c | 9 +++—— 1 file changed, 3 insertions(+), 6 …
Nov 21 2014
USB-SIS: Deletion of an unnecessary check before the function call “usb_put_dev”
Author: Markus Elfring <elfring@users.sourceforge.net> The usb_put_dev() function tests whether its argument is NULL and then returns immediately. Thus the test around the call is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring Signed-off-by: Greg Kroah-Hartman — drivers/usb/misc/sisusbvga/sisusb.c | 3 +– 1 file changed, 1 insertion(+), 2 deletions(-) diff …
Nov 21 2014
USB: PCI-quirks: Deletion of unnecessary checks before the function call “pci_dev_put”
Author: Markus Elfring <elfring@users.sourceforge.net> The pci_dev_put() function tests whether its argument is NULL and then returns immediately. Thus the test around the call is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring Signed-off-by: Greg Kroah-Hartman — drivers/usb/host/pci-quirks.c | 12 ++++——– 1 file changed, 4 insertions(+), 8 deletions(-) diff …