Switch to pcap_create instead of pcap_live_open. See #1291

This commit is contained in:
dmiller 2018-08-20 22:08:56 +00:00
parent 4deb24fec4
commit 7f1485f56d
2 changed files with 47 additions and 23 deletions

View file

@ -4073,16 +4073,6 @@ int DnetName2PcapName(const char *dnetdev, char *pcapdev, int pcapdevlen) {
#endif
/* Compute exponential sleep time for my_pcap_open_live(). Returned
* value is 5 to the times-th power (5^times) */
static unsigned int compute_sleep_time(unsigned int times){
unsigned int i=0;
unsigned int result=1;
for(i=0; i<times; i++)
result*=5;
return result;
}
/* This function is used to obtain a packet capture handle to look at
* packets on the network. It is actually a wrapper for libpcap's
* pcap_open_live() that takes care of compatibility issues and error
@ -4092,7 +4082,7 @@ pcap_t *my_pcap_open_live(const char *device, int snaplen, int promisc, int to_m
char err0r[PCAP_ERRBUF_SIZE];
pcap_t *pt;
char pcapdev[128];
unsigned int failed = 0;
int failed = 0;
assert(device != NULL);
@ -4109,18 +4099,47 @@ pcap_t *my_pcap_open_live(const char *device, int snaplen, int promisc, int to_m
#else
Strncpy(pcapdev, device, sizeof(pcapdev));
#endif
do {
pt = pcap_open_live(pcapdev, snaplen, promisc, to_ms, err0r);
if (!pt) {
failed++;
if (failed >= 3) {
return NULL;
} else {
netutil_error("pcap_open_live(%s, %d, %d, %d) FAILED. Reported error: %s. Will wait %d seconds then retry.", pcapdev, snaplen, promisc, to_ms, err0r, compute_sleep_time(failed));
}
sleep( compute_sleep_time(failed) );
}
} while (!pt);
#ifdef __amigaos__
// Amiga doesn't have pcap_create
// TODO: Does Nmap still work on Amiga?
pt = pcap_open_live(pcapdev, snaplen, promisc, to_ms, err0r);
if (!pt) {
netutil_error("pcap_open_live(%s, %d, %d, %d) FAILED. Reported error: %s.", pcapdev, snaplen, promisc, to_ms, err0r);
return NULL;
}
#else
pt = pcap_create(pcapdev, err0r);
if (!pt) {
netutil_error("pcap_create(%s) FAILED: %s.", pcapdev, err0r);
return NULL;
}
#define MY_PCAP_SET(func, p_t, val) do {\
failed = func(p_t, val);\
if (failed) {\
netutil_error(#func "(%d) FAILED: %d.", val, failed);\
pcap_close(p_t);\
return NULL;\
}\
} while(0);
MY_PCAP_SET(pcap_set_snaplen, pt, snaplen);
MY_PCAP_SET(pcap_set_promisc, pt, promisc);
MY_PCAP_SET(pcap_set_timeout, pt, to_ms);
failed = pcap_activate(pt);
if (failed < 0) {
// PCAP error
netutil_error("pcap_activate(%s) FAILED: %s.", pcapdev, pcap_geterr(pt));
pcap_close(pt);
return NULL;
}
else if (failed > 0) {
// PCAP warning, report but assume it'll still work
netutil_error("pcap_activate(%s) WARNING: %s.", pcapdev, pcap_geterr(pt));
}
#endif /* not __amigaos__ */
#ifdef WIN32
if (wait == WAIT_ABANDONED || wait == WAIT_OBJECT_0) {

View file

@ -200,6 +200,11 @@ static int nsock_pcap_get_l3_offset(pcap_t *pt, int *dl) {
static int nsock_pcap_try_open(struct npool *nsp, mspcap *mp, const char *dev,
int snaplen, int promisc, int timeout_ms,
char *errbuf) {
/* TODO: Convert from pcap_open_live to pcap_create and set appropriate options.
* Maybe expose an API for setting said options. We can't assume that all
* Nsock apps will want the same options that Nmap wants, especially the
* options Nmap wants for port scanning.
*/
mp->pt = pcap_open_live(dev, snaplen, promisc, timeout_ms, errbuf);
if (!mp->pt) {
nsock_log_error("pcap_open_live(%s, %d, %d, %d) failed with error: %s",