graph: fix for unitialized graph_value nodes
And leaking tooltip/node on free.
Signed-off-by: Jens Axboe <axboe@kernel.dk>
diff --git a/graph.c b/graph.c
index 7304b00..9761a1a 100644
--- a/graph.c
+++ b/graph.c
@@ -688,6 +688,7 @@
struct graph_value *x;
x = malloc(sizeof(*x));
+ memset(x, 0, sizeof(*x));
x->value = value;
if (tooltip)
x->tooltip = strdup(tooltip);
@@ -707,6 +708,7 @@
double miny = yval / TOOLTIP_DELTA;
double maxy = yval * TOOLTIP_DELTA;
+ INIT_PRIO_TREE_NODE(&x->node);
x->node.start = miny;
x->node.last = maxy;
if (x->node.last == x->node.start)
@@ -777,13 +779,18 @@
return 0;
}
-static void graph_free_values(struct graph_value *values)
+static void graph_free_values(struct graph_label *l, struct graph_value *values)
{
struct graph_value *i, *next;
for (i = values; i; i = next) {
next = i->next;
free(i->value);
+ if (i->tooltip) {
+ free(i->tooltip);
+ prio_tree_remove(&l->prio_tree, &i->node);
+ l->tooltip_count--;
+ }
free(i);
}
}
@@ -794,7 +801,7 @@
for (i = labels; i; i = next) {
next = i->next;
- graph_free_values(i->values);
+ graph_free_values(i, i->values);
free(i);
}
}