* Re: [PATCH] media: pci: ttpci: av7110: Fix possible buffer overflow in debiirq()
@ 2020-05-30 15:27 Markus Elfring
0 siblings, 0 replies; only message in thread
From: Markus Elfring @ 2020-05-30 15:27 UTC (permalink / raw)
To: Jia-Ju Bai, linux-media; +Cc: linux-kernel, Mauro Carvalho Chehab, Sean Young
> To fix this possible bug, data[0] is assigned to a local variable, which
> replaces the use of data[0].
How do you think about a wording variant like the following?
Thus assign the first element of the data array to a local variable
so that it can be used as an array index together with the data structure
member “ci_slot” in a safer way.
Would you like to add the tag “Fixes” to the commit message?
Are you going to take this possibility into account for any more patches?
Regards,
Markus
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2020-05-30 15:28 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-05-30 15:27 [PATCH] media: pci: ttpci: av7110: Fix possible buffer overflow in debiirq() Markus Elfring
Unnamed repository; edit this file 'description' to name the repository.
This inbox may be cloned and mirrored by anyone:
git clone --mirror http://archive.lwn.net:8080/linux-media/0 linux-media/git/0.git
# If you have public-inbox 1.1+ installed, you may
# initialize and index your mirror using the following commands:
public-inbox-init -V2 linux-media linux-media/ http://archive.lwn.net:8080/linux-media \
linux-media@vger.kernel.org lwn-linux-media@archive.lwn.net
public-inbox-index linux-media
Example config snippet for mirrors.
Newsgroup available over NNTP:
nntp://archive.lwn.net/lwn.kernel.linux-media
AGPL code for this site: git clone https://public-inbox.org/public-inbox.git