C Memory free in loop error -> 大小为 1 的无效读取

Question

我对循环中的malloc'ed char 指针的free() 有疑问。

在char *x=malloc(30) 中，我存储了一个字符串，我用add(x,queue) 将其推入堆栈。问题是我每次迭代都malloc-ing 它，所以我需要在每次迭代中free 它，否则我在valgrind 中有内存泄漏。但是如果我将free(x); 放在while 循环的末尾，valgrind 就会遇到strcmp 的问题：

Invalid read of size 1
at 0x403045D: strcmp

by 0x048871: main(main.c 149) = else if(!strcmp(x,l))
address is 0 bytes inside block of size 30 free d

by 0x80488b8: main(main.c 164) = free(x);
block was allocked at
at 0x402c17c:malloc by main(main.c 129) =char *x=malloc(30);

这很奇怪，因为在另一个分配之前我不需要那个char *x。 有没有办法以不同的方式分配它？

while ((a = getchar()) != EOF) {
    if (a == '<'){
        act = 1;
    }
    if (act == 1) {
        char *x=malloc(30);
        scanf("%29[^>]s",x);

        if(*x=='/'){

            void *l;
            l=pop_from_queue(queue);

            if(l==NULL)
                valid=1;
            else if(!strcmp(x+1,l))
                valid=0;
            else
                valid=1;

            act=0;
        }else{
            push_to_queue(queue,x); //push to stack
            count++;
            act=0;

        }
        free(x);       
     }
}

初始化栈：

typedef struct {
  void **data;
  int head;
  int size;
  int count;

} queue_t;

queue_t* create_queue(int capacity){
  queue_t *queue=calloc(sizeof(queue_t),sizeof(queue_t));
  queue->head=0;
  queue->data=calloc(capacity+10,sizeof(void*));
  queue->count=0;
  queue->size=capacity+1;
  return queue;
}

弹出功能：

void* pop_from_queue(queue_t *queue){

  if(queue->count==0)
    return NULL;
  else
    queue->count--;

  if(queue->head<0)
    return NULL;

  if(queue->head!=0)
    queue->head--;

  return queue->data[queue->head];

}

推送功能：

bool push_to_queue(queue_t *queue, void *data){
  if(queue->count==queue->size){
    queue->size+=10;
    queue->data=realloc(queue->data,queue->size*sizeof(void*));
  }
  queue->data[queue->head++]=data;
  queue->count++;

  return true;
}

Answer 1

在取消引用 x 之前，您没有验证 scanf() 是否实际处理了任何数据。

如果scanf() 不读取任何数据并将其存储在x 中，则从x 读取的任何数据都将无效。

Answer 2

你不能在push_to_queue(queue,x);之后做free(x);，你必须在free(x);之后才if(*x=='/')

因为您释放了推送的值，所以当您弹出它时，您会获得一个已经空闲的内存，并且该内存被 strcmp

使用

这么简单我以前没见过！

注意：无论 strcmp

的结果如何，如果不是 NULL，则需要释放通过 pop 获得的值