FIX: CoverityScan 123358 and 133864 - Unchecked return value

FIX: CoverityScan 121362 - Pointer to local outside scope

FIX: mental note to self, don't code when tried. "size" should have been "sizeof".   I need to fix my build for armsrc again.

FIX:  forgot a semicolon

FIX: Coverity scan 1333852, Make sure (card) is not null before accessing it.

CHG: Some testing of timings on 14443b,
FIX: CoverityScan 133854 , out-of-bounds read,  when calculating crc.

FIX: COverityScan 133865, out-of-bounds access,   Forgot that bits is not used in Logtrace but bytes.

FIX: CoverityScan 133857, cast to uint32_t will stop eventual problems.  However its a uint_16 so it shouldnt be a problem to start with.

FIX. COVERITYscan 133860,  initialize to zero.  This function is still not working but at least now it doesnt crash the client.

FIX: CoverityScan,  1333853,  out-of-bounds,  well, if I have 3 items in a array, don't start reading 4...

Merge pull request #18 from alexgrin/iceman_master

A few fixes

FIX: LEGIC - potential stack corruption calculating CRC from user input

FIX: CoverityScan 123465 - Resource leak

CHG: LEGIC - allow offline mode due to existing offline command

Merge branch 'master' of https://github.com/iceman1001/proxmark3

FIX: CoverityScan 133858 -uninitialized data

chg:  another way of checking a array for NULL

fix: CoverityScan  133855 -out-of-bounds  bits was used as index to cmd array.

Merge pull request #17 from alexgrin/iceman_master

Fixing warning:

Fixing warning:
warning: ignoring return va 'scanf', declared with attribute warn_unused_result [-Wunused-result]

Merge pull request #16 from alexgrin/iceman_master

Fixing clang compilation warning

Fixing clang compilation warning
  warning: shifting a negative signed value is undefined [-Wshift-negative-value]

FIX:  removed a compiler warning,  the stringlen is not used anymore.

minor textual fix

CHG:  Moved some includes, defines, structs from .c into header file.

FIX:  getting rid of a compiler warning message.

ADD:  started to rework the wiegand converting.  Extracted it from the 'lf hid' file.  Looks decent but still needs work.

Merge pull request #15 from alexgrin/iceman_master

Adding a 'recoverpw' command for T55xx to try to recover corrupt pass…

Adding a 'recoverpw' command for T55xx to try to recover corrupt password written by a handheld cloner by fiddling with bits in the expected password.

CHG:  moved out some help-texts into its own usage-functions.

CHG:  Moved a 'include' statement to header file

REM:  cleaning up some commented code.

FIX:   Found a minor bug in 'LF CMDREAD'  where it on device side didn't compare with the right char 'h' instead of 'H'.   Re-wrote that whole part anyway,  I changed periods to be max 0xFFFF in length,  doubtful that a zero or one delay will be bigger than 65535...

CHG:  The jablotron demod now deals with the strangeness of hex symbols A-F in cardid the same way a reader prints it.

ADD: Binary Encoded Digit -> Decimal
ADD: Decimal -> Binary Encoded Digit

Ie
  decimal 2345 becomes 0x2345
  and
  hex 0x2345 becomes decimal 2345

CHG: help text changes in cmdlfjablotron.c
CHG: increase byte size to uint16_t in crc.c CRC16Legic

chg: textual change.

CHG:  redundant 'return' statement removed

CHG: better printing output of card ID
CHG: better detecting in preambl. 16bits with 1, and a zero bit.  Also cleared this topbit when cloneing and simulating.

CHG: textual changes to help text.
CHG: removed the 'd' as input,  the clone/sim commands is simpler.
CHG: printing of card ID is now zero padded.

ADD: LF JABLOTRON functionality.  with clone/sim and detection in LF SEARCH.
CHG: removed no-existing crc8ja code
FIX: a check in ASKbiphaseDemod to make sure graphbuff is not empty.

FIX:  that OEM variable that is unused.

FIX:  Fixed some warnings I didn't take notice off.
- memcpy should have been "memset"
- forgot to add function declaration into header file.

CHG:  removed the wiegand converting,  since I'm not happy with how its implemented.  This is rework in progress.

CHG:  trying to figure out why I can't get the seconds out..

ADD:  Added some new analyse functions like CHKSUM ADD/SUBTRACKT with one's complement

spacing..

CHG:  forgot to add the reflect function

CHG: merged the forum user @jason 's fixes to LEGIC.  *UNTESTED*
CHG: changed the CRC implementations.

ADD:  some new usb commmands from EMV,  and for LEGIC.

CHG:  removed the linking to radixsort.c

Merge branch 'master' of https://github.com/iceman1001/proxmark3

Update README.md

CHG: added a comment

CHG: some clearing of memory

Merge pull request #14 from k02a/patch-2

