C, Segmentation fault while using dynamic array in struct

Question

I'm trying to add new element to dynamic array in C (I know that I must free all memory. I will do it later), but I get this error every time:

But, what is strange, if I compile from terminal, like that, code works properly.

So, where is the error and how i can beat it? Thank you!

All my code:

main.c

    #include <stdio.h>
#include <stdlib.h>

typedef struct vector
{
    int size;
    int *array;
    int alreadyIn;
}vector;

vector *vectorInit(int size)
{
    vector *newVec = (vector *)malloc(sizeof(vector));
    if(!newVec){printf("No memory!\n"); return NULL;}
    newVec->size = size;
    newVec->array = (int *)malloc(size * sizeof(int));
    return newVec;
}

void allocNewMemory(vector *vect, int howMuch)
{
    vect->array = (int *)realloc(vect->array ,(vect->size + howMuch) * sizeof(int));
    vect->size += howMuch;
}

void pushBack(vector *vect, int number)
{
    int howMuch = 5;
    if(vect && vect->alreadyIn < vect->size)
    {
        vect->array[vect->alreadyIn] = number;
        vect->alreadyIn++;
    }
    else
    {
        printf("Alloc new memory for %d elements...\n", howMuch);
        allocNewMemory(vect, howMuch);
        pushBack(vect, number);
    }
    
}

void printVector(vector *vect)
{
    for (int i = 0; i < vect->alreadyIn; i++)
    {
        printf("%d ", vect->array[i]);
    }
    printf("\n");
}

int main()
{
    int startSize = 4;
    vector * vec = vectorInit(startSize);

    for (int i = 0; i < 6; i++)
    {
        pushBack(vec, i+1);
    }
    

    printVector(vec);


    return 0;
}

You are not initializing your alreadyIn member. malloc() does not guarantee that all returned memory is initialized with zeros. It contains garbage. So alreadyIn is garbage. Hint: learn to use a debugger. You will make your life a lot easier. — Mike Nakis
– Mike Nakis, Commented Jan 6, 2021 at 21:38
Looks like you are already in the debugger. Learn to use it effectively. In this case, dump the variables at the point of the error and look for values that are incorrect. — kaylum
– kaylum, Commented Jan 6, 2021 at 21:39

Some programmer dude · Accepted Answer · 2021-01-06 21:38:22Z

2

You never initialize the alreadyIn member in the structure. That means its value will be indeterminate (and seemingly garbage or random).

You need to explicitly initialize it to zero:

vector *vectorInit(int size)
{
    vector *newVec = malloc(sizeof(vector));
    if(!newVec)
    {
        printf("No memory!\n");
        return NULL;
    }
    newVec->size = size;
    newVec->array = malloc(size * sizeof(int));
    newVec->alreadyIn = 0;  // Remember to set this to zero
    return newVec;
}

This problem should have been easy to detect in the debugger.

Also note that I removed the casts from malloc. One should not cast the result of malloc, or really any function returning void *.

answered Jan 6, 2021 at 21:38

Some programmer dude

411k36 gold badges420 silver badges655 bronze badges

Sign up to request clarification or add additional context in comments.

Collectives™ on Stack Overflow

C, Segmentation fault while using dynamic array in struct

1 Answer 1

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related