Возможный дубликат:
Есть ли бесплатный (ptr), где ptr является NULL поврежденной памятью?
Я пишу функцию C, которая освобождает указатель, если он был malloc() ed. Указатель может быть либо NULL (в случае, когда произошла ошибка, так и у кода не было возможности выделить что-либо) или выделено с помощью malloc(). Можно ли использовать free(ptr); вместо if (ptr != NULL) free(ptr);?
gcc вообще не жалуется даже на -Wall -Wextra -ansi -pedantic, но это хорошая практика?
Цитирование стандарта C, 7.20.3.2/2 из ISO-IEC 9899:
void free(void *ptr);
Если
ptr- нулевой указатель, никаких действий не происходит.
Не проверяйте NULL, он только добавляет больше фиктивного кода для чтения и, следовательно, является плохой практикой.
Однако при использовании malloc и co, вы должны всегда проверять указатели NULL. В этом случае NULL означает, что что-то пошло не так, скорее всего, нет памяти.
Не рекомендуется проверять NULL перед вызовом free. Проверка просто добавляет ненужный беспорядок в ваш код, и free(NULL) гарантированно будет в безопасности. Из раздела 7.20.3.2/2 стандарта C99:
Свободная функция заставляет пространство, на которое указывает ptr, освободиться, т.е. сделано доступный для дальнейшего распределения. Если ptr является нулевым указателем, никаких действий не происходит.
Случайно googling появляется http://linux.die.net/man/3/free, который гласит:
Если ptr равно NULL, операция не выполняется.
По-моему, нет, по крайней мере, не в вашем случае.
Если вы не могли выделить память, вы должны были проверить этот WAY перед вызовом бесплатного.