Changeset 129e282

Timestamp:

2015-10-30T10:31:09Z (8 years ago)

Author:

dequis <dx@…>

Branches:

master

Children:

ad8a810

Parents:

0edc2fb

git-author:

dequis <dx@…> (27-10-15 05:43:57)

git-committer:

dequis <dx@…> (30-10-15 10:31:09)

Message:

ipc: Fix strict aliasing warnings

Turned out to be as simple as just using memcpy.

As with all strict aliasing warnings, these only appeared when
compiling with optimization, and might have caused Bad Things™

This page is cool:

​http://dbp-consulting.com/tutorials/StrictAliasing.html

File:

• ipc.c (modified) (2 diffs)

ipc.c

@@ -590 +590 @@
                                 }
 
-                                *recv_fd = *(int *) CMSG_DATA(cmsg);
+                                memcpy(recv_fd, CMSG_DATA(cmsg), sizeof(int));
                                 /*
                                 fprintf( stderr, "pid %d received fd %d\n", (int) getpid(), *recv_fd );
@@ -758 +758 @@
         cmsg->cmsg_type = SCM_RIGHTS;
         cmsg->cmsg_len = CMSG_LEN(sizeof(send_fd));
-        *(int *) CMSG_DATA(cmsg) = send_fd;
+        memcpy(CMSG_DATA(cmsg), &send_fd, sizeof(int));
         msg.msg_controllen = cmsg->cmsg_len;
 #endif