segmentation fault in during dynamic memory allocation with malloc

Question

This might be very silly question to ask.
I am using malloc to memory allocation.
Program compiles fine but segmentation fault occurs while launching.
Here is min code:

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

int main()
{
    int row,col,i;
    int **mat;
    row = 3; //made row as static for testing
    col = 4; //made col as static for testing
    *mat = (int *)malloc(sizeof(int)*row);
    for (i=0;i<row;i++)
       mat[i] = (int *) malloc (sizeof(int)*col);
}

i compiled with: gcc -ggdb test.c
on gdb its giving:

(gdb) run
Starting program: /slowfs/swe91/abjoshi/clients/fw_cl_for_seg_fault/a.out

Program received signal SIGSEGV, Segmentation fault.
0x00000000004004cc in main () at 1test.c:10
10          *mat = (int *)malloc(sizeof(int)*row);

Note: gcc version 4.5.2

*mat = (int *)malloc(sizeof(int)*row); mat doesn't point to anything yet. — Charlie Burns
– Charlie Burns, Commented Sep 18, 2013 at 18:21
@CharlieBurns: that's what i am assigning memory to pointer then i will store values in that memory location. Please let me know if i am missing something. — Abhi
– Abhi, Commented Sep 18, 2013 at 18:25

AnT stands with Russia · Accepted Answer · 2013-09-18 18:33:38Z

10

The common malloc idiom is as folows: to allocate an array of n elements pointed by pointer p use the following universal syntax

p = malloc(n * sizeof *p);

That will help you to avoid errors as the one in your code. Note the key points here: don't cast the result of malloc and don't use types under sizeof. And in general: as much as possible, avoid mentioning types anywhere outside declarations.

In your case the first allocation should look as

mat = malloc(row * sizeof *mat);

The allocations inside the cycle should look as

for (i = 0; i < row; ++i)
   mat[i] = malloc(col * sizeof *mat[i]);

The important benefit of this approach is that the code is type-independent: at any moment you can change the declaration of mat from int **mat to, say, double **mat, but the allocation code will continue to remain valid - you won't have to make any changes there.

edited Sep 18, 2013 at 18:33

answered Sep 18, 2013 at 18:28

AnT stands with Russia

323k44 gold badges548 silver badges793 bronze badges

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

1 Comment

sai Over a year ago

If I am assigning the void pointer returned from malloc directly to the array pointer, then I am getting the following error: "error: invalid conversion from ‘void*’ to ‘float*’ [-fpermissive]". How do I resolve it?

Charlie Burns · Accepted Answer · 2013-09-18 18:29:23Z

3

Instead of

*mat = (int *)malloc(sizeof(int)*row);

I think you want

mat = (int **)malloc(sizeof(int *) * row);

As it is, you are dereferencing mat which is uninitialized. You need to malloc() mat before you deference it.

You can leave the cast off the malloc as well:

mat = malloc(sizeof(int *) * row);

As well as the sizeof(int *) reference:

mat = malloc( row * sizeof(*mat));

edited Sep 18, 2013 at 18:29

answered Sep 18, 2013 at 18:23

Charlie Burns

7,05623 silver badges29 bronze badges

Comments

Victor von Cacahuete · Accepted Answer · 2013-09-19 20:58:26Z

2

Compiler would have told it with -Wall

$ gcc -Wall y.c
y.c: In function ‘main’:
y.c:14:1: warning: control reaches end of non-void function [-Wreturn-type]
y.c:11:8: warning: ‘mat’ is used uninitialized in this function [-Wuninitialized]

edited Sep 19, 2013 at 20:58

answered Sep 18, 2013 at 18:29

Victor von Cacahuete

1946 bronze badges

Comments

Andy · Accepted Answer · 2013-09-18 18:23:37Z

1

In the line *mat = (int *) malloc(sizeof(int)*row); you are trying to dereference mat. mat is currently uninitialized.

answered Sep 18, 2013 at 18:23

Andy

7942 gold badges6 silver badges22 bronze badges

Comments

AbhinavChoudhury · Accepted Answer · 2013-09-18 18:34:27Z

0

Here's what:

mat is a double pointer and it is uninitialized (or NULL). When you dereference it:

*mat = (int *)malloc(sizeof(int)*row);

you are dereferencing a NULL value and trying to assign to it, causing the segmentation fault.

First initialize mat with a pointer to list of (int*) pointers. Then individually take each pointer in the list and allocate (int) space for each.

mat = (int *)malloc(cols*sizeof(int *));
for (i=0;i<cols;i++)
    mat[i] = malloc(rows*sizeof(int))

answered Sep 18, 2013 at 18:34

AbhinavChoudhury

1,2172 gold badges20 silver badges38 bronze badges

Collectives™ on Stack Overflow

segmentation fault in during dynamic memory allocation with malloc

5 Answers 5

1 Comment

Comments

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

5 Answers 5

1 Comment

Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related