Merge pull request #43 from al-heisner/master

polhenarejos · web-flow · commit bb641ee134b0 · 2024-06-21T21:05:27.000+02:00
Fix for multiples of 64 bytes on cmd_list_keys
diff --git a/src/hsm/cmd_list_keys.c b/src/hsm/cmd_list_keys.c
@@ -60,5 +60,9 @@ int cmd_list_keys() {
             res_APDU[res_APDU_size++] = f->fid & 0xff;
         }
     }
+    if ((apdu.rlen + 2 + 10) % 64 == 0) {     // FIX for strange behaviour with PSCS and multiple of 64
+        res_APDU[res_APDU_size++] = 0;
+	res_APDU[res_APDU_size++] = 0;
+    }
     return SW_OK();
 }

Original file line number	Diff line number	Diff line change
`@@ -60,5 +60,9 @@ int cmd_list_keys() {`
`60`	`60`	`res_APDU[res_APDU_size++] = f->fid & 0xff;`
`61`	`61`	`}`
`62`	`62`	`}`
	`63`	`+ if ((apdu.rlen + 2 + 10) % 64 == 0) { // FIX for strange behaviour with PSCS and multiple of 64`
	`64`	`+ res_APDU[res_APDU_size++] = 0;`
	`65`	`+ res_APDU[res_APDU_size++] = 0;`
	`66`	`+ }`
`63`	`67`	`return SW_OK();`
`64`	`68`	`}`