RT-Thread一般讨论

webclient重定向导致内存问题

发布于 2019-09-30 14:57:11 浏览：750

本帖最后由 wenbodong 于 2019-9-30 14:58 编辑

webclient软件包版本： v2.1.0

get函数如下。

int webclient_get(struct webclient_session *session, const char *URI)
{
    int rc = WEBCLIENT_OK;
    int resp_status = 0;

    RT_ASSERT(session);
    RT_ASSERT(URI);

    rc = webclient_connect(session, URI);
    if (rc != WEBCLIENT_OK)
    {
        /* connect to webclient server failed. */
        return rc;
    }

    rc = webclient_send_header(session, WEBCLIENT_GET);
    if (rc != WEBCLIENT_OK)
    {
        /* send header to webclient server failed. */
       return rc;
    }

    /* handle the response header of webclient server */
    resp_status = webclient_handle_response(session);

    LOG_D("get position handle response(%d).", resp_status); 

    if (resp_status > 0)
    {
        const char *location = webclient_header_fields_get(session, "Location");

        /* relocation */
        if ((resp_status == 302 || resp_status == 301) && location)
        {
            char *new_url;

            new_url = web_strdup(location);
            if (new_url == RT_NULL)
            {
                return -WEBCLIENT_NOMEM;
            }

            /* close old client session */
            webclient_close(session);

            /* create new client session by location url */
            session = webclient_session_create(WEBCLIENT_HEADER_BUFSZ);
            if (session == RT_NULL)
            {
                return  -WEBCLIENT_NOMEM;
            }

            rc = webclient_get(session, new_url);

            web_free(new_url);
            return rc;
        }
    }

    return resp_status;
}

在接收到服务器的重定向指示后会重新创建session。

if ((resp_status == 302 || resp_status == 301) && location)
       {
           char *new_url;

           new_url = web_strdup(location);
           if (new_url == RT_NULL)
           {
               return -WEBCLIENT_NOMEM;
           }

           /* close old client session */
           webclient_close(session);

           /* create new client session by location url */
           session = webclient_session_create(WEBCLIENT_HEADER_BUFSZ);

但是这是在函数内部重建创建啊，外层函数的session还是原来的（而且还被关闭了。。。）。后续使用其他函数时，如webclient_read，感觉会出问题啊。
一时没想到怎么测试重定向，只是看到这个实现觉得有内存问题。如果开发者在测试时没遇到问题，也可能是个巧合。即：为新session分配的内存的首地址与原session一致（因为是先释放原session后紧接着创建新session）。

3 个回答

Cheney_Chen 2019-09-30

This guy hasn't written anything yet

先释放清空之后，再重新申请的 session，应该没什么问题。
webclient 测试用中每次都有测试重定向，你也可以拿下面重定向的地址测试看看：

http://www.rt-thread.com/service/location302

钟为先 2020-05-14

This guy hasn't written anything yet

我就遇到了这个问题，我在调用webclient_read读取文件的时候，申请了一个新的session，用来上报读取进度，然后出现了内存泄漏，这要怎么解决

wenbodong 2020-05-18

This guy hasn't written anything yet

钟为先发表于 2020-5-14 16:19
我就遇到了这个问题，我在调用webclient_read读取文件的时候，申请了一个新的session，用来上报读取进度， ...

最新版本v2.1.2应该解决了这个问题。

/* relocation */
if ((resp_status == 302 || resp_status == 301) && location)
{
        char *new_url;

        new_url = web_strdup(location);
        if (new_url == RT_NULL)
        {
                return -WEBCLIENT_NOMEM;
        }

        /* clean webclient session */
        webclient_clean(session);
        /* clean webclient session header */
        session->header->length = 0;
        rt_memset(session->header->buffer, 0, session->header->size);

        rc = webclient_get(session, new_url);

        web_free(new_url);
        return rc;
}

在重定向时，不是释放整个session，而是清理必要的内容，重新使用该session。

因为我在项目中并没有使用这个webclient_get函数，所以没进行细致的测试。我使用的是sim800c模块，用http下载固件以升级系统。如果使用webclient_get一次性的下载的话，来不及将固件存储到本地flash。所以我基于http的部分下载功能新增了几个函数，也就没管这个重定向的问题了。