Addition of depending library flags...

Merge pull request #13 from k02a/patch-1

Improved compilation compability on Linux

Addition of depending library flags...

Inclusion of termcap library or ncurses library solves the compilation errors on some Linux distributions, for example a currently updated Slackware Linux 14.2 x86_64 vanilla distribution (which I run).

Output:
bash-4.3# make -C proxmark3
[...]
g++ -DQT_SHARED -I/usr/lib64/qt/include/QtGui -I/usr/lib64/qt/include/QtCore -Wall -O3 obj/proxmark3.o obj/uart.o obj/util.o obj/sleep.o obj/nonce2key/crapto1.o obj/nonce2key/crypto1.o obj/nonce2key/nonce2key.o obj/nonce2key/crypto1_bs.o obj/loclass/cipher.o obj/loclass/cipherutils.o obj/loclass/des.o obj/loclass/ikeys.o obj/loclass/elite_crack.o obj/loclass/fileutils.o obj/mifarehost.o obj/parity.o obj/crc.o obj/crc16.o obj/crc64.o obj/iso14443crc.o obj/iso15693tools.o obj/data.o obj/graph.o obj/ui.o obj/cmddata.o obj/lfdemod.o obj/cmdanalyse.o obj/cmdhf.o obj/cmdhf14a.o obj/cmdhf14b.o obj/cmdhf15.o obj/cmdhfepa.o obj/cmdhflegic.o obj/cmdhficlass.o obj/cmdhfmf.o obj/cmdhfmfu.o obj/cmdhfmfhard.o obj/cmdhfmfdes.o obj/cmdhftopaz.o obj/cmdhw.o obj/cmdlf.o obj/cmdlfio.o obj/cmdlfhid.o obj/cmdlfawid.o obj/cmdlfem4x.o obj/cmdlfhitag.o obj/cmdlfti.o obj/cmdparser.o obj/cmdmain.o obj/cmdlft55xx.o obj/cmdlfpcf7931.o obj/cmdlfviking.o obj/cmdlfpresco.o obj/cmdlfpyramid.o obj/cmdlfguard.o obj/cmdlfnedap.o obj/pm3_binlib.o obj/scripting.o obj/cmdscript.o obj/pm3_bitlib.o obj/aes.o obj/protocols.o obj/sha1.o obj/sha256.o obj/cmdcrc.o obj/reveng/preset.o obj/reveng/reveng.o obj/reveng/cli.o obj/reveng/bmpbit.o obj/reveng/model.o obj/reveng/poly.o obj/reveng/getopt.o obj/tea.o obj/prng.o obj/radixsort.o obj/bucketsort.o obj/proxgui.o obj/proxguiqt.o obj/proxguiqt.moc.o -L/opt/local/lib -L/usr/local/lib -lreadline -lpthread -lm ../liblua/liblua.a -ldl -L/usr/lib64/qt/lib -lQtGui -lQtCore -o proxmark3
/usr/lib64/gcc/x86_64-slackware-linux/5.3.0/../../../../lib64/libreadline.so: undefined reference to `tgetstr'
/usr/lib64/gcc/x86_64-slackware-linux/5.3.0/../../../../lib64/libreadline.so: undefined reference to `tputs'
/usr/lib64/gcc/x86_64-slackware-linux/5.3.0/../../../../lib64/libreadline.so: undefined reference to `BC'
/usr/lib64/gcc/x86_64-slackware-linux/5.3.0/../../../../lib64/libreadline.so: undefined reference to `tgetent'
/usr/lib64/gcc/x86_64-slackware-linux/5.3.0/../../../../lib64/libreadline.so: undefined reference to `tgetflag'
/usr/lib64/gcc/x86_64-slackware-linux/5.3.0/../../../../lib64/libreadline.so: undefined reference to `tgoto'
/usr/lib64/gcc/x86_64-slackware-linux/5.3.0/../../../../lib64/libreadline.so: undefined reference to `UP'
/usr/lib64/gcc/x86_64-slackware-linux/5.3.0/../../../../lib64/libreadline.so: undefined reference to `tgetnum'
/usr/lib64/gcc/x86_64-slackware-linux/5.3.0/../../../../lib64/libreadline.so: undefined reference to `PC'
collect2: fel: ld returnerade avslutningsstatus 1
Makefile:172: receptet för målet "proxmark3" misslyckades
make[1]: *** [proxmark3] Fel 1
make[1]: Lämnar katalogen "/home/github/iceman1001/proxmark3/client"
Makefile:12: receptet för målet "client/all" misslyckades
make: *** [client/all] Fel 2
make: Lämnar katalogen "/home/github/iceman1001/proxmark3"

Improved compilation compability on Linux

Compilation of Lua on a currently updated Slackware Linux 14.2 x86_64 vanilla distribution will cause some errors, due to a missing flags in the Makefile. I noticed that adding the termcap library solves the problem.

I also found a thread (http://lua.2524044.n2.nabble.com/readline-termcap-ncurses-td5726148.html
) where other Linux distributions might have similar problems with Lua. Included version of liblua, derived from Lua 5.2.2, also seems to depend on the Termcap library and/or ncurses library.

Output:
bash-4.3$ make -C proxmark3
make: Går till katalogen "/home/github/iceman1001/proxmark3"
make -C client all
make[1]: Går till katalogen "/home/github/iceman1001/proxmark3/client"
Compiling liblua, using platform linux
cd ../liblua && make linux
make[2]: Går till katalogen "/home/github/iceman1001/proxmark3/liblua"
make all SYSCFLAGS="-DLUA_USE_LINUX" SYSLIBS="-Wl,-E -ldl -lreadline"
make[3]: Går till katalogen "/home/github/iceman1001/proxmark3/liblua"
gcc -O3 -Wall -DLUA_COMPAT_ALL -DLUA_USE_LINUX    -c -o lapi.o lapi.c
[...]
gcc -O3 -Wall -DLUA_COMPAT_ALL -DLUA_USE_LINUX    -c -o linit.o linit.c
ar rcu liblua.a lapi.o lcode.o lctype.o ldebug.o ldo.o ldump.o lfunc.o lgc.o llex.o lmem.o lobject.o lopcodes.o lparser.o lstate.o lstring.o ltable.o ltm.o lundump.o lvm.o lzio.o lauxlib.o lbaselib.o lbitlib.o lcorolib.o ldblib.o liolib.o lmathlib.o loslib.o lstrlib.o ltablib.o loadlib.o linit.o
ranlib liblua.a
gcc -O3 -Wall -DLUA_COMPAT_ALL -DLUA_USE_LINUX    -c -o lua.o lua.c
gcc -o lua   lua.o liblua.a -lm -Wl,-E -ldl -lreadline
/usr/lib64/gcc/x86_64-slackware-linux/5.3.0/../../../../lib64/libreadline.so: undefined reference to `tputs'
/usr/lib64/gcc/x86_64-slackware-linux/5.3.0/../../../../lib64/libreadline.so: undefined reference to `tgoto'
/usr/lib64/gcc/x86_64-slackware-linux/5.3.0/../../../../lib64/libreadline.so: undefined reference to `tgetflag'
/usr/lib64/gcc/x86_64-slackware-linux/5.3.0/../../../../lib64/libreadline.so: undefined reference to `UP'
/usr/lib64/gcc/x86_64-slackware-linux/5.3.0/../../../../lib64/libreadline.so: undefined reference to `tgetent'
/usr/lib64/gcc/x86_64-slackware-linux/5.3.0/../../../../lib64/libreadline.so: undefined reference to `tgetnum'
/usr/lib64/gcc/x86_64-slackware-linux/5.3.0/../../../../lib64/libreadline.so: undefined reference to `PC'
/usr/lib64/gcc/x86_64-slackware-linux/5.3.0/../../../../lib64/libreadline.so: undefined reference to `tgetstr'
/usr/lib64/gcc/x86_64-slackware-linux/5.3.0/../../../../lib64/libreadline.so: undefined reference to `BC'
collect2: fel: ld returnerade avslutningsstatus 1
Makefile:63: receptet för målet "lua" misslyckades
make[3]: *** [lua] Fel 1
make[3]: Lämnar katalogen "/home/github/iceman1001/proxmark3/liblua"
Makefile:106: receptet för målet "linux" misslyckades
make[2]: *** [linux] Fel 2
make[2]: Lämnar katalogen "/home/github/iceman1001/proxmark3/liblua"
Makefile:203: receptet för målet "lua_build" misslyckades
make[1]: *** [lua_build] Fel 2
make[1]: Lämnar katalogen "/home/github/iceman1001/proxmark3/client"
Makefile:12: receptet för målet "client/all" misslyckades
make: *** [client/all] Fel 2
make: Lämnar katalogen "/home/github/iceman1001/proxmark3"

