有关lua返回userdata致使内存不回收的问题

maxiaoli921

最近在使用openresty，添加第三方的lua模块（luagd）时，发现：若将结构体按照userdata返回给lua，即使定义userdata时，同时指定了他的__gc回收函数，那么他也没有回收，内存会持续增长。当在lua中主动调用他的回收函数，那么内存将会释放!

包括使用client9/libinjection的lua模块，自己编译的libmodsecurity等模块，也是一样的状况！

请问__gc为何没生效，还是说我的使用有问题？

附：

luagd.c

luaL_newmetatable(L, GD_IMAGE_PTR_TYPENAME);
    lua_pushliteral(L, "__index");
    lua_pushvalue(L, -3);
    lua_settable(L, -3);
    luaL_openlib(L, NULL, LgdMetatable, 0);
    lua_pop(L, 1);

其中LgdMetatable如下：

static const luaL_reg LgdMetatable[] =
{
{ "__gc", LgdImageDestroy },
{ NULL, NULL }
};

LgdImageDestroy如下：

static int LgdImageDestroy(lua_State *L) {
gdImagePtr im = getImagePtr(L, 1);
if (im)
gdImageDestroy(im);
return 0;
}

doujiang24

Hello,

我想应该是你用错了，userdata 是 Lua gc 自动回收的，__gc 这个元方法只是让 userdata 在释放之前，先执行这个 gc 方法

你的内存持续增长，应该是你申请的 userdata 一直在被引用，你应该修正对应的逻辑

https://www.lua.org/manual/5.1/manual.html#2.10

在 2016年6月7日上午10:41，齐健 <maxia...@gmail.com>写道：

     最近在使用openresty，添加第三方的lua模块（luagd）时，发现：若将结构体按照userdata返回给lua，即使定义userdata时，同时指定了他的__gc回收函数，那么他也没有回收，内存会持续增长。当在lua中主动调用他的回收函数，那么内存将会释放!
     包括使用client9/libinjection的lua模块，自己编译的libmodsecurity等模块，也是一样的状况！
     请问__gc为何没生效，还是说我的使用有问题？

附：
luagd.c

luaL_newmetatable(L, GD_IMAGE_PTR_TYPENAME);
    lua_pushliteral(L, "__index");
    lua_pushvalue(L, -3);
    lua_settable(L, -3);
    luaL_openlib(L, NULL, LgdMetatable, 0);
    lua_pop(L, 1);

其中LgdMetatable如下：
static const luaL_reg LgdMetatable[] =
{
{ "__gc", LgdImageDestroy },
{ NULL, NULL }
};

  LgdImageDestroy如下：
static int LgdImageDestroy(lua_State *L) {
gdImagePtr im = getImagePtr(L, 1);
if (im)
gdImageDestroy(im);
return 0;
}

--

maxiaoli921

Hello,

背景：

luagd是github上下载的代码

编译后生成gd.so

上传图片内容为body

真实图片类型为typ

每次在access_by_lua阶段调用

local im = nil

local data = "">

if typ == "png" then

im = gd.createFromPngStr(data)

if im then

data ="">

end

-----------------

以上代码就结束了，得到了重构后的data，类型为string

后面代码是对data进行处理，与im无关

-----------------

如此，每次上传图片均会增大内存，很久都不释放

若手动调用if im then im:destroy() end，则立即释放

ps:源码中__gc调用destroy()，我调用时，需注释掉__gc，并放开destroy()

-----------------

难道最后需要im = nil吗？即使如此，问题依旧！

On Saturday, June 11, 2016 at 9:27:55 PM UTC+8, doujiang wrote:

Hello,

我想应该是你用错了，userdata 是 Lua gc 自动回收的，__gc 这个元方法只是让 userdata 在释放之前，先执行这个 gc 方法

你的内存持续增长，应该是你申请的 userdata 一直在被引用，你应该修正对应的逻辑

https://www.lua.org/manual/5.1/manual.html#2.10

在 2016年6月7日上午10:41，齐健 <maxia...@gmail.com>写道：
     最近在使用openresty，添加第三方的lua模块（luagd）时，发现：若将结构体按照userdata返回给lua，即使定义userdata时，同时指定了他的__gc回收函数，那么他也没有回收，内存会持续增长。当在lua中主动调用他的回收函数，那么内存将会释放!
     包括使用client9/libinjection的lua模块，自己编译的libmodsecurity等模块，也是一样的状况！
     请问__gc为何没生效，还是说我的使用有问题？

附：
luagd.c

luaL_newmetatable(L, GD_IMAGE_PTR_TYPENAME);
    lua_pushliteral(L, "__index");
    lua_pushvalue(L, -3);
    lua_settable(L, -3);
    luaL_openlib(L, NULL, LgdMetatable, 0);
    lua_pop(L, 1);

其中LgdMetatable如下：
static const luaL_reg LgdMetatable[] =
{
{ "__gc", LgdImageDestroy },
{ NULL, NULL }
};

  LgdImageDestroy如下：
static int LgdImageDestroy(lua_State *L) {
gdImagePtr im = getImagePtr(L, 1);
if (im)
gdImageDestroy(im);
return 0;
}

--

maxiaoli921

刚刚代码中的gd是

local gd = require "gd"

得到的

agentzh

Hello!

2016-06-13 19:47 GMT-07:00 Miracle:
> 背景：
> luagd是github上下载的代码
> 编译后生成gd.so
>
> 上传图片内容为body
> 真实图片类型为typ
> 每次在access_by_lua阶段调用
>
> local im    = nil
> local data = nil
>
> if typ == "png" then
>     im = gd.createFromPngStr(data)
>     if im then
>         data = gd.pngStr(im)
>     end
> end
>
> -----------------
> 以上代码就结束了，得到了重构后的data，类型为string
> 后面代码是对data进行处理，与im无关
> -----------------
> 如此，每次上传图片均会增大内存，很久都不释放

看起来应当是你用的那个 luagd
库的内存泄漏问题。如果你自己不能调试和分析的话，最简单的做法便是尝试联系这个库的作者或维护者了。抱歉，我并没有时间帮你去仔细调试和分析
luagd 这个库，毕竟并不是 OpenResty 维护的组件。

Regards,
-agentzh