allocate memory in function

[Pushker Pradhan]

This is similar to my post a few days ago, I'm now doing it correctly but
somehow I get Segmentation errors:
My function gen_matrix allocates memory for x and returns x with some
content:
void gen_matrix(int **x, int numRows, int numCols)
{
long i;
*x = (int *)calloc( (numRows*numCols), sizeof(int) );
for(i = 0; i < numRows*numCols; i++) {
*x = rand();
}
}

Call in main.c:
int *mat = NULL;
int numrows, numcols;
numrows = atoi(argv[1]);
numcols = atoi(argv[2]);
gen_matrix(&mat, numrows, numcols);

This is the error (the memory allocation: calloc fails!):
psp1@leto:~/pdc$ ./main 4 4
Bus Error (core dumped)

Can somebody tell what's wrong?

 

[Richard Heathfield]

This is similar to my post a few days ago, I'm now doing it correctly but
somehow I get Segmentation errors:
My function gen_matrix allocates memory for x and returns x with some
content:
void gen_matrix(int **x, int numRows, int numCols)
{
long i;
*x = (int *)calloc( (numRows*numCols), sizeof(int) );

Unnecessary cast. May be hiding a failure to #include <stdlib.h>

No check for success of call.

*x = calloc(numRows * numCols, sizeof **x);
if(*x != NULL)
{

for(i = 0; i < numRows*numCols; i++) {
*x = rand();

You meant: (*x) = rand();

 

[Pushker Pradhan]

Actually I tried without the cast but still get the same errors, yes I have
included stdlib.h
I now check for *x == NULL but the calloc fails for some other reason it
seems.

--
Pushkar Pradhan

This is similar to my post a few days ago, I'm now doing it correctly but
somehow I get Segmentation errors:
My function gen_matrix allocates memory for x and returns x with some
content:
void gen_matrix(int **x, int numRows, int numCols)
{
long i;
*x = (int *)calloc( (numRows*numCols), sizeof(int) );

Unnecessary cast. May be hiding a failure to #include <stdlib.h>

No check for success of call.

*x = calloc(numRows * numCols, sizeof **x);
if(*x != NULL)
{

for(i = 0; i < numRows*numCols; i++) {
*x = rand();

You meant: (*x) = rand();

--
Richard Heathfield : (e-mail address removed)
"Usenet is a strange place." - Dennis M Ritchie, 29 July 1999.
C FAQ: http://www.eskimo.com/~scs/C-faq/top.html
K&R answers, C books, etc: http://users.powernet.co.uk/eton

 

[Pushker Pradhan]

Please ignore this post, Richard's reply was helpful. Thank you

--
Pushkar Pradhan

Actually I tried without the cast but still get the same errors, yes I have
included stdlib.h
I now check for *x == NULL but the calloc fails for some other reason it
seems.

--
Pushkar Pradhan

This is similar to my post a few days ago, I'm now doing it correctly but
somehow I get Segmentation errors:
My function gen_matrix allocates memory for x and returns x with some
content:
void gen_matrix(int **x, int numRows, int numCols)
{
long i;
*x = (int *)calloc( (numRows*numCols), sizeof(int) );

Unnecessary cast. May be hiding a failure to #include <stdlib.h>

No check for success of call.

*x = calloc(numRows * numCols, sizeof **x);
if(*x != NULL)
{

for(i = 0; i < numRows*numCols; i++) {
*x = rand();

You meant: (*x) = rand();

--
Richard Heathfield : (e-mail address removed)
"Usenet is a strange place." - Dennis M Ritchie, 29 July 1999.
C FAQ: http://www.eskimo.com/~scs/C-faq/top.html
K&R answers, C books, etc: http://users.powernet.co.uk/eton