Replace comma between expressions with semicolons.
Using a ',' in place of a ';' can have unintended side effects.
Although that is not the case here, it is seems best to use ';'
unless ',' is intended.
Found by inspection.
No functional change intended.
C
Remove duplicate header which is included twice.
Signed-off-by: Chen Ni
---
tools/testing/selftests/filesystems/utils.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/tools/testing/selftests/filesystems/utils.c
b/tools/testing/selftests/filesystems/utils.c
index e553c89c5b19..06419bf4ba19
Replace comma between expressions with semicolons.
Using a ',' in place of a ';' can have unintended side effects.
Although that is not the case here, it is seems best to use ';'
unless ',' is intended.
Found by inspection.
No functional change intended.
C
Remove unnecessary semicolons reported by Coccinelle/coccicheck and the
semantic patch at scripts/coccinelle/misc/semicolon.cocci.
Signed-off-by: Chen Ni
---
tools/testing/selftests/kvm/access_tracking_perf_test.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/testing
Remove unnecessary semicolons reported by Coccinelle/coccicheck and the
semantic patch at scripts/coccinelle/misc/semicolon.cocci.
Signed-off-by: Chen Ni
---
tools/testing/selftests/timers/set-timer-lat.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/testing
Replace comma between expressions with semicolons.
Using a ',' in place of a ';' can have unintended side effects.
Although that is not the case here, it is seems best to use ';'
unless ',' is intended.
Found by inspection.
No functional change intended.
C
Replace a comma between expression statements by a semicolon.
Signed-off-by: Chen Ni
---
tools/testing/selftests/net/psock_fanout.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/tools/testing/selftests/net/psock_fanout.c
b/tools/testing/selftests/net/psock_fanout.c
Use devm_kstrdup() instead of kstrdup() and check its return value to
avoid memory leak.
Fixes: 49bddc73d15c ("libnvdimm/of_pmem: Provide a unique name for bus
provider")
Signed-off-by: Chen Ni
---
Changelog:
v2 -> v3:
1. Use devm_kstrdup() instead of kstrdup()
v1 -> v2:
1.
Check the return value of kstrdup() and add kfree() for kstrdup() to
avoid memory leak.
Fixes: 49bddc73d15c ("libnvdimm/of_pmem: Provide a unique name for bus
provider")
Signed-off-by: Chen Ni
---
Changelog:
v1 -> v2:
1.Add a fixes tag.
2.Update commit message.
---
drivers/nvd
Add kfree() for kstrdup() in order to avoid memory leak.
Signed-off-by: Chen Ni
---
drivers/nvdimm/of_pmem.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/drivers/nvdimm/of_pmem.c b/drivers/nvdimm/of_pmem.c
index 10dbdcdfb9ce..fe6edb7e6631 100644
--- a/drivers/nvdimm/of_pmem.c
+++ b
From: Xu Wang
Use offset_in_page macro instead of (addr & ~PAGE_MASK).
Signed-off-by: Xu Wang
---
kernel/events/uprobes.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/kernel/events/uprobes.c b/kernel/events/uprobes.c
index bb0862873dba..44d175d81ed6 100644
--- a/ke
From: Xu Wang
usb_free_coherent() is safe with NULL addr and this check is
not required.
Signed-off-by: Xu Wang
---
drivers/net/wireless/zydas/zd1211rw/zd_usb.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/net/wireless/zydas/zd1211rw/zd_usb.c
b/drivers/net/wi
From: Xu Wang
usb_free_coherent() is safe with NULL addr and this check is
not required.
Signed-off-by: Xu Wang
---
drivers/input/misc/cm109.c | 8 ++--
1 file changed, 2 insertions(+), 6 deletions(-)
diff --git a/drivers/input/misc/cm109.c b/drivers/input/misc/cm109.c
index c09b9628ad34.
13 matches
Mail list logo