Blame SOURCES/arpwatch-2.1a15-lookupiselect.patch

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