REM:  removed unused code.

FIX: Moved a free() call.

Experimeting with finding the "WDT_HIT" bug in "Hf mf mifare",  which not shows if "hf mf dbg" is set to 3 or more...

FIX: Possible usage of NULL.

CHG: Syntax suger

CHG: Syntax suger

CHG: removed a debug statement

CHG:  Added the urls to the new Wiki on github inside the flasher.c usuage text.
CHG:  Syntax suger

CHG:  Added some more identifications to token list,  courtesy from a forum user

ADD:  added the possibility to choose which block num to attack with "hf mf mifare"   Before it only worked on block 0.

FIX: Some uninitialized variables, some syntax suger,  and some extra WDT_HIT calls inside "ReaderMifare" to stop it from trigger when running multiple times in a row.

CHG:  Remake the "hf mf chkkeys"  It now prints the found keys like  "hf mf nested" does. It also tries to read key B if Key A was found. It also continues to the next key to find,  when there is one match. Earlier all keys in .dic file, was tested even if it was found.

CHG: syntax suger,  text edit

CHG: Syntax suger

FIX: Coverity scans,  uninitialized variables,
CHG: unfolded some loops..

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

FIX: Coverity Scan complains about resourceleaks, array reads outside of index, uninitialized variables.

FIX: Coverity Scan complains that the realloc might fail..

FIX: Coverity Scans complain about uninitialized usage of struct. Setting s->ODD and s->EVEN to zero should do the trick.

CHG,  modified the configuration script

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

FIX: #lld -> %#PRIu64"  since the arguments are uin64_t

FIX: Coverity scan,  forgot one fread ...
CHG: Syntax suger

ADD: Some instructions for configure CoverityScan using alternative compiler like the ARM-NONE-EABI-GCC
FIX: stupid clock() == -1 on systems who doesn't have it..

FIX: Coverity Scan,  #if - else -#endif statement most likely one row row.

FIX: Coverity Scans,  z_stream.opaque not initialized,  use Z_NULL;

FIX: Coverity scan, complain on z_stream.opaque is not initialized.  Google it,says use: Z_NULL

FIX: Coverity Scan warnings on not using the fread return value.

CHG: Syntax suger,   space -> tabs. etc.
FIX: Coverity Scan says out-of-bounds,  Increased the databuffer. Legic List command needs to be re-done.

FIX: Coverity Scan says fgetc() returns a INT,

CHG:  syntax suger

REM:  removed a print. The printing is done in the calling function.

CHG: Syncronized so all different parts uses the same implementation of Crapto1 v3.3

FIX: Coverity Scan resource leak

CHG: started to clean up the crapto1 imp in client/nonce2key/ folder.

CHG: minor change initializing array.

FIX: CoverityScan complains on getchar(),  not reading the returning value

CHG:   The nonce from "hf mf nested" is fixed to this one,  why is that one selected?  Why not just all zeros or ones?

CHG: Syntax suger

FIX: Coverity scan,  remove a resource leak,  wrong varible type.

REM: Removed  #define llx PRIx64,   all these defines are located in proxmark3.h

ADD: Wrote a little help text to the nonce2key tools.

CHG:  Crapto1 v3.3 for the ARMSRC

CHG:  Crapto1 v3.3  now with comments,  and I've tried to unnest some loops.

ADD: another key found

FIX: the "make clean" for mfkey/Makefile now deletes the executables.

FIX: makeing sure the "make clean" now deletes the nonce2key.exe file.

ADD:  a markdown file for the readme.txt file.  I want to be able to show the build status from TravisCI and Coverity Scans

CHG:  Reversed @piwi 's changes to parity.  I'm getting assembler compiler errors.

ADD:  added the "llu" macro shortcut as format specifier.

CHG: changed output for the "hf mf nested"  to show KEY: A or B instead of 0/1

FIX: Coverity fix with resource leakage

ADD: Implemented a addon to "hf mf nested"  if there is KEY_A 's found but not the same KEY_B,  try just read it from tag.
Makes the command a bit slower when it does these reads.   Plus some annoying debug statements,

CHG: changed the output for "hf mf nested"  to write Key A or B instead of 0 & 1..

FIX: Coverity Scan fixes, a bunch of them.

CHG:  Updated the crapto1 imp to latest vesion3.3 ( Thanks @blapost !)

FIX: forgot a lua object for the call to ConvertAsciiToHex.   Thanks @romaric for finding it.

CHG:  lets see if I can get the perl script to work if I request "git fetch --all" before running the make command in Travis-CI

FIX: one of these days I will start being more observant about argument order

FIX: wrong format specifier syntax

