Re: [PATCH] isdn/gigaset: check endpoint null in gigaset_probe

On 7/26/19 9:22 PM, Paul Bolle wrote: Phong Tran schreef op vr 26-07-2019 om 20:35 [+0700]: This fixed the potential reference NULL pointer while using variable endpoint. Reported-by: syzbot+35b1c403a14f5c89e...@syzkaller.appspotmail.com Tested by syzbot: https://groups.google.com/d/msg

[PATCH] isdn/gigaset: check endpoint null in gigaset_probe

This fixed the potential reference NULL pointer while using variable endpoint. Reported-by: syzbot+35b1c403a14f5c89e...@syzkaller.appspotmail.com Tested by syzbot: https://groups.google.com/d/msg/syzkaller-bugs/wnHG8eRNWEA/Qn2HhjNdBgAJ Signed-off-by: Phong Tran --- drivers/isdn/gigaset/usb

Re: [PATCH 5.1 00/96] 5.1.17-stable review

On 7/8/19 10:12 PM, Greg Kroah-Hartman wrote: This is the start of the stable review cycle for the 5.1.17 release. There are 96 patches in this series, all will be posted as a response to this one. If anyone has any issues with these being applied, please let me know. Responses should be made b

Re: [PATCH 4.19 00/90] 4.19.58-stable review

Hi Greg, On 7/8/19 10:12 PM, Greg Kroah-Hartman wrote: This is the start of the stable review cycle for the 4.19.58 release. There are 90 patches in this series, all will be posted as a response to this one. If anyone has any issues with these being applied, please let me know. Responses shoul

[PATCH V2] media: usb: technisat-usb2: fix buffer overflow

/syzkaller-bugs/CySBCKuUOOs/t3PvVheSAAAJ Signed-off-by: Phong Tran --- Change Log: * V2: add IR_MAX_BUFFER_INDEX and adjust the while loop condition as comments --- drivers/media/usb/dvb-usb/technisat-usb2.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/media/usb

[PATCH] media: usb: technisat-usb2: fix buffer overflow

/syzkaller-bugs/CySBCKuUOOs/0hKq1CdjCwAJ Signed-off-by: Phong Tran --- drivers/media/usb/dvb-usb/technisat-usb2.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/usb/dvb-usb/technisat-usb2.c b/drivers/media/usb/dvb-usb/technisat-usb2.c index c659e18b358b

[PATCH V2] net: usb: asix: init MAC address buffers

[inline] worker_thread+0x189c/0x2460 kernel/workqueue.c:2417 kthread+0x4b5/0x4f0 kernel/kthread.c:254 ret_from_fork+0x35/0x40 arch/x86/entry/entry_64.S:355 Signed-off-by: Phong Tran --- Changes since v1: - replace memset() by array init --- drivers/net/usb/asix_devices.c | 6 +++--- 1 file

Re: [PATCH] net: usb: asix: init MAC address buffers

Hello Dan, On Mon, Jul 1, 2019 at 10:30 PM Dan Williams wrote: > > On Mon, 2019-07-01 at 06:45 +0700, Phong Tran wrote: > > This is for fixing bug KMSAN: uninit-value in ax88772_bind > > > > Tested by > > https://groups.google.com/d/msg/syzkaller-bugs/aFQurGotng4/cF

[PATCH] net: usb: asix: init MAC address buffers

[inline] worker_thread+0x189c/0x2460 kernel/workqueue.c:2417 kthread+0x4b5/0x4f0 kernel/kthread.c:254 ret_from_fork+0x35/0x40 arch/x86/entry/entry_64.S:355 Signed-off-by: Phong Tran --- drivers/net/usb/asix_devices.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/net/usb

[PATCH 06/15] ARM: imx: cleanup cppcheck shifting errors

[arch/arm/mach-imx/iomux-mx3.h:93]: (error) Shifting signed 32-bit value by 31 bits is undefined behaviour Signed-off-by: Phong Tran --- arch/arm/mach-imx/iomux-mx3.h | 64 +-- 1 file changed, 32 insertions(+), 32 deletions(-) diff --git a/arch/arm/mach

[PATCH] power: supply: core: fix clang -Wunsequenced

: multiple unsequenced modifications to 'list' [-Werror,-Wunsequenced] Link: https://github.com/ClangBuiltLinux/linux/issues/460 Signed-off-by: Phong Tran --- drivers/power/supply/power_supply_core.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/

[PATCH V2] of: fix clang -Wunsequenced for be32_to_cpu()

note: expanded from macro '__swab32' ___constant_swab32(x) : \ ^ ./include/uapi/linux/swab.h:18:12: note: expanded from macro '___constant_swab32' (((__u32)(x) & (__u32)0x00ffUL) << 24) |\

Re: [PATCH] of: replace be32_to_cpu to be32_to_cpup

On Tue, Apr 30, 2019 at 8:32 PM Christoph Hellwig wrote: > > On Tue, Apr 30, 2019 at 04:00:44PM +0700, Phong Tran wrote: > > > > diff --git a/include/linux/of.h b/include/linux/of.h > > index e240992e5cb6..1c35fc8f19b0 100644 > > --- a/include/linux/of.h > > ++

