On 08/18/2017 03:05 AM, Dan Carpenter wrote:
This is a static checker fix.  "cal_num" is 10.  We're declaring the
tx_dt[] and rx_td[] arrays as 3 element arrays.  The static checker
complains that we do:

        tx_dt[cal] = (vdf_y[1]>>20)-(vdf_y[0]>>20);

"cal" is the iterator and it is in the 0-9 range so it looks like
we could corrupt memory.

Signed-off-by: Dan Carpenter <[email protected]>
---
I'm pretty sure this patch is correct and absolutely harmless.  But the
code is pretty involved and I didn't test it.  So you may want to review
this one carefully.

I believe the patch is correct. In testing, I also confirmed that this branch is executed, thus the memory corruption does happen.

Acked-by: Larry Finger <[email protected]>

Thanks,

Larry

Reply via email to