diff options
| author |   Romain Forlot <romain.forlot@iot.bzh> | 2018-07-23 17:32:04 +0200 |
|---|---|---|
| committer | Romain Forlot <romain.forlot@iot.bzh> | 2018-12-13 15:02:55 +0100 |
| commit | 7fd47c1f35871041ef8959c0591fcd8cd5cff724 (patch) | |
| tree | e9b2cc801b9410a43c6fe0cf84b6d66a8258af4a | |
| parent | 97e31a3f1ce9d884ac043e570f54caaa47bf88f5 (diff) | |
Release arguments once C function returned.
There was a leack after poping arguments from Lua interpreter.
Now when a Lua2C function is called, if there is something to keep
from the argument then you should call json_object_get() on it
to preserve it.
Change-Id: I43e79ed73a507ac2ca7ed4cdc3f16ec009392194
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
| -rw-r--r-- | ctl-lib/ctl-lua.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/ctl-lib/ctl-lua.c b/ctl-lib/ctl-lua.c index ab0abb6..ac22767 100644 --- a/ctl-lib/ctl-lua.c +++ b/ctl-lib/ctl-lua.c @@ -692,11 +692,12 @@ int Lua2cWrapper(void* luaHandle, char *funcname, Lua2cFunctionT callback) { json_object *argsJ = LuaPopArgs(source, luaState, LUA_FIRST_ARG + 1); int err = (*callback) (source, argsJ, &responseJ); + json_object_put(argsJ); // push error code and eventual response to LUA int count = 1; lua_pushinteger(luaState, err); - if (responseJ)count += LuaPushArgument(source, responseJ); + if (responseJ) count += LuaPushArgument(source, responseJ); return count; } |