Re: [PATCH v5 02/14] perf tools: Prevent calling machine__delete() on non-allocated machine

[Wangnan (F)]

On 2015/12/15 20:36, Jiri Olsa wrote:

On Mon, Dec 14, 2015 at 10:39:11AM +0000, Wang Nan wrote:

SNIP

@@ -137,12 +138,15 @@ void machine__exit(struct machine *machine)
  void machine__delete(struct machine *machine)
  {
        machine__exit(machine);
-       free(machine);
+       if (machine->allocated)
+               free(machine);
+       else
+               pr_warning("WARNING: delete a non-allocated machine. Skip.\n");

we used WARN_ONCE several times already in similar cases

jirka

Will switch to:

@@ -136,13 +138,13 @@ void machine__exit(struct machine *machine)

 void machine__delete(struct machine *machine)
 {
-       machine__exit(machine);
-       free(machine);
+       WARN_ONCE((machine->allocated ? free(machine), 0 : -1),
+                 "WARNING: deleting a non-allocated machine. Skip.\n");
 }

Thank you.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/