Author: Julia Lawall <Julia.Lawall@lip6.fr> Convert a zero return value on error to a negative one, as returned elsewhere in the function. 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; } …
Catégorie : Linux
Aug 07 2014
netfilter: nf_tables: fix error return code
Author: Julia Lawall <Julia.Lawall@lip6.fr> Convert a zero return value on error to a negative one, as returned elsewhere in the function. 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; } …
Aug 07 2014
solos-pci: fix error return code
Author: Julia Lawall <Julia.Lawall@lip6.fr> Convert a zero return value on error to a negative one, as returned elsewhere in the function. 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; } …
Aug 07 2014
avr32: fix error return code
Author: Julia Lawall <Julia.Lawall@lip6.fr> Convert a zero return value on error to a negative one, as returned elsewhere in the function. 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; } …
Aug 07 2014
brcmfmac: fix memory leakage in msgbuf
Author: Arend van Spriel <arend@broadcom.com> The kbuild robot came up with the following warning: tree: …/kernel/git/linville/wireless-next.git master head: dc6be9f54a4ecb0a09765d1f515ed947d86b7528 commit: 9a1bb60250d2b6b546a62e5b73f55c4f1d22016b [5/13] brcmfmac: Adding msgbuf protocol. coccinelle warnings: drivers/net/wireless/brcm80211/brcmfmac/msgbuf.c:1309:1-28: alloc with no test, possible model on line 1318 Looking into the issue, it turned out that the referred allocation buffer was not being released in …
Aug 07 2014
kernel/printk/printk.c: fix bool assignements
Author: Neil Zhang <zhangwm@marvell.com> Fix coccinelle warnings. Signed-off-by: Neil Zhang Signed-off-by: Andrew Morton Signed-off-by: Linus Torvalds — kernel/printk/printk.c | 6 +++— 1 file changed, 3 insertions(+), 3 deletions(-) diff –git a/kernel/printk/printk.c b/kernel/printk/printk.c index df202fe..de1a6bb 100644 — a/kernel/printk/printk.c +++ b/kernel/printk/printk.c @@ -919,7 +919,7 @@ static bool __read_mostly ignore_loglevel; static int __init ignore_loglevel_setup(char *str) { …
Aug 07 2014
drivers: video: fbdev: atmel_lcdfb.c: fix error return code
Author: Julia Lawall <Julia.Lawall@lip6.fr> Convert a zero return value on error to a negative one, as returned elsewhere in the function. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // ( if@p1 (\(ret < 0\|ret != 0\)) { ... return ret; } | ret@p1 = 0 ) ... …
Aug 07 2014
mfd: twl6040: Fix error return code
Author: Julia Lawall <Julia.Lawall@lip6.fr> Convert a zero return value on error to a negative one, as returned elsewhere in the function. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // ( if@p1 (\(ret < 0\|ret != 0\)) { ... return ret; } | ret@p1 = 0 ) ... …
Aug 06 2014
pcmcia/vrc4171: Remove typedefs for enums and struct
Author: Himangi Saraogi <himangi774@gmail.com> The Linux kernel coding style guidelines suggest not using typedefs for structure and enum types. This patch gets rid of the typedefs for vrc4171_slot_t, vrc4171_slotb_t and vrc4171_socket_t. Also, the names of the enums and the struct are changed to drop the _t, to make the name look less typedef-like. The following …
Aug 06 2014
pcmcia: Remove typedef in structs and emum
Author: Himangi Saraogi <himangi774@gmail.com> The Linux kernel coding style guidelines suggest not using typedefs for structure and enum types. This patch gets rid of the typedefs for cirrus_state_t, vg46x_state_t and pcic_id. Also, the names of the structs are changed to drop the _t, to make the name look less typedef-like. The following Coccinelle semantic patch …