FIX:  'hf 14a reader' - when card SAK was 0x00, it calls GetHF14AMfU_Type()  to try to identify if it is a UL/NTAG etc. The bug is that it ignored the return value.
when return_value == UL_ERROR,  it shall not print the mfu tagtype annotation.

---faulty behavior
proxmark3> hf 14a reader
 UID : 65 93 7f d1
ATQA : 00 04
 SAK : 00 [2]
Tag is not Ultralight | NTAG | MY-D  [ATQA: 00 04 SAK: 00]

TYPE : MIFARE Ultralight (MF0ICU1) <magic>
proprietary non iso14443-4 card found, RATS not supported
Answers to chinese magic backdoor commands: NO

CHG: 'hf iclass replay' added help text.
CHG: 'hf iclass snoop'  added help text.
CHG: 'hf iclass reader' added help text.
ADD: 'hf iclass reader' added the possibility to read only one tag instead of looping.
CHG: 'sprint_hex_ascii' function now replaces unprintable chars with '.',   added this call to printIclassDumpInfo

CHG:  'hf iclass decrypt'  - adjusted the loops, to only decrypt Application 1.  However I've noticed not all blocks in Application 1 is encrypted. :/  Still needs to be adjusted.
CHG: 'hf iclass reader'  - added some output accessrights from the CopyReader source code.

And ofcourse,  moved around stuff,  like the usages in cmdhficlass.c

CHG:  adding the HID wiegand calcs again.  Still need to set the bit37 indicator etc.

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

CHG: coverity complains about not reading the value from mifare_send_short
CHG: ubuntu 14.04 gcc4.8.4 complains about mem_avail still. Don't know why.

Update README.md

added instructions for installing the blacklist / udev rules for Ubuntu

CHG:  forgot to remove unused function.

CHG: looks bad but works.

CHG: missing file extension.

CHG: textual changes.
CHG: remove unused #includes  and added some comments

CHG: some indala output is now only in debug mode
CHG: added 34|37 bits AWID format.  I'm guessing 37bit is wrong since the parity is still there. Which it shouldn't

CHG:  change the indala output abit,  to only show when in debug mode.  (ie:  data setdebug 1)

ADD: 'lf animal' commands.  CLONE/SIM/READ/DEMOD  of FXD-B animal tags. Still some work left.

syntax suger, some tabs fixed

CHG: 'analyse hid' added @holiman 's permute functions to the output to verify

Merge pull request #63 from micolous/log-nonce

nonce2key: printf->PrintAndLog

CHG:  cleaning up stuff

CHG: added a key B wien,

fixed warning about size_t

CHG:  warning needs %lu string formatter

fix a warning

CHG: size_t is unsigned. switched to the correct string formatter

LF sim

 `lf sim` - use config values for setting the frequency.

nonce2key: printf->PrintAndLog

Merge pull request #62 from micolous/fix-includes

Fix some missing include statements which impact using proxmark3 client as a library

Fix some missing include statements which impact using proxmark3 client as a library.

ADD: new found cloner pwd.

CHG: used @marshmellow42 's version to fix lf continuous demod problems.

CHG: made some debugs statements more clear,  when "LF SEARCH" is used with "DATA SETDEBUG 1"

CHG: 'lf awid brute' - extra check if device gotten offline
CHG: 'lf t55x7 brute' - extra check if device gotten offline
CHG: 'lf nedap' - Nedap is supposed to be Diphase.

CHG: mental note to self,  capslock is bad.

FIX: 'LF IO FSKDEMOD' - no call to turn of the antenna when exit
Thanks to @samyk

https://github.com/Proxmark/proxmark3/issues/203

FIX: 'lf hid fskdemod' - no turning of the antenna afterwards.
FIX: 'CMD_AWID_DEMOD

Thanks to @samyk
ref: https://github.com/Proxmark/proxmark3/issues/203

ADD some documentation of changes

CHG: 'LF VISA2000' adding askedgedetection when decoding allows for some traces to be correct demodulated.

