Commit 7d65864b authored by Tzvetomir Stoyanov (VMware)'s avatar Tzvetomir Stoyanov (VMware) Committed by Arnaldo Carvalho de Melo
Browse files

tools lib traceevent: Handle possible strdup() error in tep_add_plugin_path() API 

Free allocated resources and return -1 in case strdup() fails in
tep_add_plugin_path() API.

Link: https://lore.kernel.org/r/CAM9d7chfvJwodpVrHGc5E2J80peRojmYV_fD8x3cpn9HFRUw2g@mail.gmail.com
Link: https://lore.kernel.org/linux-trace-devel/20200714103027.2477584-9-tz.stoyanov@gmail.com
Link: https://lore.kernel.org/linux-trace-devel/20200716092014.2613403-9-tz.stoyanov@gmail.com



Suggested-by: default avatarNamhyung Kim <namhyung@kernel.org>
Signed-off-by: default avatarTzvetomir Stoyanov (VMware) <tz.stoyanov@gmail.com>
Acked-by: default avatarNamhyung Kim <namhyung@kernel.org>
Cc: Andrew Morton <akpm@linux-foundation.org>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: linux-trace-devel@vger.kernel.org
Link: http://lore.kernel.org/lkml/20200722011755.720803193@goodmis.org


Signed-off-by: default avatarSteven Rostedt (VMware) <rostedt@goodmis.org>
Signed-off-by: default avatarArnaldo Carvalho de Melo <acme@redhat.com>
parent db885ed4

tools/lib/traceevent/event-plugin.c

+4 −0
Original line number Diff line number Diff line
@@ -665,6 +665,10 @@ int tep_add_plugin_path(struct tep_handle *tep, char *path, 
		return -1;



	dir->path = strdup(path);

	if (!dir->path) {

		free(dir);

		return -1;

	}

	dir->prio = prio;

	dir->next = tep->plugins_dir;

	tep->plugins_dir = dir;