Segmentation Fault in strcpy()

Your typedef defines Truck as a struct struck *, i.e. a pointer. So it’s size will be 4 or 8 depending on the architecture and not the size of the struct

Use sizeof(*Truck) to get the actual size of the struct.

You also need to allocate memory for the characters. The easiest way would be using strdup().

Truck CTruck(const char* id, const char* mat, int value, const char* place) {
    Truck nT = malloc(sizeof (*Truck));
    nT->value = value;
    nT->id = strdup(id);
    nT->mat = strdup(mat);
    nT->place = strdup(place);
    return nT;
}

However, I would suggest changing your typedef so it’s an alias for the struct, not for a pointer to it:

typedef struct {
    char* id;
    char* mat;
    int value;
    char* place;
} Truck;

In your function you then use this:

Truck *nT = malloc(sizeof(Truck));

Leave a Comment