From 7ec7e43f3be20d7c79b5b62252ed368aca79e54a Mon Sep 17 00:00:00 2001 From: Michal Labedzki Date: Thu, 18 Jun 2015 09:10:55 +0200 Subject: Bluetooth: Coverity fixes Try to fix Coverity issues in Bluetooth HCI and androiddump. Change-Id: Id2ed35130eb4dbb0698b7a54afccdba56af62bfd Reviewed-on: https://code.wireshark.org/review/8983 Petri-Dish: Alexis La Goutte Tested-by: Petri Dish Buildbot Reviewed-by: Michal Labedzki --- extcap/androiddump.c | 12 +++++++++++- 1 file changed, 11 insertions(+), 1 deletion(-) (limited to 'extcap') diff --git a/extcap/androiddump.c b/extcap/androiddump.c index eec27601dd..c96be11e4a 100644 --- a/extcap/androiddump.c +++ b/extcap/androiddump.c @@ -254,6 +254,11 @@ static struct extcap_dumper extcap_dumper_open(char *fifo, int encap) { pcap = pcap_open_dead_with_tstamp_precision(encap_ext, PACKET_LENGTH, PCAP_TSTAMP_PRECISION_NANO); extcap_dumper.dumper.pcap = pcap_dump_open(pcap, fifo); + if (!extcap_dumper.dumper.pcap) { + if (verbose) + fprintf(stderr, "ERROR: Cannot save dump file\n"); + exit(1); + } extcap_dumper.encap = encap; pcap_dump_flush(extcap_dumper.dumper.pcap); #else @@ -270,6 +275,11 @@ static struct extcap_dumper extcap_dumper_open(char *fifo, int encap) { } extcap_dumper.dumper.wtap = wtap_dump_open(fifo, WTAP_FILE_TYPE_SUBTYPE_PCAP_NSEC, encap_ext, PACKET_LENGTH, FALSE, &err); + if (!extcap_dumper.dumper.wtap) { + if (verbose) + fprintf(stderr, "ERROR: Cannot save dump file\n"); + exit(1); + } extcap_dumper.encap = encap; wtap_dump_flush(extcap_dumper.dumper.wtap); #endif @@ -1440,6 +1450,7 @@ static int capture_android_bluetooth_external_parser(char *interface, *bt_local_tcp_port, *bt_server_tcp_port, result); } + memset(&server, 0 , sizeof(server)); server.sin_family = AF_INET; server.sin_port = GINT16_TO_BE(*bt_local_tcp_port); server.sin_addr.s_addr = inet_addr(bt_local_ip); @@ -1526,7 +1537,6 @@ static int capture_android_bluetooth_external_parser(char *interface, if ((sock = socket(AF_INET, SOCK_STREAM, IPPROTO_TCP)) == INVALID_SOCKET) { if (verbose) printf("ERROR1: %s\n", strerror(errno)); - closesocket(sock); return 1; } -- cgit v1.2.3