aboutsummaryrefslogtreecommitdiffstats
path: root/samsung-ipc/devices/aries/aries.c
diff options
context:
space:
mode:
authorDenis 'GNUtoo' Carikli <GNUtoo@cyberdimension.org>2020-02-20 23:50:05 +0100
committerDenis 'GNUtoo' Carikli <GNUtoo@cyberdimension.org>2020-02-21 00:21:09 +0100
commitfa4003fb44db3225bfa90ad299be03315063a819 (patch)
tree78652516f6eceaa677544685dc69389d25b6fa6e /samsung-ipc/devices/aries/aries.c
parent86d3646e6c2dfad59c97aa39e944d97f4b3feed5 (diff)
downloadhardware_replicant_libsamsung-ipc-fa4003fb44db3225bfa90ad299be03315063a819.tar.gz
hardware_replicant_libsamsung-ipc-fa4003fb44db3225bfa90ad299be03315063a819.tar.bz2
hardware_replicant_libsamsung-ipc-fa4003fb44db3225bfa90ad299be03315063a819.zip
ipc.c: fix integer comparison sign
With: ./configure CC=clang CFLAGS=-W -Wall -Wno-unused \ --no-create --no-recursion we have: CC ipc.lo ipc.c:71:16: warning: comparison of integers of different signs: 'size_t' (aka 'unsigned int') and 'int' [-Wsign-compare] if (length == -1) ~~~~~~ ^ ~~ This is caused by the following code: size_t length; [...] length = read(fd, &buffer, sizeof(buffer)); if (length == -1) goto error; However, as per the function definition, read uses ssize_t which is the signed version of size_t: ssize_t read(int fd, void *buf, size_t count); Signed-off-by: Denis 'GNUtoo' Carikli <GNUtoo@cyberdimension.org>
Diffstat (limited to 'samsung-ipc/devices/aries/aries.c')
0 files changed, 0 insertions, 0 deletions