Quentin LAMBERT
Author's posts
Jul 27 2009
fs/xfs: Correct redundant test
Author: Julia Lawall <julia@diku.dk> bp was tested for NULL a few lines before, followed by a return, and there is no intervening modification of its value. A simplified version of the semantic match that finds this problem is as follows: (http://www.emn.fr/x-info/coccinelle/) // @r exists@ local idexpression x; expression E; position p1,p2; @@ if (x == …
Jul 27 2009
fs/xfs: Correct redundant test
Author: Julia Lawall <julia@diku.dk> bp was tested for NULL a few lines before, followed by a return, and there is no intervening modification of its value. A simplified version of the semantic match that finds this problem is as follows: (http://www.emn.fr/x-info/coccinelle/) // @r exists@ local idexpression x; expression E; position p1,p2; @@ if (x == …
Jul 27 2009
net/netlabel: Correct redundant test
Author: Julia Lawall <julia@diku.dk> entry was tested for NULL near the beginning of the function, followed by a return, and there is no intervening modification of its value. A simplified version of the semantic match that finds this problem is as follows: (http://www.emn.fr/x-info/coccinelle/) // @r exists@ local idexpression x; expression E; position p1,p2; @@ if …
Jul 27 2009
drivers/net: Correct redundant test
Author: Julia Lawall <julia@diku.dk> res has already been tested. It seems that this test should be on the recently returned value mmio. A simplified version of the semantic match that finds this problem is as follows: (http://www.emn.fr/x-info/coccinelle/) // @r exists@ local idexpression x; expression E; @@ if (x == NULL || …) { … when …
Jul 24 2009
ISDN: ARRAY_SIZE changes
Author: Karsten Keil <keil@b1-systems.de> These changes were a direct result of using a semantic patch More information can be found at http://www.emn.fr/x-info/coccinelle/ Modified some of the changes to avoid the extra define. Signed-off-by: Stoyan Gaydarov Signed-off-by: Karsten Keil — drivers/isdn/act2000/capi.c | 3 +– drivers/isdn/act2000/module.c | 31 +++++++++++++++—————- drivers/isdn/hardware/eicon/message.c | 4 +— drivers/isdn/hardware/eicon/os_4bri.c | 3 +– …
Jul 22 2009
Staging: hv: Transform some kzalloc calls to kcalloc
Author: Nicolas Palix <npalix@diku.dk> Here is a short excerpt of the semantic patch performing this transformation: (http://www.emn.fr/x-info/coccinelle/) // @@ expression E1,E2,E3; @@ – kzalloc(E1*sizeof(E2),E3) + kcalloc(E1,sizeof(E2),E3) @@ expression E1,E3; type T; @@ – kzalloc(E1*sizeof(T),E3) + kcalloc(E1,sizeof(T),E3) // Signed-off-by: Nicolas Palix Cc: Hank Janssen Cc: Haiyang Zhang Signed-off-by: Greg Kroah-Hartman — drivers/staging/hv/storvsc_drv.c | 6 +++— 1 …
Jul 22 2009
ARRAY_SIZE changes
Author: Stoyan Gaydarov <sgayda2@uiuc.edu> These changes were a direct result of using a semantic patch More information can be found at http://www.emn.fr/x-info/coccinelle/ Signed-off-by: Stoyan Gaydarov Signed-off-by: Jesper Nilsson — arch/cris/arch-v32/mach-a3/io.c | 2 +- arch/cris/arch-v32/mach-fs/io.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff –git a/arch/cris/arch-v32/mach-a3/io.c b/arch/cris/arch-v32/mach-a3/io.c index c22f67e..090ceb9 100644 — a/arch/cris/arch-v32/mach-a3/io.c +++ …
Jul 22 2009
V4L/DVB (12478): ARRAY_SIZE changes
Author: Mauro Carvalho Chehab <mchehab@redhat.com> These changes were a direct result of using a semantic patch More information can be found at http://www.emn.fr/x-info/coccinelle/ [mchehab@redhat.com: fix a merge conflict] Signed-off-by: Stoyan Gaydarov Signed-off-by: Mauro Carvalho Chehab — drivers/media/video/gspca/conex.c | 2 +- drivers/media/video/gspca/etoms.c | 4 ++– drivers/media/video/gspca/spca501.c | 2 +- drivers/media/video/gspca/spca506.c | 2 +- drivers/media/video/gspca/sunplus.c | 4 …
Jul 22 2009
Staging: ARRAY_SIZE changes
Author: Stoyan Gaydarov <sgayda2@uiuc.edu> These changes were a direct result of using a semantic patch More information can be found at http://www.emn.fr/x-info/coccinelle/ Signed-off-by: Stoyan Gaydarov Signed-off-by: Greg Kroah-Hartman — drivers/staging/comedi/drivers/amplc_pci230.c | 2 +- drivers/staging/comedi/drivers/cb_das16_cs.c | 2 +- drivers/staging/comedi/drivers/das08_cs.c | 3 +– drivers/staging/comedi/drivers/ni_6527.c | 2 +- drivers/staging/comedi/drivers/ni_65xx.c | 2 +- drivers/staging/comedi/drivers/ni_660x.c | 2 +- drivers/staging/comedi/drivers/ni_670x.c | …
Jul 21 2009
powerpc: ARRAY_SIZE changes in pasemi and powermac code
Author: Stoyan Gaydarov <sgayda2@uiuc.edu> These changes were a direct result of using a semantic patch More information can be found at http://www.emn.fr/x-info/coccinelle/ Signed-off-by: Stoyan Gaydarov Signed-off-by: Benjamin Herrenschmidt — arch/powerpc/platforms/pasemi/idle.c | 2 +- arch/powerpc/platforms/powermac/feature.c | 4 ++– 2 files changed, 3 insertions(+), 3 deletions(-) diff –git a/arch/powerpc/platforms/pasemi/idle.c b/arch/powerpc/platforms/pasemi/idle.c index 43911d8..75b296b 100644 — a/arch/powerpc/platforms/pasemi/idle.c +++ …