RfidResearchGroup / proxmark3

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

HitagS needs love #878

Open doegox opened 4 years ago

doegox commented 4 years ago

There are many (23) bad operations flagged in HitagS code by coverity It seems they've been solved in the official repo, at least for the parts I looked at (https://github.com/Proxmark/proxmark3/pull/721/files#diff-589b396aa9cc54b383991e0b312a9950R1053-R1057)

See the massive PR https://github.com/Proxmark/proxmark3/pull/721

iceman1001 commented 4 years ago

yeah. hitag2 / hitagS the bad child. They basically took hitag2 made a copy and adapted it to S.. So it takes a huge space and full of legacy code.