Thread: malloc problem

Hi,

I have a serious problem with my allocation of memory, and quite frankly I have no clue why the following code won't work.

Code:

...
	  if ((dMob = malloc(sizeof(*dMob))) == NULL)
	  {
		  log(LOG_E, "load_profile: Cannot allocate memory.");
		  abort();
	  }
	  if ((dMob->pcdata = malloc(sizeof(*dMob->pcdata))) == NULL)
	  {
		  log(LOG_E, "load_profile: Cannot allocate memory.");
		  abort();
	  }
	  dMob->pcdata->password="Test.";  /*   char *password;    */

...

It crashes, and throws a segmentation fault after getting to that last statement there. So whats wrong with this thing? Its not having problems allocating the memory. If I remove the last line, the program continues on fine. Buts its like pcdata wasn't allocated, even though I just allocated it.

Any help is greatly appreciated. Thank you very much.

-Jase

What kind of data type is pc_data. Is that a pointer to the same structure? If so, you should use sizeof() the whole structure and not the sizeof() a pointer in the structure. Is this going to be a linked list?

Yes, its a linked list. To give you an idea of my structures...

Code:

struct DMOB
{
     char     *name;
     ...
     PC_DATA  *pcdata;
};

struct PC_DATA
{
     char   *password;
};

Like I said, it doesn't seem to have a problem allocating the memory, but when I try to access it, it acts like it wasn't allocated in the first place. -ponder- I'm going to try an if check after the allocation to see if returns NULL or not.

I found the problem. It had to do with the function clear_mobile which made dMob equal an empty data structure of DMOB, and thus pcdata was allocated, but when dMob took on the values of DMOB *clear_mobile; it made pcdata=NULL.

Thanks for your help.