diff options
author | Guy Harris <guy@alum.mit.edu> | 2015-08-28 16:31:33 -0700 |
---|---|---|
committer | Guy Harris <guy@alum.mit.edu> | 2015-08-28 23:32:18 +0000 |
commit | 1663224c24b38dac3218c8690d1035f5b63a548f (patch) | |
tree | b71b7ecc93d2754d5cb5219ac486f26c69d9f664 /wiretap/pcapng_module.h | |
parent | 15c9923d642cb321854a05257b598e066f008b0b (diff) | |
download | wireshark-1663224c24b38dac3218c8690d1035f5b63a548f.tar.gz |
Block sizes are unsigned 32-bit quantities; don't stuff them into an int.
*Especially* don't stuff the amount of remaining data in a block into an
int that will then be passed to file_skip() as an amount to skip ahead,
as a Really Large Value will turn into a negative value and produce
various forms of bizarre and tricky-to-debug behavior.
Change-Id: I4d0a6b36fe50df84925690ad688a3ab0433ceb17
Reviewed-on: https://code.wireshark.org/review/10299
Reviewed-by: Guy Harris <guy@alum.mit.edu>
Diffstat (limited to 'wiretap/pcapng_module.h')
-rw-r--r-- | wiretap/pcapng_module.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/wiretap/pcapng_module.h b/wiretap/pcapng_module.h index 1707a4d04f..c735380ab6 100644 --- a/wiretap/pcapng_module.h +++ b/wiretap/pcapng_module.h @@ -41,7 +41,7 @@ /* * Reader and writer routines for pcap-ng block types. */ -typedef gboolean (*block_reader)(FILE_T, int, gboolean, struct wtap_pkthdr *, +typedef gboolean (*block_reader)(FILE_T, guint32, gboolean, struct wtap_pkthdr *, Buffer *, int *, gchar **); typedef gboolean (*block_writer)(wtap_dumper *, const struct wtap_pkthdr *, const guint8 *, int *); |