ec20的AT驱动导致内存泄漏？

发布于 2021-11-16 17:29:03 浏览：1559 订阅该版

[tocm]

## 问题现象
ec20使用at驱动下载文件，有的时候导致stm32内存爆满。且内存无法回收，一直会持续到下次重启。
之前发布过一次ec20下载大文件会导致stm32内存爆满的问题。一直以为是写文件的速度不够快，导致内存被全部申请，但查看底层代码之后，发现关闭client的时候最终会释放申请的内存。所以这不是导致本次问题的原因。

## 问题分析
* 1. 在下载文件开始时，打印出来的urc中connectid为1.
* 2. 下载文件过程中，由于某些原因（暂时判断为硬件原因），单片机没有收到ec20下发的数据，到了一定时间后，驱动会返回超时，从而关闭下载文件的client。
* 3. 但是过了几秒后，ec20又开始下发，connectid为1的urc，驱动层又开始不断的申请内存。导致内存爆满，此时已经无法再释放内存了。

下载文件函数部分代码（删除了没必要的部分）
```c
    int webclient_get_file(const char* URI, const char* filename)
    {
        session = webclient_session_create(WEBCLIENT_HEADER_BUFSZ);
    
        fd = open(filename, O_WRONLY | O_CREAT | O_TRUNC, 0);
   
        if (session->content_length < 0) {
            while (1) {
                length = webclient_read(session, ptr, WEBCLIENT_RESPONSE_BUFSZ);
                if (length > 0) {
							
                    write(fd, ptr, length);
                    total_length += length;
                    LOG_RAW(">");
                } else {
                    break;
                }
            }
        }
        close(fd);
        webclient_close(session);
        web_free(ptr);
    }
```
关闭socket源代码
```c
/**
 * close socket by AT commands.
 *
 * @param current socket
 *
 * @return  0: close socket success
 *         -1: send AT commands error
 *         -2: wait socket event timeout
 *         -5: no memory
 */
static int ec20_socket_close(struct at_socket *socket)
{
    int result = RT_EOK;
    at_response_t resp = RT_NULL;
    int device_socket = (int) socket->user_data;
    struct at_device *device = (struct at_device *) socket->device;

resp = at_create_resp(64, 0, 5 * RT_TICK_PER_SECOND);
    if (resp == RT_NULL)
    {
        LOG_E("no memory for resp create.");
        return -RT_ENOMEM;
    }

/* default connection timeout is 10 seconds, but it set to 1 seconds is convenient to use.*/
    result = at_obj_exec_cmd(device->client, resp, "AT+QICLOSE=%d,1", device_socket);

if (resp)
    {
        at_delete_resp(resp);
    }

return result;
}
```

## 结论
由于电平转换电路存在问题。在发送关闭socket的at指令时，ec20没有收到。导致上层webclient已经关闭，但ec20又继续下发connectid为1的urc，这时申请的内存无人释放。
在关闭socket的函数中，是否需要判断resp的内容，或者是加一些其他的逻辑来保护。
```c
if (resp)
    {
        at_delete_resp(resp);
    }
```