diff options
author | Peter Wu <peter@lekensteyn.nl> | 2017-03-13 17:02:23 +0100 |
---|---|---|
committer | Michael Mann <mmann78@netscape.net> | 2017-03-13 21:46:38 +0000 |
commit | 98558fd81cad9d578ea5085fe8465334e4b266fd (patch) | |
tree | 62919d27d2bbd60edfc9406a557f284002d62e15 /epan/value_string.h | |
parent | f1d0533ee1f051aabbc1ba8e3e6a9b51dbb2c228 (diff) |
Fix buffer overrun in try_bytesprefix_to_str
bytes_string list contains a list of prefixes to match, not the other
way round (matching prefixes in the list).
Bug: 13479
Change-Id: Ie625dc5db30bd55158d688a0101f35d0bf6906af
Fixes: v2.3.0rc0-2644-g540b555729 ("Introduce "bytes_string" type, similar to "value_string"")
Reviewed-on: https://code.wireshark.org/review/20532
Petri-Dish: Peter Wu <peter@lekensteyn.nl>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Michael Mann <mmann78@netscape.net>
Diffstat (limited to 'epan/value_string.h')
-rw-r--r-- | epan/value_string.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/epan/value_string.h b/epan/value_string.h index 92d24deb97..07a2eb92e2 100644 --- a/epan/value_string.h +++ b/epan/value_string.h @@ -293,12 +293,12 @@ try_bytesval_to_str(const guint8 *val, const size_t val_len, const bytes_string WS_DLL_PUBLIC const gchar * -bytesprefix_to_str(const guint8 *prefix, const size_t prefix_len, const bytes_string *bs, const char *fmt) +bytesprefix_to_str(const guint8 *haystack, const size_t haystack_len, const bytes_string *bs, const char *fmt) G_GNUC_PRINTF(4, 0); WS_DLL_PUBLIC const gchar * -try_bytesprefix_to_str(const guint8 *prefix, const size_t prefix_len, const bytes_string *bs); +try_bytesprefix_to_str(const guint8 *haystack, const size_t haystack_len, const bytes_string *bs); /* MISC (generally do not use) */ |