Является ли этот код непереносимым или небезопасным

Question

Я пишу код, зависящий от calloc, и задавался вопросом, было бы безопасно переназначить указатель на пространство стека, если calloc не удалось, а затем установить указатель на NULL перед вызовом free() или вообще пропустить его. Прекрасно работает с моей 386-й линейкой.

char *str = NULL; 
int usestackspace = 0; 
char str1[16] = {0}; 

str = (char *)calloc(1, sizeof(pid_t)); 

if (str == NULL) { 
    sleep(1); 
    str = (char *)calloc(1, sizeof(pid_t)); 
} 

if (str == NULL) { 
    fprintf(stderr, "watchdog: %s\n", strerror(errno)); 
    usestackspace = 1; 
    str = str1; 
} 

if (str == NULL) { 
    return -1; 
} 

Answer 1

Пока str1 не выходит за рамки код довольно много хорошо. У вас есть одна реальная ошибка (которая должна быть генерируя предупреждение, хотя это, вероятно, не имеет значения во время выполнения):

• str = &str1 не является допустимым присваивание без явного приведения. Вероятно, вы хотите str = str1.

Одна из возможных проблем:

• Если вы были в зависимости от неявного параметра к нулю памяти по calloc, вам необходимо инициализировать str1. Используйте char str1[16] = { 0 } или позвоните по телефону memset, например.

И пару мелких нот:

1. Вы не должны отбрасывать возвращаемое значение calloc в программе C.

2. Вы free(str) и str = NULL в if заявлении, но и нет-OPS - условие if оператор гарантирует, что str уже являетсяNULL.