Merge pull request #12 from k02a/patch-1

Inclusion of unused oem variable

Inclusion of unused oem variable

When compilation of the current source code, I am getting the warning of setting the unused variable "oem". I have included this, along with minor spelling corrections/language updates in the menu system.

gcc -std=c99 -O3 -mpopcnt -march=native -g -I. -I../include -I../common -I../zlib -I/opt/local/include -I../liblua -Wall -DHAVE_GUI -DZ_SOLO -DZ_PREFIX -DNO_GZIP -DZLIB_PM3_TUNED  -c -o obj/cmdlfhid.o cmdlfhid.c
cmdlfhid.c: I funktion "CmdHIDWiegand":
cmdlfhid.c:292:11: varning: variabeln "oem" sätts men används inte [-Wunused-but-set-variable]
  uint32_t oem;
           ^

Before:
pm3 --> lf hid wiegand 0 101 2001
HID 26 bit | FC: 101 CN: 2001 | Wiegand Code: 0000002004CA0FA2
[...]
HID 40 bit | FC: 101 CN: 2001 | Wiegand Code: 0000000000000FA2

After:
pm3 --> lf hid wiegand 0 101 2001
HID 26 bit | OEM: 0 FC: 101 CN: 2001 | Wiegand code: 0000002004CA0FA2
[...]
HID 40 bit | OEM: 0 FC: 101 CN: 2001 | Wiegand code: 0000000000000FA2

