[PATCH][next] media: allegro: fix potential null dereference on header

2020-07-20 Thread Colin King
From: Colin Ian King The pointer header is an alias to msg and msg is being null checked. However, if msg is null then header is also null and this can lead to a null pointer dereference on the assignment type = header->type. Fix this by only dereferencing header after the null check on msg. Add

[PATCH][next][V2] media: allegro: fix potential null dereference on header

2020-07-22 Thread Colin King
From: Colin Ian King The pointer header is an alias to msg and msg is being null checked. However, if msg is null then header is also null and this can lead to a null pointer dereference on the assignment type = header->type. Fix this just using header->type after the null check and removing the

[PATCH] staging: unisys: visorhba: remove redundant initialization of variables scsicmd_id and rc

2020-07-23 Thread Colin King
From: Colin Ian King The variables scsicmd_id and rc is being initialized with a value that is never read and are being updated later with a new value. The initializations are redundant and can be removed. Addresses-Coverity: ("Unused value") Signed-off-by: Colin Ian King --- drivers/staging/u

[PATCH] staging: rtl8723bs: remove redundant assignment to variable ret

2020-07-29 Thread Colin King
From: Colin Ian King The variable ret is being assigned an error return value that is never read, the control passes to a return statement and ret is never referenced. Remove the redundant assignment. Also remove an empty line. Addresses-Coverity: ("Unused value") Signed-off-by: Colin Ian King

[PATCH] staging: ion: fix spelling mistake in function name "detatch" -> "detach"

2020-08-05 Thread Colin King
From: Colin Ian King There is a spelling mistake in the function name ion_dma_buf_detatch. Fix it by removing the extraneous t. Signed-off-by: Colin Ian King --- drivers/staging/android/ion/ion.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/android/

[PATCH][next] staging: wfx: fix a handful of spelling mistakes

2020-08-05 Thread Colin King
From: Colin Ian King There are various spelling mistakes in comments and error messages. Fix these. Signed-off-by: Colin Ian King --- drivers/staging/wfx/data_rx.c | 2 +- drivers/staging/wfx/data_tx.c | 2 +- drivers/staging/wfx/debug.c | 4 ++-- drivers/staging/wfx/hif_rx.c | 2 +- driver

[PATCH][next][V2] staging: wfx: fix a handful of spelling mistakes

2020-08-06 Thread Colin King
From: Colin Ian King There are various spelling mistakes in comments and error messages. Fix these. Signed-off-by: Colin Ian King --- V2: add in some more fixes as spotted by Randy Dunlap --- drivers/staging/wfx/data_rx.c | 2 +- drivers/staging/wfx/data_tx.c | 2 +- drivers/staging/wfx/debu

[PATCH] media: atomisp: fix spelling mistake "unsupport" -> "unsupported"

2020-08-06 Thread Colin King
From: Colin Ian King There are spelling mistakes in some dev_err messages. Fix these. Signed-off-by: Colin Ian King --- drivers/staging/media/atomisp/pci/atomisp_ioctl.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/media/atomisp/pci/atomisp_ioctl.c

[PATCH] staging: kpc2000: kpc_dma: fix spelling mistake "for for" -> "for"

2020-08-18 Thread Colin King
From: Colin Ian King There are a couple of duplicated "for" spelling mistakes in dev_err error messages. Fix these. Signed-off-by: Colin Ian King --- drivers/staging/kpc2000/kpc_dma/fileops.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/kpc2000/kpc_dm

[PATCH][next] staging: spmi: hisi-spmi-controller: fix spelling mistake "controlller" -> "controller"

2020-08-20 Thread Colin King
From: Colin Ian King There is a spelling mistake in the MODULE_ALIAS, fix it. Signed-off-by: Colin Ian King --- drivers/staging/hikey9xx/hisi-spmi-controller.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/hikey9xx/hisi-spmi-controller.c b/drivers/staging

[PATCH] staging: rtl8192e: fix missing failure check on a call to dev_alloc_name

2020-08-21 Thread Colin King
From: Colin Ian King Currently the second call to dev_alloc_name is not checking if this failed. Add the check and perform necessary cleanup on an error. Addresses-Coverity: ("Unchecked return value") Fixes: 94a799425eee ("rtl8192e: Import new version of driver from realtek") Signed-off-by: Col

<    1   2   3   4