Skip to content

Comments

Fix: align CMAC output parameter and initialize key_index to remove compiler warnings#1033

Open
dbarzin wants to merge 2 commits intoProxmark:masterfrom
dbarzin:master
Open

Fix: align CMAC output parameter and initialize key_index to remove compiler warnings#1033
dbarzin wants to merge 2 commits intoProxmark:masterfrom
dbarzin:master

Conversation

@dbarzin
Copy link

@dbarzin dbarzin commented Oct 16, 2025

Summary:
This patch fixes two GCC warnings:

  • cmac.c — align function parameter with header declaration
  • mifare/mifarehost.c — initialize and guard key_index to silence “may be used uninitialized” and ensure safe access.

Result:

  • No more -Warray-parameter in cmac.c
  • No more uninitialized or unsafe access warning in mifarehost.c
  • No API/ABI changes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant