diff options
| author | Juha-Pekka Heikkila <juhapekka.heikkila@gmail.com> | 2019-10-09 16:33:58 +0300 |
|---|---|---|
| committer | Juha-Pekka Heikkila <juhapekka.heikkila@gmail.com> | 2019-10-15 16:31:40 +0300 |
| commit | 93f306e43c946ba0748c5ed0e4549ead52b870a6 (patch) | |
| tree | a2d42f2f7958ec508051f6bbb481bb43d2167058 /tests/sw_sync.c | |
| parent | eac47b303459255206676d9b3b3b293ac43ff05c (diff) | |
tests/sw_sync: fix gcc warning
casting unsigned char pointer to int pointer causes gcc to be unhapy with
comment: "warning: dereferencing type-punned pointer will break
strict-aliasing rules"
change:
*((int *) CMSG_DATA(cmsg)) = timeline;
to:
memcpy(CMSG_DATA(cmsg), &timeline, sizeof(timeline));
so everyting is nicely defined for compiler.
Signed-off-by: Juha-Pekka Heikkila <juhapekka.heikkila@gmail.com>
Reviewed-by: Petri Latvala <petri.latvala@intel.com>
Diffstat (limited to 'tests/sw_sync.c')
| -rw-r--r-- | tests/sw_sync.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/sw_sync.c b/tests/sw_sync.c index 62d1d17c..626b6d39 100644 --- a/tests/sw_sync.c +++ b/tests/sw_sync.c @@ -228,7 +228,7 @@ static void test_sync_busy_fork_unixsocket(void) cmsg->cmsg_type = SCM_RIGHTS; cmsg->cmsg_len = CMSG_LEN(sizeof(timeline)); - *((int *) CMSG_DATA(cmsg)) = timeline; + memcpy(CMSG_DATA(cmsg), &timeline, sizeof(timeline)); msg.msg_controllen = cmsg->cmsg_len; igt_assert_f(sync_fence_wait(fence, 0) == -ETIME, |