malloc not NULL, still get segmentation fault

chubbs

Hi,
Please can you help me?

I have the following code:

Expand|Select|Wrap|Line Numbers

  
  int** adjacencyOneMode;

  int *dim = 3800;

  int iv=0;
 
  /*initalise adjacency for the one mode projection*/

  if((adjacencyOneMode = (int **)malloc((*dim)*sizeof(int*))) == NULL){

    printf("\nERROR: Failure to initalize adjacency array\n");

  }else{

    printf("allocated memory for adjacency one mode first dime\n");

  }
 
  for(iv=0; iv<*dim; iv++){

    printf("Here = %d\n", iv);

    adjacencyOneMode[iv] = (int *)(malloc((*dim)*sizeof(int)));

    printf("allocated aom[%d]\n", iv);

    if(adjacencyOneMode[iv] == NULL){

      printf("ERORR - cannot allocate memory for aom[] = NULL\n");

    }

  }

  printf("Finished adjacency OneMOde\n");

[OUTPUT]

Here 0
allocated aom[0]
Here 1
allocated aom[1]
Here 2
allocated aom[2]
Here 3
allocated aom[3]
Here 4
allocated aom[4]
Here 5
allocated aom[5]
Here 6

[/OUTPUT]

I run the program that contains this code a number of times for different conditions - sometimes it works sometimes it doesn't work. This is the segment of code where I get a segmentation fault when it doesn't work. I'm sorry but its really not obvious to me why its not working and I'm pulling my hair out. On the 7th iteration it is not outputting that malloc returned NULL.

Could it be an error somewhere else and the problem is only manifesting itself here? I do allocate a lot of memory in the program before this stage, although I always check that it was allocated correctly and free it when no longer needed.

Any help would be greatly appreciated.
Sorry if the answer is really obvious.

chubbs.

weaknessforcats

You are accessing memory outside your allocation or you are using a null or uninitialized pointer.

Silent1Mezzo

Umm, generally you should actually post some useful information (maybe the code that's seg faulting or something like that). weaknessforcats is totally correct though, you're probably accessing memory outside of the allocation.

If you post more information we can help you further.

chubbs

Quote:

Originally Posted by weaknessforcats

You are accessing memory outside your allocation or you are using a null or uninitialized pointer.

Yes I know, but when I code:

Expand|Select|Wrap|Line Numbers

 
if((adjacencyOneMode = (int **)malloc((*dim)*sizeof(int*))) == NULL){

    printf("\nERROR: Failure to initalize adjacency array\n");

  }else{

    printf("allocated memory for adjacency one mode first dime\n");

  }

and this allocates memory for adjacencyOneMode, does this not allocate enough memory for?:

Expand|Select|Wrap|Line Numbers

 
 for(iv=0; iv<*dim; iv++){

    printf("Here %d\n", iv);

    adjacencyOneMode[iv] = (int *)(malloc((*dim)*sizeof(int)));

    printf("allocated aom[%d]\n", iv);

    if(adjacencyOneMode[iv] == NULL){

      printf("************ERORR - cannot allocate memory for aom[] = NULL\n");

    }

 }

So I guess what I am trying to say is - I thought I had initalized the pointer and I am accessing memory within the allocation. Am I not? It only does 6 iterations of the for loop - when I set the *dim = 3000? If I am not allocating them correctly can you tell from this snippet where I am going wrong?

Help is very much appreciated!!

chubbs

Expand|Select|Wrap|Line Numbers

 
 for(iv=0; iv<*dim; iv++){

    printf("Here %d\n", iv);

    adjacencyOneMode[iv] = (int *)(malloc((*dim)*sizeof(int)));

    /** THE CODE IS SEG FAULTING HERE**/

    printf("allocated aom[%d]\n", iv);

    if(adjacencyOneMode[iv] == NULL){

      printf("************ERORR - cannot allocate memory for aom[] = NULL\n");

    }

 }

Output:

Here 0
allocated aom[0]
Here 1
allocated aom[1]
Here 2
allocated aom[2]
Here 3
allocated aom[3]
Here 4
allocated aom[4]
Here 5
allocated aom[5]
Here 6

Silent1Mezzo

Ok I'm confused as to why you're dereferencing dim.

Expand|Select|Wrap|Line Numbers

 
int main() {

    int dim = 10;

    int iv;

    int **adjacencyOneMode;

    adjacencyOneMode = (int **)malloc((dim)*sizeof(int*)) ;
 
     if(adjacencyOneMode == NULL){

          printf("\nERROR: Failure to initalize adjacency array\n");

        }else{

          printf("allocated memory for adjacency one mode first dime\n");

        }
 
    for(iv=0; iv<dim; iv++){

        printf("Here %d\n", iv);

        adjacencyOneMode[iv] = (int *)(malloc((dim)*sizeof(int)));

        printf("allocated aom[%d]\n", iv);

        if(adjacencyOneMode[iv] == NULL){

        printf("************ERORR - cannot allocate memory for aom[] = NULL\n");

        }

    }

    return (1);

}

I used this and it compiles/runs find.

chubbs

Hi,

Thanks for your help. I was using "dim" as a pointer because I was reading the value in from another program.
It turns out this isn't were the problem was after all.
Before this point in the program, I was allocating alot of memory to store data read in from a number of large files. It seems at this point in the program I reached the memory limit. I've rewritten the program more efficently, and no longer get the error.

Thanks.

malloc not NULL, still get segmentation fault

re: malloc not NULL, still get segmentation fault

re: malloc not NULL, still get segmentation fault

re: malloc not NULL, still get segmentation fault

re: malloc not NULL, still get segmentation fault

re: malloc not NULL, still get segmentation fault

re: malloc not NULL, still get segmentation fault

Similar C / C++ bytes

/bytes/development

/bytes/it

/bytes/about