Commit ca3e2d5e authored by Jo-Philipp Wich's avatar Jo-Philipp Wich

plugin: do not free method name separately

A previous commit changed the allocation method for the call context to
include the method name string memory directly, so we must not treat the
method member separately anymore.

Fixes: 37aa9196 ("plugin: fix leaking invoked method name for exec plugins")
Signed-off-by: default avatarJo-Philipp Wich <jo@mein.io>
parent 02c6e1d0
......@@ -146,7 +146,7 @@ rpc_plugin_call(struct ubus_context *ctx, struct ubus_object *obj,
c->input = blobmsg_format_json(msg, true);
c->tok = json_tokener_new();
if (!c->method || !c->input || !c->tok)
if (!c->input || !c->tok)
goto fail;
plugin = c->path + sprintf(c->path, "%s/", RPC_PLUGIN_DIRECTORY);
......@@ -171,9 +171,6 @@ rpc_plugin_call(struct ubus_context *ctx, struct ubus_object *obj,
fail:
if (c)
{
if (c->method)
free(c->method);
if (c->input)
free(c->input);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment