summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDimitri Staessens <dimitri@ouroboros.rocks>2020-10-10 15:03:17 +0200
committerSander Vrijders <sander@ouroboros.rocks>2020-10-11 14:10:18 +0200
commitbf736bebbe89618e23fc5cfb19cf049314cce03d (patch)
tree1995a7db67a5e6a34b670e7ed69c957254d92ac1
parent31e2ab31a61029130fd8be4b2e8abe35bdc19659 (diff)
downloadouroboros-bf736bebbe89618e23fc5cfb19cf049314cce03d.tar.gz
ouroboros-bf736bebbe89618e23fc5cfb19cf049314cce03d.zip
tools: Fix error handling in oping write thread
The function was returning under a cleanup handler, which is not allowed. We don't do anything with the return value if the write thread ends, so just stopping the thread is fine. Signed-off-by: Dimitri Staessens <dimitri@ouroboros.rocks> Signed-off-by: Sander Vrijders <sander@ouroboros.rocks>
-rw-r--r--src/tools/oping/oping_client.c3
1 files changed, 0 insertions, 3 deletions
diff --git a/src/tools/oping/oping_client.c b/src/tools/oping/oping_client.c
index 35f49aa2..55ef6dc5 100644
--- a/src/tools/oping/oping_client.c
+++ b/src/tools/oping/oping_client.c
@@ -177,10 +177,7 @@ void * writer(void * o)
if (flow_write(*fdp, buf, client.size) < 0) {
printf("Failed to send packet.\n");
- flow_dealloc(*fdp);
- free(buf);
stop = true;
- return (void *) -1;
}
}