diff options
author | Guy Harris <guy@alum.mit.edu> | 2017-05-04 00:20:37 -0700 |
---|---|---|
committer | Guy Harris <guy@alum.mit.edu> | 2017-05-04 07:21:18 +0000 |
commit | cd7980206347563f93b134dec16161ba519ad538 (patch) | |
tree | efa23b3bbd7a85349a7a3dba8e444a204a3a2d09 | |
parent | 4ca15be3eb1f3675d7f683e0b9b433999835dccc (diff) | |
download | wireshark-cd7980206347563f93b134dec16161ba519ad538.tar.gz |
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>
-rw-r--r-- | wiretap/vwr.c | 8 |
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; |