diff options
| author | Silvio Gissi <silvio.gissi@gmail.com> | 2017-06-24 17:13:54 -0700 |
|---|---|---|
| committer | Michael Mann <mmann78@netscape.net> | 2017-06-26 10:18:30 +0000 |
| commit | bc8e63be945970da0eb6017237f60f6b2137f5d6 (patch) | |
| tree | 86658f2309e5b141af47e77825d3110eea593e37 | |
| parent | fce9fc6144f0c40bda278e8f17c5363cd6b9fa5e (diff) | |
| download | wireshark-bc8e63be945970da0eb6017237f60f6b2137f5d6.tar.gz | |
Fix Packet Lengths statistics count
Packet statistics were not showing the packets above 5120 due to
the last entry being reformatted and not parsed correctly.
Since the last entry is "reformatted" for better user string,
also "reformat" the last entry as it goes through "range processing".
Bug: 13844
Change-Id: Id49b41c08111dcad1590e034159b81ead8636c4e
Reviewed-on: https://code.wireshark.org/review/22382
Reviewed-by: Michael Mann <mmann78@netscape.net>
Petri-Dish: Michael Mann <mmann78@netscape.net>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Anders Broman <a.broman58@gmail.com>
(cherry picked from commit aefd4e1844abbc8b10732be03991047617ebd650)
Reviewed-on: https://code.wireshark.org/review/22403
| -rw-r--r-- | epan/stats_tree.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/epan/stats_tree.c b/epan/stats_tree.c index 5bc7916688..8592c197db 100644 --- a/epan/stats_tree.c +++ b/epan/stats_tree.c @@ -781,10 +781,15 @@ stats_tree_create_range_node_string(stats_tree *st, const gchar *name, stat_node *rng_root = new_stat_node(st, name, parent_id, FALSE, TRUE); stat_node *range_node = NULL; - for (i = 0; i < num_str_ranges; i++) { + for (i = 0; i < num_str_ranges - 1; i++) { range_node = new_stat_node(st, str_ranges[i], rng_root->id, FALSE, FALSE); range_node->rng = get_range(str_ranges[i]); } + range_node = new_stat_node(st, str_ranges[i], rng_root->id, FALSE, FALSE); + range_node->rng = get_range(str_ranges[i]); + if (range_node->rng->floor == range_node->rng->ceil) { + range_node->rng->ceil = G_MAXINT; + } return rng_root->id; } |