Feat/merge fixes from ledger pr (#165)

* FIXME: a mix of changes keep in mind useful function in there to print nested clarity vales in std

* Remove job which is not use across other apps

* Remove dead code in publicKey function

* Add SIGN_JWT_SECP256K1 apdu command documentation

* Add INS_GET_AUTH_PUBKEY apdu command documentation

* Fix fuzzer and add corpus

Fix overflow

Remove redundant target as it is already covered by transaction target

Return None in case of error

* Ignore fuzzing boilerplate

* Bump version and update snapshots

* Remove test use to debug that data used was invalid

Author: neithanmo

.github/workflows/codeql.yml

@@ -78,7 +78,11 @@ app/obj
 
 app/glyphs/glyphs.*
 
-hfuzz/hfuzz_workspace
+app/hfuzz-parser/hfuzz_workspace/transaction/input/*.honggfuzz.cov
+app/hfuzz-parser/hfuzz_target
+app/hfuzz-parser/hfuzz_workspace/transaction/*.TXT
+app/hfuzz-parser/hfuzz_workspace/transaction/*.fuzz
+
 
 tests_zemu/snapshots-tmp
 app/output/*.apdu

.gitignore

@@ -44,3 +44,8 @@ test_all:
 prod:
 	make PRODUCTION_BUILD=1
 
+rust_fuzz:
+	cd app/hfuzz-parser/ && cargo hfuzz run transaction
+
+
+

Makefile

@@ -21,19 +21,20 @@ cargo install honggfuzz
 Start fuzzing a desirable target:
 
 ```bash
-cd hfuzz-parser && cargo hfuzz run transaction
+make rust_fuzz
 ```
 
 ### Debugging
 
 If there are crashes reported, debug the application with the generated input to figure out where the problem is.
-The supported debuggers are: *lldb*, *rust-gdb*, *gdb*, *rust-lldb*. By default *rust-lldb* is used  but,
-this can be changed using the __HFUZZ_DEBUGGER__ env variable:
+The supported debuggers are: _lldb_, _rust-gdb_, _gdb_, _rust-lldb_. By default _rust-lldb_ is used:
 
 ```bash
-export HFUZZ_DEBUGGER="rust-gdb"
 cd hfuzz-parser
-HFUZZ_BUILD_ARGS="--features baking" cargo hfuzz run-debug transaction hfuzz_workspace/transaction/*.fuzz
+cargo hfuzz run-debug transaction hfuzz_workspace/*/*.fuzz
+
 ```
 
-*note*: There could be more than one *.fuzz* file.
+This will deploy a gdb console with a backtrace with the first crash
+
+_note_: There could be more than one _.fuzz_ file.

app/FUZZING.md

@@ -1,3 +1,3 @@
 APPVERSION_M=0
 APPVERSION_N=24
-APPVERSION_P=2
+APPVERSION_P=3