CMUX部分是否有内存泄漏？

发布于 2020-07-09 17:15:25 浏览：1741 订阅该版

2 个回答

xiangxistu 2020-07-09

这家伙很懒，什么也没写！

INWE_8633 2020-07-09

这家伙很懒，什么也没写！

guanxionghui 2020-07-10

这家伙很懒，什么也没写！

根据上面大佬提示，在函数两个地方在在释放内存空间即可。
```c
static void cmux_recv_processdata(struct cmux *cmux, rt_uint8_t *buf, rt_size_t len)
{
    rt_size_t count = len;
    struct cmux_frame *frame = RT_NULL;

cmux_buffer_write(cmux->buffer, buf, count);

frame = cmux_frame_parse(cmux->buffer);
    
    if (frame != RT_NULL)
    {
        /* distribute different data */
        if ((CMUX_FRAME_IS(CMUX_FRAME_UI, frame) || CMUX_FRAME_IS(CMUX_FRAME_UIH, frame)))
        {
            LOG_D("this is UI or UIH frame from channel(%d).", frame->channel);
            if (frame->channel > 0)
            {
                /* receive data from logical channel, distribution them */
                cmux_frame_push(cmux, frame->channel, frame);
                cmux_vcom_isr(cmux, frame->channel, frame->data_length);
            }
            else
            {
				cmux_frame_destroy(frame);  // 1111111111
                /* control channel command */
                LOG_W("control channel command haven't support.");
            }
        }
        else
        {
			cmux_frame_destroy(frame);  // 22222222
            switch ((frame->control & ~CMUX_CONTROL_PF))
            {
            case CMUX_FRAME_UA:
                LOG_D("This is UA frame for channel(%d).", frame->channel);

break;
            case CMUX_FRAME_DM:
                LOG_D("This is DM frame for channel(%d).", frame->channel);

break;
            case CMUX_FRAME_SABM:
                LOG_D("This is SABM frame for channel(%d).", frame->channel);

break;
            case CMUX_FRAME_DISC:
                LOG_D("This is DISC frame for channel(%d).", frame->channel);

break;
            }
        }
    }
}
```

xiangxistu 2020-07-10

这家伙很懒，什么也没写！

xiangxistu 2020-07-10

这家伙很懒，什么也没写！

guanxionghui 2020-07-10

这家伙很懒，什么也没写！

guanxionghui 2020-07-10

这家伙很懒，什么也没写！

@xiangxistu 咨询下，我现在对关闭CMUX然后再次打开CMUX操作，有时候内存会泄漏24byte。帮忙看看是哪里没有释放吗？

```
#include <drv_gpio.h>
static rt_uint8_t s_cmux_status = 0;

int cmux_gsm_init(void)
{
	if (s_cmux_status == 1) 
	{
		return 0;
	}
	
	......
	执行模块开关机操作
	......
	
    gsm = rt_malloc(sizeof(struct cmux));
    rt_memset(gsm, 0, sizeof(struct cmux));

gsm->ops = &cmux_ops;

cmux_init(gsm, CMUX_DEPEND_NAME, CMUX_PORT_NUMBER, RT_NULL);
	
	extern int cmux_sample(void);
	cmux_sample();
	
	s_cmux_status = 1;
	
	return 1;
}
INIT_COMPONENT_EXPORT(cmux_gsm_init);

/**
 *  pop cmux frame data from slist for different channel virtual serial
 *
 * @param cmux          cmux object
 * @param channel       the number of virtual serial
 *
 * @return  frame_data  successful
 *          RT_NULL     no message on the slist
 */
static struct cmux_frame *cmux_frame_destory(struct cmux *cmux, int channel)
{
    rt_base_t level;
    struct frame *frame = RT_NULL;
    struct cmux_frame *frame_data = RT_NULL;
    struct rt_slist_node *frame_list_find = RT_NULL;
    rt_slist_t *frame_list = RT_NULL;

frame_list = &cmux->vcoms[channel].flist;

frame_list_find = rt_slist_first(frame_list);
    while (frame_list_find != RT_NULL)
    {
        frame = rt_container_of(frame_list_find, struct frame, frame_list);
        frame_data = frame->frame;

level = rt_hw_interrupt_disable();
        rt_slist_remove(frame_list, frame_list_find);
        rt_hw_interrupt_enable(level);

LOG_D("new message for channel(%d) has been used, Message remain: %d.", channel, --cmux->vcoms[channel].frame_index);
        //rt_free(frame);
		if ((frame_data->data_length > 0) && frame_data->data)
		{
			rt_free(frame_data->data);
		}
		if (frame)
		{
			rt_free(frame);
		}
		
		frame_list = &cmux->vcoms[channel].flist;
		frame_list_find = rt_slist_first(frame_list);
    }

return frame_data;
}

int cmux_gsm_close(void)
{
	struct rt_device *device = RT_NULL;

if (s_cmux_status == 0) 
	{
		return 0;
	}	
	// 先停止ppp at
	
	// 先删除线程
	rt_thread_delete(gsm->recv_tid);
	
	// 关闭物理串口
	rt_device_close(gsm->dev);
	
	// 关闭虚拟串口
	device = rt_device_find("cmux_ctl");
	rt_device_close(device); 		// 关闭
	rt_device_unregister(device);  	// 注销
	
	device = rt_device_find("cmux_at");  	// "cmux_at" EC20_SAMPLE_CLIENT_NAME
	rt_device_close(device);  
	rt_device_unregister(device);

device = rt_device_find(PPP_CLIENT_NAME);  			// "cmux_ppp"
	rt_device_close(device);
	rt_device_unregister(device);
	
	// 事件删除
	rt_event_delete(gsm->event);
	
	// 清空全局变量
	for (int i = 0; i< CMUX_PORT_NUMBER; i++) {
		cmux_frame_destory(gsm,i);
	}
	// 释放内存空间
	rt_free(gsm->buffer);
	rt_free(gsm->vcoms);
	rt_free(gsm);
    
    rt_memset(gsm, 0, sizeof(struct cmux));
	
	s_cmux_status = 0;
	
	cmux_gsm_init();
	return 1;
}
//MSH_CMD_EXPORT_ALIAS(cmux_gsm_close, cmux_close, close gsm cmux);

int cmux_gsm_restart(void)
{
	ppp_sample_stop();
	cmux_gsm_close();
	ppp_sample_start();
}
MSH_CMD_EXPORT_ALIAS(cmux_gsm_restart, cmux_restart, cmux_gsm_restart);
```

## 测试方式：
隔一段时间执行cmux_restart，发现内存有没有被释放的.