From 47668171e34a763b1164a9ffe58a6d457a327931 Mon Sep 17 00:00:00 2001 From: Jeff Morriss Date: Wed, 11 Apr 2007 04:57:43 +0000 Subject: Fix bug 1377: http://bugs.wireshark.org/bugzilla/show_bug.cgi?id=1377 and complaints to the mailing list that a bogus (empty or non-ASCII contents) dialog box was popped up when getting the list of interfaces. The problem was that 'get_airpcap_interface_list()' wasn't setting the returned error value when it found that AirPcap was not loaded. If whatever was in that variable happened to be 1 when a non-AirPcap user requested the list of interfaces then the bogus dialog would show up. svn path=/trunk/; revision=21380 --- airpcap_loader.c | 3 +++ 1 file changed, 3 insertions(+) (limited to 'airpcap_loader.c') diff --git a/airpcap_loader.c b/airpcap_loader.c index 1e37cb6c18..e01b6bc792 100644 --- a/airpcap_loader.c +++ b/airpcap_loader.c @@ -1138,7 +1138,10 @@ get_airpcap_interface_list(int *err, char **err_str) char errbuf[PCAP_ERRBUF_SIZE]; if (!AirpcapLoaded) + { + *err = AIRPCAP_NOT_LOADED; return il; + } if (!g_PAirpcapGetDeviceList(&devsList, errbuf)) { -- cgit v1.2.3