diff options
author | Steven Rostedt <srostedt@redhat.com> | 2009-12-29 13:59:49 -0500 |
---|---|---|
committer | Steven Rostedt <rostedt@goodmis.org> | 2009-12-29 14:15:51 -0500 |
commit | fb1e0b1e12f928760c6c80b43090b3c08e39693f (patch) | |
tree | 9a4e47a70551c00bae07ca8ceef71d82f4745924 | |
parent | ad74a02a57cfa34642f05fcc483029a2d123c5cc (diff) |
trace-view: Use free_record() not free() for freeing records
Now that records can pin pages to memory, it is imperative that we
use free_record() on all records.
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
-rw-r--r-- | trace-view-store.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/trace-view-store.c b/trace-view-store.c index cc35df4..2c4467a 100644 --- a/trace-view-store.c +++ b/trace-view-store.c | |||
@@ -467,7 +467,7 @@ trace_view_store_get_value (GtkTreeModel *tree_model, | |||
467 | data = tracecmd_read_at(trace_view_store->handle, record->offset, &cpu); | 467 | data = tracecmd_read_at(trace_view_store->handle, record->offset, &cpu); |
468 | g_assert(data != NULL); | 468 | g_assert(data != NULL); |
469 | if (cpu != record->cpu) { | 469 | if (cpu != record->cpu) { |
470 | free(data); | 470 | free_record(data); |
471 | return; | 471 | return; |
472 | } | 472 | } |
473 | 473 | ||
@@ -504,7 +504,7 @@ trace_view_store_get_value (GtkTreeModel *tree_model, | |||
504 | g_value_set_string(value, s.buffer); | 504 | g_value_set_string(value, s.buffer); |
505 | break; | 505 | break; |
506 | } | 506 | } |
507 | free(data); | 507 | free_record(data); |
508 | } | 508 | } |
509 | } | 509 | } |
510 | 510 | ||
@@ -875,7 +875,7 @@ trace_view_store_new (struct tracecmd_input *handle) | |||
875 | rec->ts = data->ts; | 875 | rec->ts = data->ts; |
876 | rec->next = NULL; | 876 | rec->next = NULL; |
877 | next = &rec->next; | 877 | next = &rec->next; |
878 | free(data); | 878 | free_record(data); |
879 | count++; | 879 | count++; |
880 | data = tracecmd_read_data(handle, cpu); | 880 | data = tracecmd_read_data(handle, cpu); |
881 | } | 881 | } |