From 274b591d1867eacd9cffd6a1b38c906253023f8e Mon Sep 17 00:00:00 2001
From: Romain Vimont <rom@rom1v.com>
Date: Mon, 25 May 2020 18:41:05 +0200
Subject: [PATCH] Fix union typo

The "set clipboard" event used the wrong union type to store its text.

In practice, it worked because both are at the same offset.
---
 app/src/control_msg.c                  | 2 +-
 app/tests/test_control_msg_serialize.c | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/app/src/control_msg.c b/app/src/control_msg.c
index 252a3425..72504138 100644
--- a/app/src/control_msg.c
+++ b/app/src/control_msg.c
@@ -67,7 +67,7 @@ control_msg_serialize(const struct control_msg *msg, unsigned char *buf) {
                              (uint32_t) msg->inject_scroll_event.vscroll);
             return 21;
         case CONTROL_MSG_TYPE_SET_CLIPBOARD: {
-            size_t len = write_string(msg->inject_text.text,
+            size_t len = write_string(msg->set_clipboard.text,
                                       CONTROL_MSG_CLIPBOARD_TEXT_MAX_LENGTH,
                                       &buf[1]);
             return 1 + len;
diff --git a/app/tests/test_control_msg_serialize.c b/app/tests/test_control_msg_serialize.c
index 4dc79018..da243d91 100644
--- a/app/tests/test_control_msg_serialize.c
+++ b/app/tests/test_control_msg_serialize.c
@@ -200,7 +200,7 @@ static void test_serialize_get_clipboard(void) {
 static void test_serialize_set_clipboard(void) {
     struct control_msg msg = {
         .type = CONTROL_MSG_TYPE_SET_CLIPBOARD,
-        .inject_text = {
+        .set_clipboard = {
             .text = "hello, world!",
         },
     };