Re: [PATCH 2/2] can: m_can: m_can_class_allocate_dev(): remove impossible error return judgment

2021-02-05 Thread Xulin Sun
On 2021/2/5 下午4:19, Marc Kleine-Budde wrote: On 05.02.2021 15:25:59, Xulin Sun wrote: If the previous can_net device has been successfully allocated, its private data structure is impossible to be empty, remove this redundant error return judgment. Otherwise, memory leaks for alloc_candev

[PATCH 2/2] can: m_can: m_can_class_allocate_dev(): remove impossible error return judgment

2021-02-04 Thread Xulin Sun
If the previous can_net device has been successfully allocated, its private data structure is impossible to be empty, remove this redundant error return judgment. Otherwise, memory leaks for alloc_candev() will be triggered. Signed-off-by: Xulin Sun --- drivers/net/can/m_can/m_can.c | 5

[PATCH 1/2] can: m_can: m_can_plat_probe(): free can_net device in case probe fails

2021-02-04 Thread Xulin Sun
b9f1>] device_initial_probe+0x28/0x34 [<82e2c85c>] bus_probe_device+0xa4/0xb0 [<cc6181c3>] deferred_probe_work_func+0x7c/0xb0 [<01b85f22>] process_one_work+0x1ec/0x480 Signed-off-by: Xulin Sun --- drivers/net/can/m_can/m_can_platform.c | 10 -- 1 fil

[PATCH] net: mscc: ocelot: replace readx_poll_timeout with readx_poll_timeout_atomic

2020-05-14 Thread Xulin Sun
el0_svc_common.constprop.0+0x78/0x190 el0_svc_handler+0x70/0x90 el0_svc+0x8/0xc Signed-off-by: Xulin Sun --- drivers/net/ethernet/mscc/ocelot.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/ethernet/mscc/ocelot.c b/drivers/net/ethernet/mscc/ocelot.c index b4731df186f4

[PATCH] brcmfmac: replace strncpy() by strscpy()

2019-08-23 Thread Xulin Sun
: 'strncpy' output truncated before terminating nul copying 3 bytes from a string of the same length [-Wstringop-truncation] strncpy(iebuf, add_del_cmd, VNDR_IE_CMD_LEN - 1); ^~~~ Signed-off-by: Xulin Sun --- drivers/net/wireless/broadcom/brcm80211

Re: [PATCH 00/10] GICv3 support for kexec/kdump on EFI systems

2019-02-01 Thread Xulin Sun
On 02/01/2019 05:15 PM, Marc Zyngier wrote: Hi Xulin, On 01/02/2019 06:11, Sun Ted wrote: Hi Marc, Marc Zyngier mailto:marc.zyng...@arm.com>> 于2018 年9月22日周六 上午4:03写道: The GICv3 architecture has the remarkable feature that once LPI tables have been assigned to redistributors and

[PATCH] rtc: pcf2127: fix a kmemleak caused in pcf2127_i2c_gather_write

2018-11-06 Thread Xulin Sun
+0x34/0x38 [] 0x Signed-off-by: Xulin Sun --- drivers/rtc/rtc-pcf2127.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/rtc/rtc-pcf2127.c b/drivers/rtc/rtc-pcf2127.c index e83be18..598e56e 100644 --- a/drivers/rtc/rtc-pcf2127.c +++ b/drivers/rtc/rtc-pcf2127.c @@ -248,6 +248,9 @@

[PATCH] rtc: pcf2127: fix a kmemleak caused in pcf2127_i2c_gather_write

2018-11-06 Thread Xulin Sun
+0x34/0x38 [] 0x Signed-off-by: Xulin Sun --- drivers/rtc/rtc-pcf2127.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/rtc/rtc-pcf2127.c b/drivers/rtc/rtc-pcf2127.c index e83be18..598e56e 100644 --- a/drivers/rtc/rtc-pcf2127.c +++ b/drivers/rtc/rtc-pcf2127.c @@ -248,6 +248,9 @@

[PATCH v2] crypto: caam - free qman_fq after kill_fq

2017-07-13 Thread Xulin Sun
to remove the additional kfree to qman_fq and avoid re-calling the released qman_fq. Signed-off-by: Xulin Sun <xulin@windriver.com> --- v1->v2: Checked where the function kill_fq() is called to avoid re-calling the released qman_fq. drivers/crypto/caam/qi.c | 10 -- 1 file ch

[PATCH v2] crypto: caam - free qman_fq after kill_fq

2017-07-13 Thread Xulin Sun
to remove the additional kfree to qman_fq and avoid re-calling the released qman_fq. Signed-off-by: Xulin Sun --- v1->v2: Checked where the function kill_fq() is called to avoid re-calling the released qman_fq. drivers/crypto/caam/qi.c | 10 -- 1 file changed, 4 insertions(+), 6 delet

[PATCH] crypto: caam - free qman_fq after kill_fq

2017-07-11 Thread Xulin Sun
to remove the additional kfree to qman_fq. Signed-off-by: Xulin Sun <xulin@windriver.com> --- drivers/crypto/caam/qi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/crypto/caam/qi.c b/drivers/crypto/caam/qi.c index 1990ed4..c4b9173 100644 --- a/drivers/crypto

[PATCH] crypto: caam - free qman_fq after kill_fq

2017-07-11 Thread Xulin Sun
to remove the additional kfree to qman_fq. Signed-off-by: Xulin Sun --- drivers/crypto/caam/qi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/crypto/caam/qi.c b/drivers/crypto/caam/qi.c index 1990ed4..c4b9173 100644 --- a/drivers/crypto/caam/qi.c +++ b/drivers/crypto

Re: [PATCH] dmaengine: do not allow access outside of unmap_pool

2016-08-23 Thread Xulin Sun
>On Tue, May 17, 2016 at 01:00:46PM +0100, Colin King wrote: >> From: Colin Ian King >> >> When CONFIG_DMA_ENGINE_RAID is defined, unmap_pool[] is just 1 >> element in size, however, allows orders of 2..8 to access >> outside unmap_pool and returns an invalid address.

Re: [PATCH] dmaengine: do not allow access outside of unmap_pool

2016-08-23 Thread Xulin Sun
>On Tue, May 17, 2016 at 01:00:46PM +0100, Colin King wrote: >> From: Colin Ian King >> >> When CONFIG_DMA_ENGINE_RAID is defined, unmap_pool[] is just 1 >> element in size, however, allows orders of 2..8 to access >> outside unmap_pool and returns an invalid address. Ensure >> we fall into the