search

RfidResearchGroup / proxmark3

Iceman Fork - Proxmark3
http://www.icedev.se
GNU General Public License v3.0
3.81k stars 1.01k forks source link

Fix uint32_t overflow #2161

Closed wh201906 closed 10 months ago

wh201906 commented 10 months ago

If the timeout option of hw status is too large, the 1000 * bytes_transferred will excess the range of uint32_t and give a wrong answer. For USB connection, 20s of timeout can easily cause this bug.

github-actions[bot] commented 10 months ago

You are welcome to add an entry to the CHANGELOG.md as well

iceman1001 commented 10 months ago

You are right, there are more %llu at the device side. We only enforce it on client side.

I pushed the fix with some minor changes to variable name.