CHG: 'LF VISA2000 CLONE' fixed the wrong blocknum index in output.  Thanks @martin for pointing it out.

ADD: 'LF VISA2000'  added one checksum test when demod / clone / sim
CHG: 'LF NORALSY' when chksum fails, return 0,  to indicate for LF SEARCH that it failed.

ADD: 'analyse chksum' added a nibble xor calc - calcSumNibbleXor

CHG:  enhanced the debug output for some LF demod/decode

ADD: 'LF NORALSY'  clone/sim/read  commands.   Partly done.  some crc works,  but whole format in not mapped yet.
ADD: 'LF SEARCH'  added noralsy detection

ADD:  a Noralsy detection,  looking for preamble 0xBB0

FIX: 'lf presco'  demod fixes.
ADD: 'lf search' now also looks for Presco.

I know that the helptext is wrong (d should be p)

ADD: forgot to add the files

Syntax suger

ADD:  Visa2000 demod

ADD:  'lf visa2000'  commands.  (SIM/CLONE/READ)  almost finished.
CHG: 'lf xxxxx'  reduced number of samples from 30000 -> 20000 in "lf xxxxxx read" commands.
CHG: 'data samples'  - didn't honor the silent parameter. It now output less for the 'lf xxxxxx read' commands.

CHG: 'lf search' changed output to be visible when debugging for IO prox

CHG:  syntaxt suger

FIX:  'hf iclass dump'  / 'hf iclass readtagfile' - the faulty output from these commands are now fixed.

CHG:  'hf iclass dump' - output fixes
CHG:  'hf iclass readtagfile' - output fixes

REM:  removed a double entry.  it seems 'hf mf check' doesnt uniq-sort this file.

FIX:  'iclass write'

FIX: 'iclass write'

thanks to prof_abrasive && go_tus
ref:  http://www.proxmark.org/forum/viewtopic.php?id=4033

CHG:  got insperation from https://github.com/menshiyun/proxmark3/commits/local    and the clean variable.

CHG:  added a check if err variable is NIL.

FIX:  the changes to uart.c  timings seems to have fixed my problem with the pm3 device getting unresponsive.

CHG: "script run hard"  now iterates all sectors on the tag and output the table style like "hf mf chkkey" do.

ADD: added the 'hf mf hardnested'  to be called within LUA scripts.
ADD: 'script run hard' - a first try for a lua script to run the hardnested attack on a complete tag.

CHG: Cleaning up

ADD:  "analyse hid" -  new function that implements the 'heart of darkness' hid/iclass permute function. Its converted from the php-solution found in the paper to C. Ref: https://github.com/akw0088/HID-Card-Copy/blob/master/key-permutation/permute.php

FIX:  @matrix latest fixes

FIX: "hf mf hardnested"  - removed the call to free_candidates_memory,  on my ubuntu env it crashes all the time with it.

FIX:  that time.h issue is different on POSIX systems and WINDOWS system and in C...

FIX: wrong compile define used,   __WIN32 should be _WIN32

FIX: & 0xFF instead of uint8_t

FIX:  removed some warnings about time_t in non-windows systems.  This appeared since I fiddled in proxmark.h

CHG:  added the ping command to the header file.

CHG: "hf mf hardnested" - fixes and additions.
  - freeing candidate lists after generate_candidates calls.
  - longer timeout when waiting for responses (it takes a while to collect 58 nonces per call) From 3sec to 6sec
  - if best_first_byte[0] (best guess) has been the same for 3 nonces calls in a row, it enters the generate_candidates test.
  - when total_added_nonces increases but does not enter generate_candidates tests,  it now increases the threshold_index variable. Make the output look better

Known bugs still.
   - TestIfKeyExists sometimes crashes the client,  still after the null check.
   - proxmark3 device doesn't answer calls after entering brute_force call and fails finding a key,  where it should start collecting nonces again. This bug doesn't make sense.

CHG: textual change

