Evaluating the expression of xbt_assert must not have side effects.

[simgrid.git] / src / xbt / xbt_replay_trace_reader.c

diff --git a/src/xbt/xbt_replay_trace_reader.c b/src/xbt/xbt_replay_trace_reader.c
index d8d1eae..e9eaea9 100644 (file)
--- a/src/xbt/xbt_replay_trace_reader.c
+++ b/src/xbt/xbt_replay_trace_reader.c
@@ -36,10 +36,10 @@ const char *xbt_replay_trace_reader_position(xbt_replay_trace_reader_t reader)
   if (reader->position)
     free(reader->position);
   reader->position = bprintf("%s:%d",reader->filename,reader->linenum);
-  return (const char*)reader->position;
+  return reader->position;
 }
 
-char * const *xbt_replay_trace_reader_get(xbt_replay_trace_reader_t reader)
+const char **xbt_replay_trace_reader_get(xbt_replay_trace_reader_t reader)
 {
   ssize_t read;
   xbt_dynar_t d;
@@ -57,7 +57,7 @@ char * const *xbt_replay_trace_reader_get(xbt_replay_trace_reader_t reader)
 
   d=xbt_str_split_quoted_in_place(reader->line);
   if (xbt_dynar_length(d)==0) {
-    xbt_free(d);
+    xbt_dynar_free(&d);
     return xbt_replay_trace_reader_get(reader); /* Get next line */
   }
   return xbt_dynar_to_array(d);