From 33db73516dd983734e6aba24ff1e861b6af5c7e3 Mon Sep 17 00:00:00 2001 From: iceman1001 Date: Tue, 12 Jan 2016 23:42:42 +0100 Subject: [PATCH 1/1] FIX: Coverity, logical vs bitwise operator, remove the extra '&' for it to become bitwise. --- client/cmdhfmfdes.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/client/cmdhfmfdes.c b/client/cmdhfmfdes.c index b8db1cab..9ab5a4af 100644 --- a/client/cmdhfmfdes.c +++ b/client/cmdhfmfdes.c @@ -352,7 +352,7 @@ void GetKeySettings( uint8_t *aid){ PrintAndLog(" Can't read Application Master key settings"); } else { // Access rights. - uint8_t rights = (resp.d.asBytes[3] >> 4 && 0xff); + uint8_t rights = (resp.d.asBytes[3] >> 4 & 0xff); switch (rights){ case 0x00: str = "AMK authentication is necessary to change any key (default)"; -- 2.39.2