man getgrgid

RETURN VALUE

...

The return value may point to a static area which is overwritten by a subsequent call to getgrent(), getgrgid(), or getgrnam().

Попробуй юзать getgrgid_r

А как у вас выглядит getgrgid? И да, разве не надо до free(gr) сделать free(gr->gr_name) и т.д.?

Ты ее не выделил

Смотри внимательней.

[C] Освободить память занятую структурой.

При попытке освобождать сначала поля структуры:
free(gr->gr_name);
free(gr->gr_passwd);
Та же история:
*** glibc detected *** ./gr: free(): invalid pointer: 0x0000000012d26016 ***

Посмотрел man. Согласен с самым первым ответом. Вы вообще уверены, что тому, что возвращает getgrgid можно делать free?

А список gr_mem вы как освобождать собираетесь?

Бл*дь, ты ман то уже почитай по этой функции, что тебе указатель на структуру возвращает. Я тебе в первом посте уже процитировал, что далеко не факт, что там память вообще динамически аллоцируется, чтобы потом её free подсовывать.

if( (gr=getgrgid(gid)) == NULL)
perror("GetGrGid ");
fprintf(stdout,"Name: %s\n",gr->gr_name);
free(gr);
return 0;
}

а можно вопрос? данная ветка кода выполняется только в том случае если у Вас (gr == NULL), то есть структура не создана, вот нафига её удалять?

и да, на будущее по code style, free() освобождает память, но не чистит значение указателя и он указывает, после free(), туда же куда и указывал, так что, хоть free() и проверяет на то что значение указателя не NULL, при попытке почистить второй раз будете получать ошибку

лучше пишите

free(gr);
gr = NULL;

если у Вас (gr == NULL), то есть структура не создана, вот нафига её удалять?

Это он просто так код оформил красиво, под if там только perror стоит.

и да, судя по отсутствию специализированной функции для удаления данной структуры, её чистка вообще не предполагается, то есть это просто набор указателей на внутренние структуры данных

>>Бл*дь, ты ман то уже почитай по этой функции, что тебе указатель на структуру возвращает.
Бл*дь, спасибо, открыл глаза!

если у Вас (gr == NULL), то есть структура не создана, вот нафига её удалять?

Это он просто так код оформил красиво, под if там только perror стоит.

от же блджад! с набегу подумал что опечатка, вообще я за такое по чайнику бил, виртуально конечно

смотри внимательно

if( (gr=getgrgid(gid)) == NULL) {
       perror("GetGrGid ");
}

fprintf(stdout,"Name: %s\n",gr->gr_name);

смотри внимательно

if( (gr=getgrgid(gid)) == NULL) { 
       perror("GetGrGid "); 
} 
 
fprintf(stdout,"Name: %s\n",gr->gr_name); 

в Вашем коде вижу, у ТС, согласитесь, несколько по-другому

вообще где-то видел определение качества кода, как количество «WTF???» отнесённое к количеству строк

в Вашем коде вижу, у ТС, согласитесь, несколько по-другому

Вообще-то там тоже явно видно, просто нет отступов нормальных:

...
    if( (gr=getgrgid(gid)) == NULL)
        perror("GetGrGid ");
    fprintf(stdout,"Name: %s\n",gr->gr_name);
...

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

кстати, этот кусок кода - детский способ регулярно ломать себе ноги:

if( (gr=getgrgid(gid)) == NULL) { 
       perror("GetGrGid "); 
} 
 
fprintf(stdout,"Name: %s\n",gr->gr_name); 

потому что, даже если внезапно выяснится что структура NULL, то всё равно происходит обращение к её полям

Точно, для автоматического exit'а нужно какой-нибудь err использовать.

Вообще-то там тоже явно видно, просто нет отступов нормальных:

вот как раз без отступов ничерта и не видно, и вообще, такой код:

    if( (gr=getgrgid(gid)) == NULL) 
        perror("GetGrGid "); 

это пример очень плохого code style, и является источником мириадов факапов (вот таких как у меня случился, к примеру :)

Так понятнее?

да мне уже всё давно понятно, ещё с тех пор как dmsh указал мне на это

> при попытке почистить второй раз будете получать ошибку лучше пишите

А еще лучше разбираться в коде. Если у вас где-то бесконтрольно освобождаются одни и те же данные, то есть подозрение, что проблема несколько больше, чем необнуление указателя.

при попытке почистить второй раз будете получать ошибку лучше пишите

А еще лучше разбираться в коде. Если у вас где-то бесконтрольно освобождаются одни и те же данные, то есть подозрение, что проблема несколько больше, чем необнуление указателя.

на то есть code review, а зануление указателей позволяет сократить количество мистических ошибок и время их вылавливания

вообще RTFM.

у меня в мане написано

The return value may point to a static area, and may be overwritten by subsequent calls to getgrent(3), getgrgid(), or getgrnam(). (Do not pass the returned pointer to free(3).)