cmdwiegand: skip "parity tests failed" message, if no parity tests failed

jkramarz · jkramarz · commit b65797bf6391 · 2025-09-23T01:01:50.000+02:00
diff --git a/client/src/wiegand_formats.c b/client/src/wiegand_formats.c
@@ -1688,10 +1688,12 @@ bool HIDTryUnpack(wiegand_message_t *packed) {
                      );
     }
 
-    if (packed->Length && ((found_cnt - found_invalid_par) == 0)) { // if length > 0 and no valid parity matches
-        PrintAndLogEx(FAILED, "Parity tests failed");
+    if(found_cnt > 0){
+        if (packed->Length && ((found_cnt - found_invalid_par) == 0)) { // if length > 0 and no valid parity matches
+            PrintAndLogEx(FAILED, "Parity tests failed");
+        }
+        PrintAndLogEx(NORMAL, "");
     }
-    PrintAndLogEx(NORMAL, "");
 
     return ((found_cnt - found_invalid_par) > 0);
 }

Original file line number	Diff line number	Diff line change
`@@ -1688,10 +1688,12 @@ bool HIDTryUnpack(wiegand_message_t *packed) {`
`1688`	`1688`	`);`
`1689`	`1689`	`}`
`1690`	`1690`
`1691`		`- if (packed->Length && ((found_cnt - found_invalid_par) == 0)) { // if length > 0 and no valid parity matches`
`1692`		`- PrintAndLogEx(FAILED, "Parity tests failed");`
	`1691`	`+ if(found_cnt > 0){`
	`1692`	`+ if (packed->Length && ((found_cnt - found_invalid_par) == 0)) { // if length > 0 and no valid parity matches`
	`1693`	`+ PrintAndLogEx(FAILED, "Parity tests failed");`
	`1694`	`+ }`
	`1695`	`+ PrintAndLogEx(NORMAL, "");`
`1693`	`1696`	`}`
`1694`		`- PrintAndLogEx(NORMAL, "");`
`1695`	`1697`
`1696`	`1698`	`return ((found_cnt - found_invalid_par) > 0);`
`1697`	`1699`	`}`