在C中动态分配内存时出现段错误

Question

我一直在尝试在C中建立优先级队列。
首先，我进行一些初始化工作，例如分配空间。
以下是Initialize例程， PriorityQueue是一个指针。

void Initialize(int MaxElement, PriorityQueue H)
{
   if (MaxElement < MinPQSize)
     printf("Priority queue size is too small");

   if (!(H = (PriorityQueue)malloc(sizeof(struct HeapStruct))))
     printf("Out of space!!!");

   if (!(H->Elements = (ElementType *)malloc((MaxElement+1) * sizeof(ElementType))))
     printf("Out of space!!!");

   H->Capacity = MaxElement;
   H->Size = 0;

   H->Elements[0] = MinData;
}

这是测试代码的样子

 int MaxElement = 15;
 PriorityQueue myHeap;
 Initialize(MaxElement, myHeap);

但是，当我尝试将元素插入堆中时，会弹出分段错误。
只需从Initialize例程返回PriorityQueue指针即可解决。

 PriorityQueue Initialize(int MaxElement, PriorityQueue H)
 {
   ...
   return H;
 }
 myHeap = Initialize(MaxElement, myHeap);

那么幕后到底发生了什么？
函数返回无返回值时是否调用free（）？
提前谢谢！

Answer 1

不，即使您传入的H是一个指针，您仍在尝试在函数中更改它（使用第一个malloc ）。 为了更改某些内容，您需要向其传递一个指针。 在这种情况下，这意味着指向一个指针的指针：

void Initialize (int MaxElem, PriorityQueue *H) {
    if (MaxElem < MinPQSize)
        printf("Priority queue size is too small");

    if (!(*H = (PriorityQueue)malloc(sizeof(struct HeapStruct))))
        printf("Out of space!!!");

    if (!((*H)->Elements = (ElemType *)malloc((MaxElem+1) * sizeof(ElemType))))
        printf("Out of space!!!");

    (*H)->Capacity = MaxElem;
    (*H)->Size = 0;
    (*H)->Elements[0] = MinData;
}

如果没有额外的间接级别，则在函数中更改的H将与函数隔离- 不会反映回调用者。

您可能还需要考虑其他几点：

• 你不应该投从返回malloc ，它可以隐藏你真的想知道某些错误。
• 如果第二个malloc失败，则应释放第一个malloc的结果。
• 如果您的任何一个malloc调用均失败，则应返回而不是继续，因为如果取消引用空指针，则继续将导致未定义的行为。
• 您可能不想从通用函数中打印内容，因为这可能是不想要的行为。 如果您必须指出问题，则最好将指示返回给呼叫者，让他们以自己的方式处理。

虽然说实话，但我实际上很喜欢返回值的版本（因为您显然是在创建新事物，因此无需事先传递它）。 这样的事情应该做：

PriorityQueue Initialize (int MaxElem) {
    PriorityQueue H;

    if (MaxElem < MinPQSize) {
        printf("Priority queue size is too small");
        return NULL;
    }

    if (!(H = malloc(sizeof(*H)))) {
        printf("Out of space!!!");
        return NULL;
    }

    if (!(H->Elements = malloc((MaxElem+1) * sizeof(ElementType)))) {
        printf("Out of space!!!");
        free (H);
        return NULL;
    }

    H->Capacity = MaxElem;
    H->Size = 0;
    H->Elements[0] = MinData;

    return H;
}

PriorityQueue myHeap = Initialize (MaxElement);

Answer 2

您正在按值传递指针，请允许我说明一下：

char* c = 0;

void set_c(char* ptr)
{
    ptr = (char*) malloc(sizeof(char) * 10);       
}

// a copy of c is sent in, 
set_c(c);
// c doesn't point to the newly allocated data!

要正确设置它，必须像下面这样传递您的指针BY指针：

void set_c_correctly(char** ptr)
{
   *ptr = (char*) malloc(sizeof(char) * 10);
}

// a pointer to c is passed in
set_c_correctly(&c);

// now c points to the newly allocated data