Question

我正在使用带有FreeRTOS的嵌入式板。

在一个任务中，我定义了两个结构并使用pvPortMalloc分配内存。（一个结构是另一个的成员）

此外，我将struct的地址传递给某些函数。

但是，使用vPortFree释放内存存在一些问题。

以下是我的代码（test_task.c）：

/* Struct definition */
typedef struct __attribute__((packed)) {

        uint8_t  num_parameter;
        uint32_t member1;
        uint8_t  member2;
        uint8_t  *parameter;

}struct_member;

typedef struct __attribute__((packed)) {

        uint16_t num_member; 
        uint32_t class;
        struct_member *member;

}struct_master;

我在下面定义一个全局结构和一个数组。

uint8_t       *arr;
struct_master master:

函数定义：

void decode_func(struct_master *master, uint8_t *arr) 
{
   master->member = pvPortMalloc(master->num_member);

   for(int i = 0; i < scr->num_command; ++i){
      master->member[i].parameter = pvPortMalloc(master->member[i].num_parameter);
      do_something();
   }  
}

操作任务如下所示。

在任务结束时，我想释放内存：

void test_task()
{
  decode_func( &master, arr);
  do_operation(); 


  vPortFree(master.member);
  for (int i = 0; i < master.num_member; ++i)
      vPortFree(master.member[i].parameter);

  hTest_task = NULL;
  vTaskDelete(NULL);    
}

可以释放master.member。

但是，当程序免费尝试master.member[i].parameter时，似乎以前已经执行过释放，并且软件会自动重置。

有人知道为什么会这样吗？

Answer 1

通常，当您释放一个对象时，该对象的内容将被破坏，您将无法再访问它们。因此，当您要释放这样的嵌套分配时，需要先释放内部分配首先，然后才释放外部（主）分配。换句话说：

for (int i = 0; i < master.num_member; ++i)
    vPortFree(master.member[i].parameter);
vPortFree(master.member);

先释放参数，然后释放包含的成员数组。

Answer 2

乍一看，decode_func中为成员分配的方式是错误的。

我假设master->num_member指示struct member应该包含的master个数目。

master->member = pvPortMalloc(master->num_member);

应更正为

master->member = pvPortMalloc(master->num_member * sizeof(struct_member));

同样，在同一函数中，循环似乎也有点可疑。

   for(int i = 0; i < scr->num_command; ++i){
      master->member[i].parameter = pvPortMalloc(master->member[i].num_parameter);
      do_something();
   }

我不确定src->num_command表示什么，但是我自然认为循环应该执行到i < master->num_member为止。我认为您的循环也应如下更新，

   for(int i = 0; i < master->num_member; ++i){
      master->member[i].parameter = pvPortMalloc(master->member[i].num_parameter * sizeof(uint8_t));
      do_something();
   }

在释放内存时，请确保在释放容器结构之前先释放所包含的成员。因此，您应该首先释放所有parameter，然后释放member，并在test_task函数中更改该顺序。

还要确保在进行vTaskDelete(NULL);之前必须释放test_task消耗的所有资源，否则会发生资源泄漏。 vTaskDelete(NULL)会简单地将该特定任务的TCB标记为ready to be deleted，以便稍后空闲的任务将清除与TCB相关的资源。

为全局结构分配内存并释放它的问题

2 个答案: