From d7871914acd2ed77f43344e36e08944524a67d9e Mon Sep 17 00:00:00 2001 From: Amaury Pouly Date: Mon, 16 Jan 2017 00:10:38 +0100 Subject: Fix dangerous casts On Windows 64-bit, the size of long is 32-bit, thus any pointer to long cast is not valid. In any case, one should use intptr_t and ptrdiff_t when casting to integers. This commit attempts to fix all instances reported by GCC. When relevant, I replaced code by the macros PTR_ADD, ALIGN_UP from system.h Change-Id: I2273b0e8465d3c4689824717ed5afa5ed238a2dc --- apps/plugins/goban/util.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'apps/plugins/goban/util.c') diff --git a/apps/plugins/goban/util.c b/apps/plugins/goban/util.c index bb78316346..206606a4bb 100644 --- a/apps/plugins/goban/util.c +++ b/apps/plugins/goban/util.c @@ -73,7 +73,7 @@ void metadata_summary (void) void * align_buffer (void *buffer, size_t * buffer_size) { - unsigned int wasted = (-(long) buffer) & 3; + unsigned int wasted = (-(intptr_t) buffer) & 3; if (!buffer || !buffer_size) { @@ -88,7 +88,7 @@ align_buffer (void *buffer, size_t * buffer_size) *buffer_size -= wasted; - return (void *) (((char *) buffer) + wasted); + return PTR_ADD(buffer, wasted); } -- cgit v1.2.3