use memcpy instead of re-assign pointer

clear char array with each call

fix char* pointer not assigned

fix

use different fsk field clock detection routine ...

... to prevent double call to getFromGraphBuf()...
also adjust %hu to u and as there is a possible problem with %hu in
printf.

Merge remote-tracking branch 'upstream/master' into pm3+reveng

Merge pull request #162 from marshmellow42/CoverityFixes

Coverity fixes - mainly from @iceman1001 s fork

Merge pull request #160 from marshmellow42/master

lf viking and standalone bug fixes

undo adjustments to cmdhf14a.c and cmdhfmf.c

as these will be adjusted with the magic command adjustments

clear array before assigning

verify wait isn't null

put back viking clone bug fix

forgot i added this on a different computer recently - forgot to merge
it in...

remove redundant llx prix64 defines

remove unused commented out #includes
coverity indicates compressed_fpga_stream.opaque needs to be initialized
to Z_NULL
fgetc returns int
define llu

resource leak and malloc(x) cannot be negative

thanks iceman1001

lf t55 bruteforce lots of resource leaks...

plus strlen(Cmd) can never be less than 0
iceman1001 fixes...

additional memory leaks, overflow and unchecked ...

return values fixed

thanks to iceman1001

FIX: Coverity, Out-of-bounds. In the loop, variable i, can...

...be as much as 1051, overflowing the databuf with size 1024.

fix possible null if no file found

also initialize filename variable so not null

found by iceman1001

fix keyNbr entry error checking

coverity scan by iceman1001 found.

FIX, Coverity, Argument can't be negative. CID# 212322, ftell(f) can …

…be negative. Not allowed in malloc...
from iceman1001

FIX, Coverity, Argument can't be negative. CID# 212322, ftell(f) can …

…be negative. Not allowed in malloc...
from @iceman1001

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.
from @iceman1001 .

@iceman1001 s coverity fixes

resource leak in hf mf sniff
possible overflow in hf 14a raw - add check to fix

add check to fread call

Merge pull request #161 from wllm-rbnt/master

Fix typos

Fix typos

fix bug in lf standalone mode

clone command mixed up parameters.

fix lf viking clone mask bug

should be a 32 bit mask not a 16 bit mask
also added client feedback.

Merge pull request #158 from marshmellow42/master

q5 fixes plus a couple coverity scan fixes from @iceman1001

fix print clock as decimal instead of uint8_t

some @iceman1001 s coverty scan fixes

great work!

comment out unused includes and defines

prep for permanent removal.

Merge remote-tracking branch 'upstream/master'

add broken_bad and @iceman1001 s q5 trace and ...

wipe adjustments

