diff options
author | dimitri staessens <dimitri.staessens@intec.ugent.be> | 2016-10-21 15:28:24 +0200 |
---|---|---|
committer | dimitri staessens <dimitri.staessens@intec.ugent.be> | 2016-10-21 15:28:24 +0200 |
commit | d5a52f3951fff7ee272bd0d4cd95cd122d07fa64 (patch) | |
tree | d32f6d89550a7b8606c78c241b73b23a39ae8bed /src/lib/dev.c | |
parent | 482c44232d4deda3f89a7d85fbad99c1c64e80ec (diff) | |
download | ouroboros-d5a52f3951fff7ee272bd0d4cd95cd122d07fa64.tar.gz ouroboros-d5a52f3951fff7ee272bd0d4cd95cd122d07fa64.zip |
build: Comply with -Wextra compiler flag
This reduces the risk for some bugs, for instance due to
signed/unsigned mismatches and unused variables.
Diffstat (limited to 'src/lib/dev.c')
-rw-r--r-- | src/lib/dev.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/src/lib/dev.c b/src/lib/dev.c index 146070b7..c349846f 100644 --- a/src/lib/dev.c +++ b/src/lib/dev.c @@ -467,6 +467,9 @@ int flow_alloc(char * dst_name, char * src_ae_name, struct qos_spec * qos) irm_msg_t * recv_msg = NULL; int fd = -1; + /* FIXME: add qos support */ + (void) qos; + if (dst_name == NULL) return -EINVAL; @@ -756,7 +759,7 @@ ssize_t flow_write(int fd, void * buf, size_t count) ssize_t flow_read(int fd, void * buf, size_t count) { int idx = -1; - int n; + ssize_t n; uint8_t * sdu; if (fd < 0 || fd >= AP_MAX_FLOWS) @@ -794,7 +797,7 @@ ssize_t flow_read(int fd, void * buf, size_t count) return -1; } - memcpy(buf, sdu, MIN(n, count)); + memcpy(buf, sdu, MIN(n, (ssize_t) count)); shm_rdrbuff_remove(ai.rdrb, idx); |