ADD: some clocking for   'hf mf mifare',  'hf mf nested',  'hf mf chk'

REM: removed a dublett at end of file.

FIX: this should remove some warnings while compiling on linux and when running Travis-CI in a docker container.
some format specifier for uint64_t
and getting the git history in the perl script.

CHG: Tried to unfold some loops inside ARM/crypto1 imp. It can't hurt,  feels stabil.

FIX:  Some <= fix for pwpivi 's hardnested suggested on the Pm3 forum.

CHG: Tested to unfold some loops inside the crypto1 implementation to see if it generates some speed ups.  Feels ok.

FIX: Coverity, out-of-bounds read,   124 is less than (0x10*8)+2 (130)
FIX: Coverity, dead-code, bitcomparison always false.   Copy-paste error?  but  0x0F can never be equal 0x10..  I guessing its 0x01. @pwpivi have to correct me if its wrong.

FIX: Coverity,  out-of-bounds read ...

FIX: Coverity,  resource leak, and variable can't be negative,

FIX: GCC 4.8 complains the pointer is set before its used in a call to free().  Doesn't complain on Mingw,  only Ubuntu.15.10

FIX: Coverity,  strlen(Cmd) can never be lesser than zero.. *douh*

FIX: Coverity, resource leaks 'nd more.. in "lf t55xx bruteforce"  nasty piece of code...

FIX:  forgot to change a modified call from @pwpivi 's last update.

ADD: @pwpiwi 's latest changes to 'hf mf hardnested'

REM:  Removed lot of obselete code from before.

ADD:  "lf hid bruteforce"  for a simple bruteforce attact against a HID reader.  *work in progress*

FIX: tcgetattr for Linux in Ukbhit(), should return -1 if fail.

FIX: Coverity, Unchecked return value,  CID #121288,   lets do the same check that is everywhere is this call is used.

FIX: Coverity, CID# 121351, #121371, #121372,     old code that is not used any more,  I should remove this.

FIX:  textual changes

FIX: Coverity, resource leak, CID #121357,  Mat needs to be free

FIX: Coverity, logical vs bitwise operator,  remove the extra '&' for it to become bitwise.

FIX: Coverity, Unintended sign extension, data[7] would have become int, then uint64_t.  Should work better now with adding typecasting.

FIX: Coverity, Resource leak, CID #121361,  filehandle f needs to be free

FIX: Coverity, Resource leak,  CID# 121360,   keyBlock needs to be free

FIX: Coverity, out-of-bounds write, CID#121340, CID#121341, CID#121342, CID#121343,   wrong size in check, sprintf always adds a null terminator,  so if filepath would have been 996 chars long, this might had happend... but no more.

CHG: Syntax suger

FIX: Coverity, CID#121314,  Explicit null dereferenced,  in really odd occasions buf would be NULL, and sending NULL to memcpy dereferences it.  Not sure about this fix.

FIX: Coverity, CID# 121337,  Out-of-bounds.  In the loop, variable i, can be as much as 1051, overflowing the databuf with size 1024.

FIX: Coverity,  uninitialized scalar variable,  filename array could be NULL..

FIX: Coverity,  CID #121346,  resouce leak,   close filehandle.

FIX: Coverity, out-of-bounds, CID#121330, CID#121331, CID#121332, CID#121333,
 keyNbr has to be smaller then ICLASS_KEYS_MAX (since the Iclass_Key_Table array is initialised with it).

FIX: Coverity, Dereference null return,  CID #212329,  filehandle could be NULL

FIX, Coverity, Unsigned compared against 0.  CID #212326,  keyNBr will never be negative.

FIX, Coverity, Argument can't be negative,  CID #212324,  ftell(f) can be negative,  not allowed in malloc.

FIX: Coverity , Argument can't be negative,  CID #121323,   ftell(f) can be negative, not allowed in malloc.

FIX: forgot to close the filehandle :(

FIX, Coverity,  Argument can't be negative. CID# 212322,    ftell(f) can be negative. Not allowed in malloc...

CHG:  syntax suger

FIX: Coverity, Identical code for different branches,  CID #121315,     added a message and different return value.

FIX: Coverity, unintended sign extention, CID #121363,   (numbits << 16) becomes int,  then uint64_t.  But the signness might set all upper bits to 1 in the process.

FIX: Coverity, unchecked return value, CID #121292,..
basicallty the flush queue commmand is replaced with clearCommandBuffer();.

FIX: Coverity, out-of-bounds write, CID# 121336,  s_index should take factor in consideration when looping.  Not sure about this one.

FIX: another thing struck me, the g_index wasn't increased, meaning the "un-decimation" always worked on the same first byte of GraphBuffer.