Fix order of EV_* for CCID thread.
authorNIIBE Yutaka <gniibe@fsij.org>
Thu, 6 Dec 2018 23:35:27 +0000 (08:35 +0900)
committerNIIBE Yutaka <gniibe@fsij.org>
Thu, 6 Dec 2018 23:35:27 +0000 (08:35 +0900)
Since openpgp thread has lower priority than CCID thread, there is no
case where two consecutive messages EV_EXEC_ACK_REQUIRED and
EV_EXEC_FINISHED are handled simultaneously (openpgp thread needs to
work some time to send EV_EXEC_FINISHED, after sending
EV_EXEC_ACK_REQUIRED).  But just in case, when there are two messages
at once, EV_EXEC_ACK_REQUIRED should been handled before
EV_EXEC_FINISHED.

ChangeLog
src/gnuk.h

index c4ff511..477d907 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2018-12-07  NIIBE Yutaka  <gniibe@fsij.org>
+
+       * src/gnuk.h (EV_EXEC_ACK_REQUIRED): Have precedence
+       than EV_EXEC_FINISHED.
+
 2018-12-06  NIIBE Yutaka  <gniibe@fsij.org>
 
        * src/usb-ccid.c (ccid_thread): Priority of handling
index 383241b..2082c3d 100644 (file)
@@ -25,8 +25,8 @@ void ccid_card_change_signal (int how);
 
 /* CCID thread */
 #define EV_RX_DATA_READY      1 /* USB Rx data available  */
-#define EV_EXEC_ACK_REQUIRED  2 /* OpenPGPcard Execution ACK required*/
-#define EV_EXEC_FINISHED      4 /* OpenPGPcard Execution finished */
+#define EV_EXEC_FINISHED      2 /* OpenPGPcard Execution finished */
+#define EV_EXEC_ACK_REQUIRED  4 /* OpenPGPcard Execution ACK required */
 #define EV_TX_FINISHED        8 /* CCID Tx finished  */
 #define EV_CARD_CHANGE       16