[PATCH] of: replace be32_to_cpu to be32_to_cpup

UL) << 24) |\ ^ Signed-off-by: Phong Tran --- include/linux/of.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/linux/of.h b/include/linux/of.h index e240992e5cb6..1c35fc8f19b0 100644 --- a/include/linux/of.h +++ b/include/linux/of.h

[PATCH] arm64: defconfig: Fix warning override MMC_BLOCK_MINORS

Remove the duplicate of defination of CONFIG_MMC_BLOCK_MINORS. Fixes 22b0898 ("arm64: add defconfig options for Allwinner SoCs") Signed-off-by: Phong Tran --- arch/arm64/configs/defconfig | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/arm64/configs/defconfig b/arch/arm

[PATCH V3 1/2] staging: android: ion_test: unregister the misc device

Add the remove() method for deregister from misc device when it's unloaded. Signed-off-by: Phong Tran --- drivers/staging/android/ion/ion_test.c | 12 1 file changed, 12 insertions(+) diff --git a/drivers/staging/android/ion/ion_test.c b/drivers/staging/android/ion/ion_t

[PATCH V3 2/2] staging: android: ion_test: unregister the platform device

The driver has to unregister from platform device when it's unloaded Signed-off-by: Phong Tran --- drivers/staging/android/ion/ion_test.c | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/drivers/staging/android/ion/ion_test.c b/drivers/staging/android/ion/ion_t

[PATCH V3 0/2] Fix some issues of staging ion test driver

