Blame SOURCES/arpwatch-2.1a15-lookupiselect.patch

1b6bc8
diff -up ./arpwatch.c.iselect ./arpwatch.c
1b6bc8
--- ./arpwatch.c.iselect	2012-10-15 16:01:24.701335291 +0200
1b6bc8
+++ ./arpwatch.c	2012-10-15 16:07:18.626322639 +0200
1b6bc8
@@ -162,50 +162,52 @@ void dropprivileges(const char* user)
1b6bc8
 }
1b6bc8
 
1b6bc8
 char *
1b6bc8
-get_first_dev(pcap_t **pd, int *linktype, char *errbuf)
1b6bc8
+try_dev(char *interface, pcap_t **pd, int *linktype, char *errbuf)
1b6bc8
 {
1b6bc8
-	static char interface[IF_NAMESIZE + 1];
1b6bc8
 	register int snaplen, timeout;
1b6bc8
-	pcap_if_t *alldevs;
1b6bc8
-	pcap_if_t *dev;
1b6bc8
-	char *ret = NULL;
1b6bc8
 
1b6bc8
 	snaplen = max(sizeof(struct ether_header),
1b6bc8
 				  sizeof(struct fddi_header)) + sizeof(struct ether_arp);
1b6bc8
 	timeout = 1000;
1b6bc8
 
1b6bc8
-	if (pcap_findalldevs(&alldevs, errbuf) == -1) {
1b6bc8
-		(void)fprintf(stderr, "%s: lookup_device: %s\n",
1b6bc8
-					  prog, errbuf);
1b6bc8
-		exit(1);
1b6bc8
+	*pd = pcap_open_live(interface, snaplen, 1, timeout, errbuf);
1b6bc8
+	if (NULL == pd) {
1b6bc8
+		syslog(LOG_ERR, "pcap open %s: %s", interface,  errbuf);
1b6bc8
+		return NULL;
1b6bc8
 	}
1b6bc8
+	*linktype = pcap_datalink(*pd);
1b6bc8
+	/* Must be ethernet or fddi */
1b6bc8
+	if (*linktype != DLT_EN10MB && *linktype != DLT_FDDI) {
1b6bc8
+		syslog(LOG_ERR, "(%s) Link layer type %d not ethernet or fddi",
1b6bc8
+			   interface, *linktype);
1b6bc8
+		pcap_close(*pd);
1b6bc8
+		return NULL;
1b6bc8
+	}
1b6bc8
+	return interface;
1b6bc8
+}
1b6bc8
 
1b6bc8
-	for (dev = alldevs; dev; dev = dev->next) {
1b6bc8
-		strncpy(interface, dev->name, strlen(dev->name)+1);
1b6bc8
-
1b6bc8
-		*pd = pcap_open_live(interface, snaplen, 1, timeout, errbuf);
1b6bc8
-		if (*pd == NULL) {
1b6bc8
-			syslog(LOG_ERR, "pcap open %s: %s, trying next...", interface, errbuf);
1b6bc8
-			continue;
1b6bc8
-			/* exit(1); */
1b6bc8
-		}
1b6bc8
+char *
1b6bc8
+iterate_dev(char *arginterface, pcap_t **pd, int *linktype, char *errbuf)
1b6bc8
+{
1b6bc8
+	static char interface[IF_NAMESIZE + 1];
1b6bc8
+	pcap_if_t *alldevs;
1b6bc8
+	pcap_if_t *dev;
1b6bc8
 
1b6bc8
-		*linktype = pcap_datalink(*pd);
1b6bc8
-		/* Must be ethernet or fddi */
1b6bc8
-		if (*linktype != DLT_EN10MB && *linktype != DLT_FDDI) {
1b6bc8
-			syslog(LOG_ERR, "(%s) Link layer type %d not ethernet or fddi, trying next...",
1b6bc8
-				   interface, *linktype);
1b6bc8
-			pcap_close(*pd);
1b6bc8
+	if (NULL != arginterface) {
1b6bc8
+		return try_dev(arginterface, pd, linktype, errbuf);
1b6bc8
+	} else {
1b6bc8
+		if (pcap_findalldevs(&alldevs, errbuf) == -1) {
1b6bc8
+			(void)fprintf(stderr, "%s: lookup_device: %s\n",
1b6bc8
+						  prog, errbuf);
1b6bc8
+			exit(1);
1b6bc8
 		}
1b6bc8
-		else {
1b6bc8
-			/* First match, use it */
1b6bc8
-			ret = interface;
1b6bc8
-			break;
1b6bc8
+		for (dev = alldevs; dev && (arginterface == NULL); dev = dev->next) {
1b6bc8
+			strncpy(interface, dev->name, strlen(dev->name)+1);
1b6bc8
+			arginterface = try_dev(interface, pd, linktype, errbuf);
1b6bc8
 		}
1b6bc8
-
1b6bc8
+		pcap_freealldevs(alldevs);
1b6bc8
+		return arginterface;
1b6bc8
 	}
1b6bc8
-	pcap_freealldevs(alldevs);
1b6bc8
-	return (ret);
1b6bc8
 }
1b6bc8
 
1b6bc8
 int
1b6bc8
@@ -315,8 +317,8 @@ main(int argc, char **argv)
1b6bc8
 	} else {
1b6bc8
 
1b6bc8
 		/* Determine interface if not specified */
1b6bc8
-		if (interface == NULL &&
1b6bc8
-			(interface = get_first_dev(&pd, &linktype, errbuf)) == NULL) {
1b6bc8
+		interface = iterate_dev(interface, &pd, &linktype, errbuf);
1b6bc8
+		if (interface == NULL) {
1b6bc8
 			(void)fprintf(stderr, "%s: lookup_device: no suitable interface found\n",
1b6bc8
 						  prog);
1b6bc8
 			exit(1);