CHG:  added some includes / libarys for HomeBrew AND QT5 compiling on MAC OSX.  They are commented away, but if you need it replace the other two lines and uncomment.
 All credit to @koalazak  [ref] https://github.com/Proxmark/proxmark3/pull/184/commits/ed1525805c940fe738fc05286ffd22e0ce3a0fbf

CHG: minor changes to code,  mostly newlines,  change a value to #DEFINE  END_OF_LIST_MARKER
CHG: GOOD_BYTES_REQUIRED changed from 28 > 13,  makes it nonce collecting part faster, but the fail rate goes also up. its a value to fiddle with.

chg: name error?

CHG:  moved includes into header file where it belong

ADD:  added the nonce_struct needed for @marshmellow42 's enhanced "sim x" attack.

chg: changed comment

syntax suger

CHG: Now it prints the correct UID bytes etc.

CHG:  adjusted the debug message to the correct mfkey32v2 with right number of parameters.

Applied @icsom 's fixes from:  https://github.com/icsom/proxmark3/commit/b577582490431854ae977d8235b9943ab0184b4a

FIX: this should fix the compilation error for MAC users.

chg:  added to print the author.

@marshmellow42 's fix the AskEdgeDetect cleaning tool. https://github.com/marshmellow42/proxmark3/commit/de53156e1ea6982f03a6c39adf1ae17a57056132

@marshmellow42 's fix askAMP https://github.com/marshmellow42/proxmark3/commit/16ea2b8ca301b008e109528a96fd9ba12815b7a9

CHG:  wrong minor version value.  0x1 -> 0x0  for desfire ev1 detection

CHG: Thanks to @Mackwa for looking up the expected Desfire EV2 major version byte.

ADD:  added the skeleton for analysing dates in a given byte array.

ADD: added some identification on between old Desfire, Desfire EV1 and Desfire EV2
DEL: removed two old des-commands files which wasn't used.

FIX:  this corrects the bug when running "hardnested" serveral times a faulty wrongsized key is returned. Just needed to clear the variable every run.

Merge pull request #8 from zhovner/master

Typo in hf 14a sim help

Typo in hf 14a sim help

Update cmdlfawid.c

FIX:  when writing the timeout was triggered,  Increased it to make the writing part stable.

Update README.md

CHG:  printing of progress dots is better now.

ADD:  added a comment about Q5 settings. needs to be verified

ADD:  added a comment about the Q5, which I think is wrong in these commands. Needs to be tested.

ADD:  added 'lf io sim <version> <facility.code> <cardnumber>'  to the sim command
ADD:  added 'lf io clone <version> <facility.code> <cardnumber>'  to the clone command

CHG:  added an entry to copy the 77-mm-usb-device-blacklist.rules file into  /etc/udev/rules.d  and reload the rules.
All needs to be executed as root,  so I use sudo.

  pm3:>make udev

Is all that is needed to install it on Linux systems.

CHG:  added the possibility to "remagic" the new found Mifare Ultralight tags, which answers to chinese backdoor commands for uid change.
The script now deals with to different kind of tags.  Mifare Classic 1k gen1  and  Mifare Ultralight gen1 (7bytes uid)

-Classic will get    UID: 01 02 03 04
-Ultralight will get UID: 53 80 71 02 00 D9 80

ADD: added the possiblity to use AWID formatlength of 26 and 50.
 'LF AWID SIM'
 'LF AWID CLONE'
 'LF AWID BRUTE'

CHG:  Changed cardnumber printing from %d (signed) to  %u (unsigned) for AWID50,   it now prints correct in all cases. Thanks to @ntk for reminding me about it.

ADD:  AWID 50bit demod

ADD:  AWID 50bit demod,  thanks to @FrancoisM and @ntk

Update README.md

textual updates.

ADD:   Added a new main command group "analyse",  first one is the find the missing XOR in a LCR,  it was a python script under /tools/xorfind.py

CHG: output when finding key it the same as from the client.

CHG: just made the autopwn script main loop a bit more verbal

FIX:  changed from ssized_t -> size_t
FIX:  changed off_t -> size_t