[PATCH v2 1/2] staging: most: sound: add sanity check for function argument

This patch checks the function parameter 'bytes' before doing the subtraction to prevent memory corruption. Signed-off-by: Christian Gromm Reported-by: Dan Carpenter --- v2: change if condition to avoid difference of subtraction becomes negative. drivers/staging/most/sound/sound.c | 2 ++ 1 fi

[PATCH 2/2] staging: most: sound: use non-safe list iteration

This patch replaces the safe list iteration function with the non-safe one, as no list element is being deleted. Signed-off-by: Christian Gromm Reported-by: Dan Carpenter --- drivers/staging/most/sound/sound.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/stagi

[PATCH 0/2] staging: most: sound: implement improvements reported in code review

This patch set fixes two problems found during code audit. Christian Gromm (2): staging: most: sound: add sanity check for function argument staging: most: sound: use non-safe list iteration drivers/staging/most/sound/sound.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) -- 2

[PATCH 1/2] staging: most: sound: add sanity check for function argument

This patch zero checks the function parameter 'bytes' before doing the subtraction to prevent memory corruption. Signed-off-by: Christian Gromm Reported-by: Dan Carpenter --- drivers/staging/most/sound/sound.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/staging/most/sound/soun

Re: [PATCH 1/2] staging: most: sound: add sanity check for function argument

On Tue, Feb 02, 2021 at 12:38:09PM +0100, Christian Gromm wrote: > This patch zero checks the function parameter 'bytes' before doing the > subtraction to prevent memory corruption. > > Signed-off-by: Christian Gromm > Reported-by: Dan Carpenter > --- > drivers/staging/most/sound/sound.c | 2 ++

Re: [PATCH] staging/mt7621-dma: mtk-hsdma.c->hsdma-mt7621.c

On Tue, Feb 02, 2021 at 11:31:17AM +0300, Dan Carpenter wrote: > Apparently this was already merged? Never mind then. Once it's merged > it can't be changed. No big stress... Sorry, yes, already in my tree... ___ devel mailing list de...@linuxdriverpr

Re: [PATCH] staging/mt7621-dma: mtk-hsdma.c->hsdma-mt7621.c

Apparently this was already merged? Never mind then. Once it's merged it can't be changed. No big stress... regards, dan carpenter ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-