Использование malloc со структурами в c

Вам нужно выделить память как для записи в целом, так и отдельно для каждого поля. Например:

void string_realloc_and_copy (char **dest, const char *src)
{
  size_t len = strlen (src);
  *dest = realloc (*dest, len + 1);
  memcpy (*dest, src, len + 1);
}

typedef struct
{
  char *name;
  char *title;
} record;

record * record_new ()
{
  record *r = malloc (sizeof (record));
  r->name = NULL;
  r->title = NULL;
  return r;
}

void record_free (record *r)
{
  free (r->name);
  free (r->title);
  free (r);
}

void record_set_name (record *r, const char *name)
{
  string_realloc_and_copy (&r->name, name);
}

void record_set_title (record *r, const char *title)
{
  string_realloc_and_copy (&r->title, title);
}

Теперь, чтобы создать запись и заполнить ее значениями, прочитанными от пользователя:

record *r;
char buffer[100 + 1];

r = record_new ();

printf("Enter a first name \n");
if (scanf ("%100s", buffer) == 1) {
  record_set_name (r, buffer);
}

...

Есть некоторые проблемы здесь:

void setFirst(fr*friends, int* counter, int i) {
   // THE MALLOC FUNCTION!
   friends=(fr*) malloc(BUFFER*sizeof(fr));  <-- This is not doing what you're thinking

sizeof(fr) будет размером, необходимым для 4 указателей на символ. Например, если вы работаете на 32-битной платформе x86, для указателя на char требуется 4 байта, таким образом:

sizeof(fr) == 4 x 4 == 16 bytes

Итак, теперь вы выделяете 16*BUFFER или 16x50 = 800 байт. Это позволяет вам иметь массив из 50 структур 'fr'.

fr * friend
        |
        +--------> FirstName*
            |      LastName*
            |      home*
            |      cell*
            +----> FirstName*
            |       LastName*
            |      home*
            |      cell*
            ...

Итак, у вас есть память для 50 структур, но содержимое этих структур все еще не имеет памяти. Вам нужно выделить память для каждого члена структуры (и не забудьте также освободить их всех), или вы можете сделать их статическими членами с массивами вместо указателей.

Вторая проблема:

if(friends != NULL)  <-- if malloc was successful
{
     free(friends);  <-- release the memory

Ты только что потерял всех своих друзей. :)
Вам нужно освободить память, но в конце программы или в конце того места, где вы ее используете. Если вы назначите, а затем сразу же освободите, то память исчезнет, ​​и вы больше не сможете получить к ней доступ.

Здесь есть еще несколько вещей, которые следует учитывать, но для начала рассмотрим следующее.

В setFirst вы free обрабатываете свой friends буфер, по сути говоря: "Мне это больше не нужно". Когда вы делаете это, эта память просто уходит. Если вы собираетесь динамически выделять структуры для вызывающей стороны, вы должны либо предоставить отдельную функцию освобождения, либо сообщить пользователю, что он несет ответственность за очистку этой структуры.

Кроме того, вы всегда меняете только локальную копию указателя друзей. Если вы хотите указать указатель вызывающего объекта на новый буфер, вам нужно изменить тип аргумента на fr**.

Ваша структура содержит только указатели, а не выделенную память. Вам было бы лучше определить его для хранения массивов, в которые вы записываете имена и т.д.:

typedef struct friends_contact{

    char First_Name[20];
    char Last_Name[20];
    char home[20];
    char cell[20];
} fr;

Здесь я сделал каждое поле длиной 20 символов, но вы можете изменить это по своему усмотрению.

Редактировать: да, конечно, вы можете использовать динамическую память, но стоит ли заморачиваться? Преимущество динамических строк в том, что они могут быть точно нужного размера; вы можете сэкономить несколько байтов, и вы гарантируете, что сможете вписать имена в поля. Но много ли имен длиннее 20 символов и имеет ли смысл сокращать некоторые из них? С malloc есть много неудобных распределений (каждое из которых может дать сбой) и, конечно же, освобождения.

В качестве компромисса можно сделать телефонные номера фиксированного размера (они не меняются), а имена динамическими; затем распределите имена с помощью strdup (что также может привести к сбою).

typedef struct friends_contact{

    char *First_Name;
    char *Last_Name;
    char home[12];
    char cell[12];
} fr;