From a19c12b4a973d8f514db3824ed16f664fb9f1151 Mon Sep 17 00:00:00 2001 From: Dario Lombardo Date: Fri, 14 Oct 2016 14:36:27 +0200 Subject: udpdump: fix issues from coverity. Change-Id: I1d82d8166abe8eda6588ae2970ae9f2d096adf9d Reviewed-on: https://code.wireshark.org/review/18198 Petri-Dish: Dario Lombardo Tested-by: Petri Dish Buildbot Reviewed-by: Jaap Keuter --- extcap/udpdump.c | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) (limited to 'extcap') diff --git a/extcap/udpdump.c b/extcap/udpdump.c index 1ff4e4b513..a83b7215d6 100644 --- a/extcap/udpdump.c +++ b/extcap/udpdump.c @@ -168,7 +168,7 @@ static int setup_dumpfile(const char* fifo, FILE** fp) } *fp = fopen(fifo, "w"); - if (!fp) { + if (!(*fp)) { g_warning("Error creating output file: %s", g_strerror(errno)); return EXIT_FAILURE; } @@ -233,19 +233,22 @@ static int dump_packet(const char* proto_name, const guint16 listenport, const c static void run_listener(const char* fifo, const guint16 port, const char* proto_name) { struct sockaddr_in clientaddr; - int clientlen; + int clientlen = 0; socket_handle_t sock; char buf[PKT_BUF_SIZE]; ssize_t buflen; - FILE* fp; + FILE* fp = NULL; if (signal(SIGINT, exit_from_loop) == SIG_ERR) { g_warning("Can't set signal handler"); return; } - if (setup_dumpfile(fifo, &fp) == EXIT_FAILURE) + if (setup_dumpfile(fifo, &fp) == EXIT_FAILURE) { + if (fp) + fclose(fp); return; + } if (setup_listener(port, &sock) == EXIT_FAILURE) return; -- cgit v1.2.1