On Tue, 21 Jul 2020 at 16:19, Alistair Francis <alistai...@gmail.com> wrote: > > On Tue, Jul 21, 2020 at 6:38 AM LIU Zhiwei <zhiwei_...@c-sky.com> wrote: > > > > Signed-off-by: LIU Zhiwei <zhiwei_...@c-sky.com> > > Reviewed-by: Alistair Francis <alistair.fran...@wdc.com>
Interestingly Coverity's latest scan has decided all these issues are 'fixed', even without the assert. I guess that the online version of Coverity has had an improvement to its checking and so is now able to figure out that it's not going to overrun the array? Still I think the assert is worth having. thanks -- PMM