aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSteven Rostedt <srostedt@redhat.com>2009-12-29 13:59:49 -0500
committerSteven Rostedt <rostedt@goodmis.org>2009-12-29 14:15:51 -0500
commitfb1e0b1e12f928760c6c80b43090b3c08e39693f (patch)
tree9a4e47a70551c00bae07ca8ceef71d82f4745924
parentad74a02a57cfa34642f05fcc483029a2d123c5cc (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.c6
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 }