Remove include statement that causes trouble (fix #151).

See: https://github.com/Proxmark/proxmark3/issues/151

Merge remote-tracking branch 'upstream/master'

Merge pull request #153 from AdamLaurie/master

add support for raw block 3/4 iclass keys

document iclass raw key changes

add support for raw block 3/4 iclass keys

additional Q5 `lf t55xx` fixes

fix occasional strange printed clock value with...

...fskdemod
Fix Q5 tag detection in lf t55xx detect
Fix param_get8 reversed parameters

Merge pull request #149 from marshmellow42/T55xx_tests

Add lf viking, lf demod/clock detection improvements

Fix incorrect memset parameters in lf t55xx wipe

thanks @iceman1001 for the catch

add added a restore graphbuffer after lf t55xx psk demodulation

fix sprint_bin bug i made :(

update changelog

Fix printdemodbuffer length tests + add length...

parameter
increase askdemod MaxBits
fix util.c sprint_bin_break and increase buffer size (had memory
overflow possible before)

minor help text changes

include default password dictionary for lf t55xx...

...bruteforce

icemans lf fixes & adjustments + lf t55xx bruteforce

Fix small fskdemod clock bug

include cmdlfviking.c in makefile and cmdlf

fix minor typos

iceman's finished t55xx detect parameter adjustments

fix const vs static...

Merge pull request #147 from ha7dcd/pcb

Updates and fixes to the PCB schematics

lf viking build / lf awid refactor / lfdemod.c debugMode==2

lf viking from other users - just put my spin on it
lf awid refactored code - possible to make it not 26bit specific now
with minor chanages
lfdemod.c now supports extra debug printing if `data setdebug` = 2 when
not on device (on client not arm)

lf demod cleanup nrz clock detect fixes

slight adjustment to lf t55xx sampling code to start at the same spot
more consistently
reduce indala detection false positives
lf t55xx commands heavily tested and seem to be stable and reliable on
most modulations/bitrates (excluding Sequence Terminator configured
cards).

remove old comment

lf t55xx and some lf demod fixes/adjustments

finally think I like the lf t55xx detect and read cmds.  pretty reliable
now.

Merge pull request #143 from marshmellow42/master

lfops cleanup, t55xx additions/fixes, PCF7931 input cleanup

Merge pull request #144 from dnet/makefile-paren-fix

fixed target "help" in Makefile

Merge pull request #146 from dnet/patch-1

Fixed copypasted hex output to match input

corrected Eagle files

check change_log.txt

cad, cam file commit

adding modified CAD and CAM file sets. Checked against DRC issues, all
looks okay, ready to manufacturing!

add some Q5 functionality - add t55xx wipe cmd

not fully tested yet

Fix viking demod bug - clean my indala demod code

NOTE lf indalademod is still available as it uses a different technique
to demodulating, which sometimes works better (and sometimes worse).

add checksum to viking demod

Add Viking demod,

fix compiler warning for int vs uint8_t for em410x sim clock variable

Fixed copypasted hex output to match input

The value `x0` was obviously copypasted but not edited to reflect the real values used in `case` matching.

fixed target "help" in Makefile

Without this commit, running `make help` produced the below output.

    $ make help
    Multi-OS Makefile, you are running on Linux
    Possible targets:
    + all - Make bootrom, armsrc and the OS-specific host directory
    + client - Make only the OS-specific host directory
    + flash-bootrom - Make bootrom and flash it
    /bin/sh: 1: Syntax error: "(" unexpected
    Makefile:20: recipe for target 'help' failed
    make: *** [help] Error 2

This issue was introduced in e335ca2846d57f660ab7f4c75b58226cca264e0e
six months ago, the above output was produced on Debian GNU/Linux, where
`/bin/sh` is usually a symbolic link to `/bin/dash`.

fix em410xsim help - add clock input to em410xsim

fix: .history was never written

fixing hf snoop:
- add: clear the trace buffer before snooping
- fix: allow trigger on odd samples as well
- fix: don't use Dbprintf() in timing tight loops

Merge remote-tracking branch 'upstream/master'

minor lf updates/cleanup

Merge branch 'topaz'

Conflicts:
CHANGELOG.md
client/cmdhf.c

Merge pull request #141 from etmatrix/tosubmit

HF snoop ported by enio's repository

- Little change to description in client
- Added description to CHANGELOG.md

- Correct little distraction on fpga/Makefile
- New patch from pwpiwi works very well for hi_sniffer.v
- Restored previous fpga_lf.bit

minor cleanups

update changelog

Add lf t55xx resetread cmd + fix clone cmds

resetread cmd to determine start of streaming bits of ata5577 or
compatible chips...
fixed lf clone bugs introduced while refactoring recently...

Merge branch 'master' into topaz. Update Changelog.

LED D is on while snoop, when user button is pressed snoop is stopped

refactor lfops t55xx functions

share t55xx configuration register definitions with client for later use
(warning - compiled but not fully tested yet)

update changelog

text cleanup t55xx cmds

The great work of Enio hf snoop is now ported into latest version in git
you can find original work here https://github.com/EnioArda/proxmark3

updates to lf t55xx commands

fix wakeup cmd per @iceman1001
fix dump and read cmds
clean up write command
allow page 1 read/write (block 3 is writable)(ata5577 only)
remove duplicate code

add wake option to t55xx read command

begin update lf commands with help text and

cmd structure.
update changelog for recent changes

add lf t5 read w/pwd warning and checks

ideas from @iceman1001 for issue #136

lf t55xx code cleanup +

@iceman1001 s cleanup of some t55xx read code, plus some timing tests to
improve lf t5 detect/read...
small typo in pcf 16 address bytes instead of 4...
add option for verbose or not on bigbuf clear cmd.

PCF-fix bytepos range from 0-3 to 0-15

adjust lf t55xx read commands

seems to make them a lot more accurate for me

Clean up pcf7931

@iceman1001 s cleanup of pcf7931 commands on client side
and split out pcf7931 from lfops.c as it is getting large

Merge pull request #140 from marshmellow42/iclass

iClass major updates

hf mf mifare: (finally) fix watchdog reset
- minor changes to sync
- try alternative strategies when debugging

fix changelog conflict

Merge remote-tracking branch 'Proxmark/master' into iclass

Conflicts:
CHANGELOG.md

fix help typo

Add mifare crypto trace decryption utility

allows manual decryption of hf 14a snoop traces of a mf card.
someday we should fix hf mf sniff...

hf mf mifare:
- gracefully exit on unsuccessful syncs instead of hard watchdog reset