diff options
| author | dimitri staessens <[email protected]> | 2016-10-21 15:28:24 +0200 |
|---|---|---|
| committer | dimitri staessens <[email protected]> | 2016-10-21 15:28:24 +0200 |
| commit | d5a52f3951fff7ee272bd0d4cd95cd122d07fa64 (patch) | |
| tree | d32f6d89550a7b8606c78c241b73b23a39ae8bed /src/tools/oping/oping_client.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/tools/oping/oping_client.c')
| -rw-r--r-- | src/tools/oping/oping_client.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/src/tools/oping/oping_client.c b/src/tools/oping/oping_client.c index 40f75785..c2471b46 100644 --- a/src/tools/oping/oping_client.c +++ b/src/tools/oping/oping_client.c @@ -39,6 +39,9 @@ void shutdown_client(int signo, siginfo_t * info, void * c) { + (void) info; + (void) c; + switch(signo) { case SIGINT: case SIGTERM: @@ -65,6 +68,8 @@ void * reader(void * o) if (fq == NULL) return (void *) 1; + (void) o; + /* FIXME: use flow timeout option once we have it */ while (client.rcvd != client.count && flow_event_wait(client.flows, fq, &timeout) != -ETIMEDOUT) { @@ -78,7 +83,7 @@ void * reader(void * o) continue; } - if (ntohl(msg->id) >= client.count) { + if ((int) ntohl(msg->id) >= client.count) { printf("Invalid id.\n"); continue; } |