Должен ли я проверить, успешно ли malloc()?

Question

Следует ли проверять после каждого malloc(), если он был успешным? Возможно ли, что malloc() терпит неудачу? Что происходит тогда?

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

arr = (int) malloc(sizeof(int)*x*y); 
if(arr==NULL){ 
    printf("Error. Allocation was unsuccessful. \n"); 
    return 1; 
} 

Какова практика в отношении этого? Могу ли я сделать это таким образом:

if(!(arr = (int) malloc(sizeof(int)*x*y)) 
    <error> 

Answer 1

Нет необходимости бросить malloc(). Да, необходимо проверить, был ли malloc() успешным или нет. Скажем, malloc() не удалось, и вы пытаетесь получить доступ к выделенной памяти указателя. Это приведет к сбою. Так что лучше поймать выделение памяти, прежде чем получить доступ к указателю.

int *arr = malloc(sizeof(int)); 
if(arr == NULL) 
{ 
printf("Memory allocation failed"); 
return; 
} 

Answer 2

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

Лично я часто обойдусь этим, используя небольшую обертку malloc, которая никогда не потерпит неудачу. Если ваше программное обеспечение не является устойчивой, безопасность критической системы не может нормально работать вокруг таНоса неисправного в любом случае, так что я хотел бы предложить что-то вроде этого:

static inline void *MallocOrDie(size_t MemSize) 
{ 
    void *AllocMem = malloc(MemSize); 
    /* Some implementations return null on a 0 length alloc, 
    * we may as well allow this as it increases compatibility 
    * with very few side effects */ 
    if(!AllocMem && MemSize) 
    { 
     printf("Could not allocate memory!"); 
     exit(-1); 
    } 
    return AllocMem; 
} 

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

Для более общего решения для функций, которые могут не я также, как правило, для реализации простой macrosuch как это:

#define PrintDie(...) \ 
    do \ 
    { \ 
    fprintf(stderr, __VA_ARGS__); \ 
    abort(); \ 
    } while(0) 

Который затем позволяет запускать функцию как:

if(-1 == foo()) PrintDie("Oh no"); 

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