> On Tue, Mar 2, 2021 at 9:34 AM <dinghao.liu@xxxxxxxxxx> wrote: > > > On Mon, Mar 1, 2021 at 1:20 PM <dinghao.liu@xxxxxxxxxx> wrote: > > > > > On Mon, Mar 1, 2021 at 9:44 AM Dinghao Liu <dinghao.liu@xxxxxxxxxx> wrote: > > ... > > > > > This issue is reported by my static analysis tool, so I don't have the > > > > vulnerable input currently. > > > > > > Should we blindly follow everything that some (non-ideal) tool > > > reports? I don't think so. > > > For all my experiments with that hardware, I haven't heard about the > > > issue with NULL pointers. Useless checks make code harder to read and > > > CPU to waste cycles. It might be maintainers of this driver consider > > > otherwise, so not my call. > > > > > > > Thanks for your advice. I also checked all use of ata_qc_from_tag() in the > > whole kernel and found all of them had return value checks except for the > > calls in sata_dwc_isr(), which is odd. > > Thanks for this information, it makes sense to me. Perhaps you need to > put this into the commit message to justify the need of the change. > OK. I will fix this and send a new patch soon. Regards, Dinghao
