Messages in this thread | | | Subject | Re: [PATCH] media: pci: ttpci: av7110: Fix possible buffer overflow in debiirq() | From | Markus Elfring <> | Date | Sat, 30 May 2020 17:27:48 +0200 |
| |
> 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
| |