[PATCH] staging: lustre: ptlrpc: lproc_ptlrpc.c - fix dereferenceing user space buffer

2014-07-31 Thread Anil Belur
drivers/staging/lustre/lustre/ptlrpc/lproc_ptlrpc.c:652:33:got char const *buffer Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/ptlrpc/lproc_ptlrpc.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/ptlrpc/lproc_ptlrpc.c b

[PATCH] staging: lustre: ldlm: ldlm_resource.c - fix dereferenceing user space buffer

2014-07-30 Thread Anil Belur
char const *buffer Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/ldlm/ldlm_resource.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre/lustre/ldlm/ldlm_resource.c b/drivers/staging/lustre/lustre/ldlm/ldlm_resource.c index 91d028b

[PATCH v3] staging: vt6655: ioctl.c - missing __user annotation

2014-07-30 Thread Anil Belur
rrect type in argument 2 (different address spaces) drivers/staging/vt6655/ioctl.c:78:51:expected void const [noderef] asn:1>*from drivers/staging/vt6655/ioctl.c:78:51:got void *data Signed-off-by: Anil Belur --- v2: - replaced the 'void __user *' casts by adding '__

[PATCH v2 1/1] staging: vt6655: ioctl.c - missing __user annotation

2014-07-30 Thread Anil Belur
From: Anil Belur v2: - private_ioctl() internally calls copy_{to,from}_user() and does not use '__user' which gives out several sparse warnings - this patch adds __user annotation to the data member of struct tagSCmdRequest as suggested by tklau...@distanz.ch - sparse warn

[PATCH 1/1] drivers: staging: vt6655: ioctl.c - missing __user annotation

2014-07-28 Thread Anil Belur
From: Anil Belur - private_ioctl() the internally calls copy_{to,from}_user() and does not use '__user' while refrencing user space pointers. - this patch passes __user annotation as a cast, when the pointer is being refernced. - this patch fixes the following sparse errors: drive

[PATCH 1/1] drivers: staging: cxt1e1: linux.c - missing __user annotation

2014-07-27 Thread Anil Belur
From: Anil Belur - Some of the functions internally call copy_{to,from}_user() but does not use '__user'. this patch fixes missing __user annotations. - this patch fixes the following sparse errors: drivers/staging/cxt1e1/linux.c:488:33: warning: incorrect type in argument 2

[PATCH 2/2] staging: rtl8723au: core: rtw_cmd.c - removed NULL pointer check before kfree()

2014-07-09 Thread Anil Belur
From: Anil Belur - as kfree() internally checks for NULL, additional check it not required. Signed-off-by: Anil Belur --- drivers/staging/rtl8723au/core/rtw_cmd.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/rtl8723au/core/rtw_cmd.c b/drivers/staging

[PATCH 1/2] staging: rtl8723au: core: rtw_ap.c - removed NULL pointer check before kfree()

2014-07-09 Thread Anil Belur
From: Anil Belur - as kfree() internally check for NULL, additional check it not required. Signed-off-by: Anil Belur --- drivers/staging/rtl8723au/core/rtw_ap.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/rtl8723au/core/rtw_ap.c b/drivers/staging

[PATCH 1/2] staging: media: bcm2048: radio-bcm2048.c - removed IRQF_DISABLED macro

2014-07-08 Thread Anil Belur
From: Anil Belur - this patch removes IRQF_DISABLED macro, as this is deprecated/noop. Signed-off-by: Anil Belur --- drivers/staging/media/bcm2048/radio-bcm2048.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/media/bcm2048/radio-bcm2048.c b/drivers

[PATCH 2/2] staging: nokia_h4p: nokia_core.c - removed IRQF_DISABLED macro

2014-07-08 Thread Anil Belur
From: Anil Belur - this patch removes the IRQF_DISABLED macro, as this is deprecated/noop. Signed-off-by: Anil Belur --- drivers/staging/nokia_h4p/nokia_core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/nokia_h4p/nokia_core.c b/drivers/staging

[PATCH v2 1/1] staging: lustre: libcfs: nidstrings.c - use ARRAY_SIZE macro

2014-07-02 Thread Anil Belur
From: Anil Belur - this patch replaces sizeof(a)/sizeof(a[0]) with the ARRAY_SIZE macro to get the number of nidstrings (and netstrfns) - v2: also for netstrfns Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/libcfs/nidstrings.c | 5 ++--- 1 file changed, 2 insertions(+), 3

[PATCH 1/1] staging: speakup: main.c - use time_after()

2014-07-01 Thread Anil Belur
From: Anil Belur - this patch fixes jiffies comparision with a safer function to prevent any overflows Signed-off-by: Anil Belur --- drivers/staging/speakup/main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/speakup/main.c b/drivers/staging/speakup

[PATCH 1/1] staging: lustre: libcfs: nidstrings.c - use ARRAY_SIZE macro

2014-06-30 Thread Anil Belur
From: Anil Belur - this patch replaces sizeof(a)/sizeof(a[0]) with the ARRAY_SIZE macro to get the number of nidstrings Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/libcfs/nidstrings.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging

[PATCH 1/1] staging: speakup: speakup_dectlk.c - use time_before_eq()

2014-06-30 Thread Anil Belur
From: Anil Belur - this replaces jiffies comparision with safer function using time_after_eq() Signed-off-by: Anil Belur --- drivers/staging/speakup/speakup_dectlk.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/speakup/speakup_dectlk.c b/drivers

[PATCH 1/1] staging: media: msi3101: sdr-msi3101.c - replace with time_before_eq()

2014-06-28 Thread Anil Belur
From: Anil Belur - this fix replaces jiffies interval comparision with safer function to avoid any overflow and wrap around ? Signed-off-by: Anil Belur --- drivers/staging/media/msi3101/sdr-msi3101.c | 16 1 file changed, 12 insertions(+), 4 deletions(-) diff --git a

[PATCH v2 1/1] staging: lustre: lnet: api-ni.c: fixed sparse warnings

2014-06-28 Thread Anil Belur
From: Anil Belur - this fixes few sparse warnings for missing static in functions: drivers/staging/lustre/lnet/lnet/api-ni.c:63:6: warning: symbol 'lnet_get_routes' was not declared. Should it be static? drivers/staging/lustre/lnet/lnet/api-ni.c:69:6: warning: symbol 'lnet_ge

[PATCH 1/1] staging: ozwpan: ozproto.c fix for "WARNING: Prefer ether_addr_copy() over memcpy() if the Ethernet addresses are __aligned(2)"

2014-05-19 Thread Anil Belur
From: Anil Belur Fixed the warning message by replacing memcpy() with ether_addr_copy() Signed-off-by: Anil Belur --- drivers/staging/ozwpan/ozproto.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/ozwpan/ozproto.c b/drivers/staging/ozwpan/ozproto.c index