Ну вот вникай или не кастуй)

Написав однозв'язний список. Чи оптимально я виділив пам'ять?
typedef struct node {
 int number;
 char *name;
 struct node *next;
} node;
node *init(int a, char *g)
{
 node *temp = (node *) malloc(sizeof(int) + sizeof(node *));
 temp->name = (char *) malloc(sizeof(g));
 temp->number = a;
 for(int i = 0; g[i] != '\n'; i++)
   temp->name[i] = g[i];
 temp->next = NULL;

 return (temp);
}
void pop(node **h)
{
 node *temp = (*h)->next;
 if(*h) free(*h);
 (*h) = temp;

 return;
}

Здається десь витік пам'яті

Не вистача

є досвіду

бросилось в глаза сразу копирование строки. Используй если не требуют другого функцию с string.h
char *strcpy(char *toHere, const char *fromHere);

char *strcpy (char *dst, const char *src);
dst — указатель на буфер назначения.
src — указатель на исходную строку.

так понятней

Дякую за пораду, але як виділяти пам'ять?

на С пишешь?

Поки так

будет напечатано 4 4 4, потому что в твоем случае sizeof(g) - g это указатель, а размер указателя постоянное число - 4))

уже должно натолкнуть тебя на умные мысли)

node *temp = (node *) malloc(sizeof(int) + sizeof(node *));

Ви виділаєте недостатньо пам'яті для структури (треба ще під  name )

Як вже вказали, тут

temp->name = (char *) malloc(sizeof(g));

Ви завжди виділятимете пам'ять лише під вказівник g, для всієї строки треба знайти її довжину + 1 і виділити відповідну к-сть байт.

В pop() ви звільняєте пам'ять ноди

if(*h) free(*h);

Але перед цим забуваєте звільнити пам'ять, виділену для (*h)->name

Про (*h)->name я знав, але про вказівник - ні

Зараз пограюсь з sizeof()

а й дійсно

ДУЖЕ дякую

Привіт, чуваки. Якщо вже зайшла мова про пам*ять, то прошу допомоги і для початківця) Під час виконання коду(кину файл далі) вилітає  така помилка. З роботою з пам*яттю ніби усе окей)