- 13 Sep, 2015 40 commits
-
-
Sudip Mukherjee authored
Using strcpy() is a security risk as the destination buffer size is not checked and we may over-run the buffer. Use strncpy() instead, while mentioning the buffer size leaving place for the NULL termination. Signed-off-by:
Sudip Mukherjee <sudip@vectorindia.org> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Shraddha Barke authored
The linux kernel coding style discourages use of braces for single statement blocks. This patch removes the unnecessary braces. The warning was detected using checkpatch.pl. Coccinelle was used to make the change. Signed-off-by:
Shraddha Barke <shraddha.6596@gmail.com> Signed-off-by:
-
Shraddha Barke authored
The linux kernel coding style discourages use of braces for single statement blocks. This patch removes the unnecessary braces. The warning was detected using checkpatch.pl. Coccinelle was used to make the change. Signed-off-by:
-
Shraddha Barke authored
The linux kernel coding style discourages use of braces for single statement blocks. This patch removes the unnecessary braces. The warning was detected using checkpatch.pl. Coccinelle was used to make the change. Signed-off-by:
-
Shraddha Barke authored
The linux kernel coding style discourages use of braces for single statement blocks. This patch removes the unnecessary braces. The error was detected using checkpatch.pl. Coccinelle was used to make the change. Signed-off-by:
-
Anish Bhatt authored
ETH_ALEN/ETH_HLEN can be used instead of WLAN_ETHADDR_LEN & WLAN_ETHHDR_LEN, replace directly or use eth_addr* functions where applicable. Signed-off-by:
Anish Bhatt <anish@chelsio.com> Signed-off-by:
-
Nayeemahmed Badebade authored
Fixed coding style issues where kernel types u16,u64,u32 should be preferred over uint16_t,uint64_t,uint32_t Signed-off-by:
Nayeemahmed Badebade <itachi.opsrc@gmail.com> Signed-off-by:
-
Ravi Teja Darbha authored
struct nbu2ss_ep *ep parameter in functions _nbu2ss_ep0_in_transfer() and _nbu2ss_ep0_out_transfer() is not used anywhere inside. Hence, removed. Signed-off-by:
Ravi Teja Darbha <ravi2j@gmail.com> Signed-off-by:
-
Ravi Teja Darbha authored
The cleanup1 label does nothing but return. Better way is to return immediately instead of using goto. Hence, removed. Signed-off-by:
-
Ravi Teja Darbha authored
Fixed NULL comparison style as suggested by checkpatch.pl Signed-off-by:
-
Ravi Teja Darbha authored
Fixed multiple blank lines warning by checkpatch.pl Signed-off-by:
-
Ravi Teja Darbha authored
FSF mailing address is no longer required to be specified. Hence removed. Signed-off-by:
-
Sudip Mukherjee authored
The member qc_idx of struct if_quotactl is unsigned and hence it can never be less than zero. Signed-off-by:
-
Sudip Mukherjee authored
If lov_sub_get() fails then it returns the error code in ERR_PTR, but here we were dereferencing sub without checking if lov_sub_get() has actually succeeded or not. And on error we can directly return the error code from lov_io_fault_start() as it return 0 on success and the error code on error. Signed-off-by:
-
Anders Fridlund authored
Remove sparse warning "symbol 'sptlrpc_plain_init' was not declared" by including ptlrpc_internal.h, which includes sptlrpc_plain_init. Signed-off-by:
Anders Fridlund <anders.fridlund@gmail.com> Signed-off-by:
-
Mike Rapoport authored
Fix the following checkpatch warning: drivers/staging/lustre/lustre/include/linux/lustre_lite.h:46: WARNING:INCLUDE_LINUX: Use #include <linux/statfs.h> instead of <asm/statfs.h> Signed-off-by:
Mike Rapoport <mike.rapoport@gmail.com> Signed-off-by:
-
Matthew Needes authored
Fixed sparse warning by removing unused variable "proc_version". Signed-off-by:
Matthew Needes <mneedes@gmail.com> Signed-off-by:
-
Shraddha Barke authored
The linux kernel coding style discourages use of braces for single statement blocks. This patch removes the unnecessary braces. The warning was detected using checkpatch.pl. Coccinelle was used to make the change. Signed-off-by:
-
Shraddha Barke authored
The linux kernel coding style discourages use of braces for single statement blocks. This patch removes the unnecessary braces. The warning was detected using checkpatch.pl. Coccinelle was used to make the change. Signed-off-by:
-
Shraddha Barke authored
The linux kernel coding style discourages use of braces for single statement blocks. This patch removes the unnecessary braces. The warning was detected using checkpatch.pl. Coccinelle was used to make the change. Signed-off-by:
-
Shraddha Barke authored
The linux kernel coding style discourages use of braces for single statement blocks. This patch removes the unnecessary braces. The warning was detected using checkpatch.pl. Coccinelle was used to make the change. Signed-off-by:
-
Shraddha Barke authored
The linux kernel coding style discourages use of braces for single statement blocks. This patch removes the unnecessary braces. The warning was detected using checkpatch.pl. Coccinelle was used to make the change. Signed-off-by:
-
Shraddha Barke authored
The linux kernel coding style discourages use of braces for single statement blocks. This patch removes the unnecessary braces. The warning was detected using checkpatch.pl. Coccinelle was used to make the change. Signed-off-by:
-
Shraddha Barke authored
This patch fixes the checkpatch.pl warning: WARNING: space prohibited between function name and open parenthesis '(' Signed-off-by: -
Shraddha Barke authored
This patch fixes the checkpatch.pl warning WARNING: Missing a blank line after declarations Signed-off-by:
-
Shraddha Barke authored
This patch removes return statement which are not required. The warning was detected by checkpatch.pl WARNING: void function return statements are not generally useful Signed-off-by:
-
Shraddha Barke authored
void pointers do not need to be cast to other pointer types. The semantic patch used to find this: @r@ expression x; void* e; type T; identifier f; @@ ( *((T *)e) | ((T *)x)[...] | ((T *)x)->f | - (T *) e ) Signed-off-by:
-
Shraddha Barke authored
void pointers do not need to be cast to other pointer types. The semantic patch used to find this: @r@ expression x; void* e; type T; identifier f; @@ ( *((T *)e) | ((T *)x)[...] | ((T *)x)->f | - (T *) e ) Signed-off-by:
-
Shraddha Barke authored
This patch uses kasprintf which combines kzalloc and sprintf. kasprintf also takes care of the size calculation. Semantic patch used is as follows: @@ expression a,flag; expression list args; statement S; @@ a = - \(kmalloc\|kzalloc\)(...,flag) + kasprintf (flag,args) <... when != a if (a == NULL || ...) S ...> - sprintf(a,args); Signed-off-by:
-
Shraddha Barke authored
The Linux kernel coding style guidelines suggest not using typedefs for structure and enum types. This patch gets rid of the typedefs for lu_site_stats_t. The following Coccinelle semantic patch detects the cases for struct type: @tn@ identifier i; type td; @@ -typedef struct i { ... } -td ; @@ type tn.td; identifier tn.i; @@ -td + struct i Signed-off-by: -
Shraddha Barke authored
The Linux kernel coding style guidelines suggest not using typedefs for structure and enum types. This patch gets rid of the typedefs for cfs_wi_sched_t. The following Coccinelle semantic patch detects the cases for struct type: @tn@ identifier i; type td; @@ -typedef struct i { ... } -td ; @@ type tn.td; identifier tn.i; @@ -td + struct i Signed-off-by: -
Leo Kim authored
This patch removes the preprocessor definition, PLAT_RK3026_TCHIP which is not used anymore. Signed-off-by:
Leo Kim <leo.kim@atmel.com> Signed-off-by:
Tony Cho <tony.cho@atmel.com> Signed-off-by:
-
Kim, Leo authored
The preprocessor definition, USE_WIRELESS is always in use. So, this patch deletes the definitions from the makefile and source codes. Signed-off-by:
-
Kim, Leo authored
This patch removes the preprocessor definition, NM73131_0_BOARD which is not used anymore. Signed-off-by:
-
Tony Cho authored
This patch adds new maintainers who develops new design for Atmel 802.11 link controller and deletes Dean Lee who cannot maintain the wireless driver anymore. Signed-off-by:
-
Johnny Kim authored
The current driver is calling skb_reserve function to align memory in frmw_to_linux function. However, the wrong value is used as the input argument. Besides, the process is not necessary because the address already is algined in dev_alloc_skb function. So, this patch removes the line for dummy headroom. Signed-off-by:
Johnny Kim <johnny.kim@atmel.com> Signed-off-by:
-
Johnny Kim authored
This patch adds new define, IDLE_MODE to change comparison statement which is wrong due to typecasting to null. Signed-off-by:
-
Chaehyun Lim authored
This patch adds error check if kzalloc is failed. Signed-off-by:
Chaehyun Lim <chaehyun.lim@gmail.com> Signed-off-by:
-
Chaehyun Lim authored
This patch use kmalloc with GFP_ATOMIC instead of WILC_MALLOC. It is inside the spin lock region. Signed-off-by:
-
Chaehyun Lim authored
This patch replaces WILC_MALLOC with kzalloc. Signed-off-by:
-