CHG: "hf mf hardnested"  device side should empty bigbuff?

CHG:  the reset of pcb_num should be before untraditional tags return.

CHG: "hf mf hardnested"
 - latest clean up from @matrix
 - the device still doesnt answer when brute_force call fails. I've been trying to get the device to init after the brute_force call.

chg: removed a useless clearing of key_count. From @matrix https://github.com/matrix/proxmark3/commit/090682764b3d18cdc7982a3aa384c8cff7e0bf8c

FIX: 'LF PYRAMID' the crc8_MAXIM and crc16_DNP was calling the wrong crc method. (update2 is the older and correct version).   This solves the pyrmid issue with wrong checksums calculated.
Thanks to @rookieatall  @marshmellow42 for pointing out the bug. ref: http://www.proxmark.org/forum/viewtopic.php?id=4006

FIX: 'hf mf hardnested' @matrix  https://github.com/iceman1001/proxmark3/commit/e0828439bfebdcf16595e7cda76e40e21f4feeb6

CHG: "hf mf hardnested"  disabled the tracelogging on deviceside during nonce acquiring.

FIX: decrease 2^39 -> 2^38. its a big searchspace anyway.
FIX: changed output and rearranged collecting nonces logic.

Still problems with "hard" keys,  the device stops responding after a "bruteforce" / "generate_candidates" call.. Very strange. shouldnt'

FIX: "hf mf hardnested"  when "key found" exting the do-while loop doesn't need to wait for device to respond.

FIX: "hf mf hardnested" merging of @matrix commit https://github.com/matrix/proxmark3/commit/bd8249afeccb0478d8dbbfb49ed68c6306182f1f

FIX:  @matrix https://github.com/matrix/proxmark3/commit/869a03c2c6267db16cd1418b9e5f2b9049b9a015
it still counts down the good bytes,
and I fixed the elapsed time.

CHG: removed unused variable

CHG:  don't consider the respons at all when sending the HALT command.

CHG: still issues left.

FIX:  several calls to nonce2key/nonce2key_ex has problems with not clearing up memory pointers laying around.
Still exists problem which needs to be dealt with.

CHG: minor textual change to fit the minimum two calls nature for the zero parity attack

FIX:  the time_t calls under mingw needs a #define _USE_32BIT_TIME_T 1  to be correct.  It seems to work in "hf mf mifare"  but not in "hf mf hardnested"

FIX:  'hf mf mifare'  - special zero parity attack vector now works. Thanks to the dude who figured this vector out: @douniwan5788  (sorry for comments, I was clearly wrong.)  @piwi - for proving me wrong.
this version uses int64_t  (signed)  to signify end-of-lists (-1). It also needs its own compare function for the qsort.  I didn't merge this into existing code which uses uint64_t. (too lazy)

Merge pull request #51 from matrix/master

Improved Mifare Plus Atack (hf mf hardnested) - part 2

Merge remote-tracking branch 'upstream/master'

CHG: 'HF MF C*' (chinese backdoor commands) According to douniwan5788 some magic/clone tags answers to the halt cmd and some not.  I think I captured his ideas.

ref: https://github.com/douniwan5788/proxmark3/commit/13b71e58fddf20c5d42b8f0af1d72c795139b86f

CHG: clean up

CHG:  just some parameter / variable name changes.  Nuttin' special.

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

FIX:  looping logic error,  doesn't need to increase with 1, if we do 8 checks every turn...

Merge pull request #49 from micolous/less-verbose-mfsim

hf {14a,mf} sim: Be less verbose by default, add option "m" to turn maths back on (Issue #45)

hf {14a,mf} sim: v is for verbose

hf mf sim: Be less verbose by default, add option "m" to turn maths back on (Issue #45)

Merge pull request #50 from micolous/mfkey-gitignore

Add .gitignore file to mfkey for binaries

Add .gitignore file to mfkey for binaries

fixup code

Merge pull request #47 from osysltd/patch-4

Update README.md

FIX:  too small string?