diff options
author | Guy Harris <guy@alum.mit.edu> | 2013-07-16 00:18:55 +0000 |
---|---|---|
committer | Guy Harris <guy@alum.mit.edu> | 2013-07-16 00:18:55 +0000 |
commit | 3e8b8f0b1a5aae310316102a8402ab46b220f2dc (patch) | |
tree | 972eff578be98b0186c7c8059fe04af7242f9c20 /wiretap/libpcap.c | |
parent | a39e5b9b4a38a2443a7cd34086c629a5adfbffdc (diff) | |
download | wireshark-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.c | 7 |
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."); |