diff options
| author | Dimitri Staessens <[email protected]> | 2020-10-10 15:03:17 +0200 |
|---|---|---|
| committer | Sander Vrijders <[email protected]> | 2020-10-11 14:10:18 +0200 |
| commit | bf736bebbe89618e23fc5cfb19cf049314cce03d (patch) | |
| tree | 1995a7db67a5e6a34b670e7ed69c957254d92ac1 /src/tools/oping | |
| parent | 31e2ab31a61029130fd8be4b2e8abe35bdc19659 (diff) | |
| download | ouroboros-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 <[email protected]>
Signed-off-by: Sander Vrijders <[email protected]>
Diffstat (limited to 'src/tools/oping')
| -rw-r--r-- | src/tools/oping/oping_client.c | 3 |
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; } } |