dect
/
libpcap
Archived
13
0
Fork 0
Code Releases Activity

Error messages returned in the pcap_t's error buffer shouldn't have a 

newline in them.

If we're in cooked mode, the packet pointer argument we pass to the callback
should point to the beginning of the constructed sll header, not to the
packet data itself.  While we're at it, have a paranoid check to make
sure that we were given enough space to construct the sll header, so we
don't stomp on the tpacket header.
This commit is contained in:
guy 2008-02-02 22:25:51 +00:00
parent beba34d0f9
commit a27e58b244
1 changed files with 29 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

32
pcap-linux.c
View File

@ -34,7 +34,7 @@
#ifndef lint
static const char rcsid[] _U_ =
"@(#) $Header: /tcpdump/master/libpcap/pcap-linux.c,v 1.136 2008-02-02 21:27:28 guy Exp $ (LBL)";
"@(#) $Header: /tcpdump/master/libpcap/pcap-linux.c,v 1.137 2008-02-02 22:25:51 guy Exp $ (LBL)";
#endif
/*
@ -1818,7 +1818,7 @@ pcap_read_linux_mmap(pcap_t *handle, int max_packets, pcap_handler callback,
if (thdr->tp_mac+thdr->tp_snaplen > handle->bufsize) {
snprintf(handle->errbuf, PCAP_ERRBUF_SIZE,
"corrupted frame on kernel ring mac "
"offset %d + caplen %d > frame len %d\n",
"offset %d + caplen %d > frame len %d",
thdr->tp_mac, thdr->tp_snaplen, handle->bufsize);
return -1;
}
@ -1854,8 +1854,34 @@ pcap_read_linux_mmap(pcap_t *handle, int max_packets, pcap_handler callback,
/* if required build in place the sll header*/
if (handle->md.cooked) {
struct sll_header *hdrp = (struct sll_header *)((char *)bp - sizeof(struct sll_header));
struct sll_header *hdrp;
/*
* The kernel should have left us with enough
* space for an sll header; back up the packet
* data pointer into that space, as that'll be
* the beginning of the packet we pass to the
* callback.
*/
bp -= SLL_HDR_LEN;
/*
* Let's make sure that's past the end of
* the tpacket header, i.e. >=
* ((u_char *)thdr + TPACKET_HDRLEN), so we
* don't step on the header when we construct
* the sll header.
*/
if (bp < (u_char *)thdr + TPACKET_HDRLEN) {
snprintf(handle->errbuf, PCAP_ERRBUF_SIZE,
"cooked-mode frame doesn't have room for sll header");
return -1;
}
/*
* OK, that worked; construct the sll header.
*/
hdrp = (struct sll_header *)bp;
hdrp->sll_pkttype = map_packet_type_to_sll_type(
sll->sll_pkttype);
hdrp->sll_hatype = htons(sll->sll_hatype);