summaryrefslogtreecommitdiff
path: root/wiretap/libpcap.c
diff options
context:
space:
mode:
authorGuy Harris <guy@alum.mit.edu>2013-07-16 00:18:55 +0000
committerGuy Harris <guy@alum.mit.edu>2013-07-16 00:18:55 +0000
commit3e8b8f0b1a5aae310316102a8402ab46b220f2dc (patch)
tree972eff578be98b0186c7c8059fe04af7242f9c20 /wiretap/libpcap.c
parenta39e5b9b4a38a2443a7cd34086c629a5adfbffdc (diff)
downloadwireshark-3e8b8f0b1a5aae310316102a8402ab46b220f2dc.tar.gz
Explain that we don't have to worry about fixed-length buffers whose
size is based on the snapshot length. svn path=/trunk/; revision=50647
Diffstat (limited to 'wiretap/libpcap.c')
-rw-r--r--wiretap/libpcap.c7
1 files changed, 6 insertions, 1 deletions
diff --git a/wiretap/libpcap.c b/wiretap/libpcap.c
index 0fb0ed9d45..6a64c3d1e6 100644
--- a/wiretap/libpcap.c
+++ b/wiretap/libpcap.c
@@ -768,7 +768,12 @@ static int libpcap_read_header(wtap *wth, FILE_T fh, int *err, gchar **err_info,
/* Disabling because this is not a fatal error, and packets that have
* one such packet probably have thousands. For discussion, see
* https://www.wireshark.org/lists/wireshark-dev/201307/msg00076.html
- * and related messages. */
+ * and related messages.
+ *
+ * The packet contents will be copied to a Buffer, which expands
+ * as necessary to hold the contents; we don't have to worry
+ * about fixed-length buffers allocated based on the original
+ * snapshot length. */
#if 0
if (hdr->hdr.incl_len > wth->snapshot_length) {
g_warning("pcap: File has packet larger than file's snapshot length.");