Another comment explaining why a packet size check isn't necessary.

Change-Id: Id7494823f9af69c6185b41983ca78ead2dae24dd
Reviewed-on: https://code.wireshark.org/review/21492
Reviewed-by: Guy Harris <guy@alum.mit.edu>

1 files changed, 8 insertions, 0 deletions

diff --git a/wiretap/vwr.c b/wiretap/vwr.c
index 10695f49a1..d39e11b8fe 100644
--- a/wiretap/vwr.c
+++ b/wiretap/vwr.c
@@ -1792,6 +1792,14 @@ static gboolean vwr_read_s3_W_rec(vwr_t *vwr, struct wtap_pkthdr *phdr,
         rf_ptr = &(rec[0]);
         rf_id = rf_ptr[0];
 
+        /*
+         * Fill up the per-packet header.
+         *
+         * We include the length of the metadata headers in the packet lengths.
+         *
+         * OCTO_MODIFIED_RF_LEN + 1 is less than WTAP_MAX_PACKET_SIZE will
+         * ever be, so we don't need to check it.
+         */
         phdr->len = OCTO_MODIFIED_RF_LEN + 1;       /* 1st octet is reserved for detecting type of frame while displaying in wireshark */
         phdr->caplen = OCTO_MODIFIED_RF_LEN + 1;