diff options
| author | taiyu <taiyu.len@gmail.com> | 2018-09-02 17:07:12 -0700 |
|---|---|---|
| committer | taiyu <taiyu.len@gmail.com> | 2018-09-02 17:07:12 -0700 |
| commit | d625d68d4a64ccf277663b26c52f4ca51d494e6a (patch) | |
| tree | c7ede86177a4ccdaffc9e8cc52b8d0827674b9fb /sway | |
| parent | 80270ba086ae3c46ca66f794c5a2111ad1efa1bb (diff) | |
| download | sway-d625d68d4a64ccf277663b26c52f4ca51d494e6a.tar.xz | |
prevent ub caused by misaligned stores/loads
Diffstat (limited to 'sway')
| -rw-r--r-- | sway/ipc-server.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/sway/ipc-server.c b/sway/ipc-server.c index ed710be5..fb5be27b 100644 --- a/sway/ipc-server.c +++ b/sway/ipc-server.c @@ -253,8 +253,8 @@ int ipc_client_handle_readable(int client_fd, uint32_t mask, void *data) { return 0; } - client->payload_length = buf32[0]; - client->current_command = (enum ipc_command_type)buf32[1]; + memcpy(&client->payload_length, &buf32[0], sizeof(buf32[0])); + memcpy(&client->current_command, &buf32[1], sizeof(buf32[1])); if (read_available - received >= (long)client->payload_length) { ipc_client_handle_command(client); @@ -832,8 +832,8 @@ bool ipc_send_reply(struct ipc_client *client, const char *payload, uint32_t pay uint32_t *data32 = (uint32_t*)(data + sizeof(ipc_magic)); memcpy(data, ipc_magic, sizeof(ipc_magic)); - data32[0] = payload_length; - data32[1] = client->current_command; + memcpy(&data32[0], &payload_length, sizeof(payload_length)); + memcpy(&data32[1], &client->current_command, sizeof(client->current_command)); while (client->write_buffer_len + ipc_header_size + payload_length >= client->write_buffer_size) { |