Ошибка сегментации после realloc(). Не удается назначить выделенную память указателю

Похоже, вы отредактировали свой код. Отредактированный код работает нормально.

#include<stdio.h>
#include<malloc.h>
#include<string.h>
// in header
typedef struct {
    int a;
    char test[20];
} MyContent;

typedef struct {
    MyContent* values; 
    // simmilar to: MyContent values[]
    // ... some other stuff
} MyData;

// in source
void myFunction(MyData* dataPtr) {
    dataPtr->values = NULL;
    MyData* tempPtr;

    for (int i = 1; i < 10; i++) {
        tempPtr = (MyData*) realloc(dataPtr->values, i * sizeof(MyContent));
        if (tempPtr == NULL) {
            if(dataPtr->values)
                free(dataPtr->values);
            printf("realloc() failed\n");
            return ;
        }
        dataPtr->values = (MyContent*)tempPtr;  // Here I get the segmentation fault
        dataPtr->values[(i-1)].a = 42+i;
        strcpy(dataPtr->values[(i-1)].test,"name");
    }
}

void PrintData(MyData* dataPtr) {
    for (int i = 1; i < 10; i++)
        printf("We have %s at %d\n",dataPtr->values[(i-1)].test,dataPtr->values[(i-1)].a);
}

main() {
    MyData Sample;
    myFunction(&Sample);
    PrintData(&Sample);
}

На первый взгляд, я не вижу проблемы, которая могла бы привести к сбою, - эта адресация на основе единиц немного странная, но не неправильная. В коде может быть проблема, которую вы не показываете, что приводит к повреждению кучи или стека, что усугубляется вызовом realloc. Или, если вы компилируете с оптимизацией, ваш отладчик может не понять, где на самом деле происходит сбой. Вы также путаете MyData и MyContent, но я предполагаю, что это только потому, что вы допустили ошибку при редактировании кода.

Также обратите внимание, что если realloc завершится ошибкой, произойдет сбой в строке после той, которую вы указали, так как вы будете писать в нулевой указатель. Вам нужно прервать, если tempPtr is NULL, а не просто освободить старый указатель. Опять же, это вызывает ошибку на другой линии, чем вы указали.

Я бы порекомендовал запустить вашу программу под управлением valgrind, чтобы увидеть, где она сообщает об ошибках - виновником, скорее всего, будет первая такая ошибка.

Ваше условие неверно для проверки вновь выделенной памяти. Должен быть:

if (tempPtr == NULL) {
  // handle error condition or continue with original 'dataPtr->values'
}
else {
  dataPtr->values = tempPtr;
}

Помните, что realloc() не обязательно переводит один блок в другой блок. Иногда он может выделять память в той же области указателя.