Hi Greg, I resend 2 patches (1) add the remove() platform driver callback to deregister the misc device (2) unregister platform device when driver is unloaded Changes from v2: Remove the extra space of patch (1) Changes from v1: Imporve the coding style Phong Tran (2): (1

[PATCH V2 0/3] Fix some issues of staging ion test driver

oding style Phong Tran (3): staging: android: ion_test: Add the MODULE_LICENSE macro staging: android: ion_test: unregister the misc device staging: android: ion_test: unregister the platform device drivers/staging/android/ion/ion_test.c | 21 - 1 file changed, 20 inser

[PATCH V2 3/3] staging: android: ion_test: unregister the platform device

The driver has to unregister from platform device when it's unloaded Signed-off-by: Phong Tran --- drivers/staging/android/ion/ion_test.c | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/drivers/staging/android/ion/ion_test.c b/drivers/staging/android/ion/ion_t

[PATCH V2 1/3] staging: android: ion_test: Add the MODULE_LICENSE macro

Base on the file comment should define GPL v2 for ion test driver Signed-off-by: Phong Tran --- drivers/staging/android/ion/ion_test.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/android/ion/ion_test.c b/drivers/staging/android/ion/ion_test.c index 654acb5..3bc461c

[PATCH V2 2/3] staging: android: ion_test: unregister the misc device

Add the remove() method for deregister from misc device when it's unloaded. Signed-off-by: Phong Tran --- drivers/staging/android/ion/ion_test.c | 12 1 file changed, 12 insertions(+) diff --git a/drivers/staging/android/ion/ion_test.c b/drivers/staging/android/ion/ion_t

[PATCH 2/3] staging: android: ion_test: unregister the misc device

Add the remove() method for deregister from misc device when it's unloaded. Signed-off-by: Phong Tran --- drivers/staging/android/ion/ion_test.c | 18 ++ 1 file changed, 18 insertions(+) diff --git a/drivers/staging/android/ion/ion_test.c b/drivers/staging/androi

[PATCH 3/3] staging: android: ion_test: unregister the platform device

The driver has to unregister from platform device when it's unloaded Signed-off-by: Phong Tran --- drivers/staging/android/ion/ion_test.c | 11 ++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/drivers/staging/android/ion/ion_test.c b/drivers/staging/androi

[PATCH 1/3] staging: android: ion_test: Add the MODULE_LICENSE macro

Base on the file comment should define GPL v2 for ion test driver Signed-off-by: Phong Tran --- drivers/staging/android/ion/ion_test.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/android/ion/ion_test.c b/drivers/staging/android/ion/ion_test.c index 654acb5..3bc461c

[PATCH 0/3] Fix some issues of staging ion test driver

Hi Greg, Arve, Riley, This patcheset fixes: (1) Define the GPL V2 for ion_test driver (2) Add the remove() platform driver callback to deregister the misc device (3) unregister platform device when driver is unloaded Phong Tran (3): staging: android: ion_test: Add the MODULE_LICENSE macro

[PATCH] PCI: xgene: Fix the map_bus callback prototype

The return value should be void __iomem *. Tested by compile only. Signed-off-by: Phong Tran --- drivers/pci/host/pci-xgene.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/pci/host/pci-xgene.c b/drivers/pci/host/pci-xgene.c index aab5547..9c3bfd8 100644 --- a

[PATCH] staging: lustre: lnet: socklnd_cb.c Fix sparse non-static symbol warning

This patch fixes some non static function declarations that cause sparse warning. Tested by compilation. Signed-off-by: Phong Tran --- .../staging/lustre/lnet/klnds/socklnd/socklnd_cb.c | 30 +++--- 1 file changed, 15 insertions(+), 15 deletions(-) diff --git a/drivers/staging

[PATCH] tools: selftests: Fixing build issue with make kselftests target

Fix the typo of ARCH when build with 'make kselftests' Change the 'X86' to 'x86'. Test by compilation Signed-off-by: Phong Tran --- tools/testing/selftests/ipc/Makefile | 6 +++--- tools/testing/selftests/kcmp/Makefile | 6 +++--- tools/testing/selftests/mem

[PATCH V3 2/3] staging: android: ion: ion.c Add a new blank line after decleration

This patch fix checkpatch.pl warning Tested by compilation only. Signed-off-by: Phong Tran --- drivers/staging/android/ion/ion.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/android/ion/ion.c b/drivers/staging/android/ion/ion.c index 2703609..cad76ae 100644 --- a/drivers

[PATCH V3 3/3] staging: android: ion: Remove redundant return of void function

This patch fix checkpatch.pl warning Tested by compilation only. Signed-off-by: Phong Tran --- drivers/staging/android/ion/ion.c | 1 - drivers/staging/android/ion/ion_carveout_heap.c | 1 - drivers/staging/android/ion/ion_chunk_heap.c| 1 - drivers/staging/android/ion

[PATCH V3 1/3] staging: android: ion: ion_dummy_driver.c Replace kzalloc() by kcalloc()

This patch fix checkpatch.pl warning Tested by compilation only. Signed-off-by: Phong Tran --- drivers/staging/android/ion/ion_dummy_driver.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/android/ion/ion_dummy_driver.c b/drivers/staging/android/ion

[Patch V2 3/3] staging: android: ion: Fix coding style

This patch fix checkpatch.pl warning - Remove the return of void function Tested by compilation. Signed-off-by: Phong Tran --- drivers/staging/android/ion/ion.c | 1 - drivers/staging/android/ion/ion_carveout_heap.c | 1 - drivers/staging/android/ion/ion_chunk_heap.c| 1

[Patch V2 2/3] staging: android: ion: ion.c Fix coding style

This patch fix checkpatch.pl warning - Add a new line after decleration Tested by compilation. Signed-off-by: Phong Tran --- drivers/staging/android/ion/ion.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/android/ion/ion.c b/drivers/staging/android/ion/ion.c index 2703609

[Patch V2 1/3] staging: android: ion: ion_dummy_driver.c Fix coding style

This patch fix checkpatch.pl warning - Replace kzalloc() by kcalloc() Tested by compilation. Signed-off-by: Phong Tran --- drivers/staging/android/ion/ion_dummy_driver.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/android/ion/ion_dummy_driver.c b/drivers

[PATCH 1/5] staging: android: ion: ion.c Fix checkpatch warning

This patch fix the coding style - Add a new line after variable declaration - Remove return of void fuction Tested by compilation Signed-off-by: Phong Tran --- drivers/staging/android/ion/ion.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/android/ion

[PATCH 4/5] staging: android: ion: ion_carveout_heap.c Fix checkpatch warning

This patch fix coding style - Remove return of void function Tested by compilation Signed-off-by: Phong Tran --- drivers/staging/android/ion/ion_carveout_heap.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/android/ion/ion_carveout_heap.c b/drivers/staging/android/ion

[PATCH 3/5] staging: android: ion: ion_system_heap.c Fix checkpatch warning

This patch fix coding rule - Remove return of void function Tested by compilation Signed-off-by: Phong Tran --- drivers/staging/android/ion/ion_system_heap.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/android/ion/ion_system_heap.c b/drivers/staging/android/ion

[PATCH 2/5] staging: android: ion: ion_chunk_heap.c Fix checkpatch warning

This patch fix coding style - Remove return of void function Tested by compilation Signed-off-by: Phong Tran --- drivers/staging/android/ion/ion_chunk_heap.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/android/ion/ion_chunk_heap.c b/drivers/staging/android/ion

[PATCH 5/5] staging: android: ion: ion_dummy_driver.c Fix checkpatch warning

This patch fix coding style - Replace kzalloc() by kcalloc() - Remove return of void function Tested by compilation Signed-off-by: Phong Tran --- drivers/staging/android/ion/ion_dummy_driver.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/staging/android/ion

staging: android: ion checkpatch.pl cleanups

Hi Greg, These patches fix checkpatch warning. Apply for staging-next branch. Regards, Phong. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please

[PATCH] staging: android: ion: ion_cma_heap.c: Fix checkpatch warning

This patch fix coding style: - Remove "fail memory allocation" waring - Remove return of void function Tested by compilation only Signed-off-by: Phong Tran --- drivers/staging/android/ion/ion_cma_heap.c | 9 ++--- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git