From 5a2fa3f29a7194df7e30ab45e3f4c3c640d017ef Mon Sep 17 00:00:00 2001 From: "Avi Halachmi (:avih)" Date: Sun, 22 Mar 2020 23:45:06 +0200 Subject: lua: readdir: fix double closedir, use one more autofree The double closedir is a regression from the previous commit, which also forgot to use the autofree context with the fullpath string. --- player/lua.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/player/lua.c b/player/lua.c index 833605689e..f20e2a8084 100644 --- a/player/lua.c +++ b/player/lua.c @@ -1019,7 +1019,7 @@ static int script_readdir(lua_State *L, void *tmp) } add_af_dir(tmp, dir); lua_newtable(L); // list - char *fullpath = NULL; + char *fullpath = talloc_strdup(tmp, ""); struct dirent *e; int n = 0; while ((e = readdir(dir))) { @@ -1041,8 +1041,6 @@ static int script_readdir(lua_State *L, void *tmp) lua_pushstring(L, name); // list index name lua_settable(L, -3); // list } - closedir(dir); - talloc_free(fullpath); return 1; } -- cgit v1.2.3