(no subject)

Когда мне на code review отдают подобные piece of art (несущественные подробности опущены) :

 #define HELLO_WORLD_STRING_LENGTH 13
 process_main_loop() {
 while(1) {
 char* p = malloc(HELLO_WORLD_STRING_LENGTH); // HELLO_WORLD_STRING_LENGTH is 13
 if (NULL == p) {
 printf("Error");
 continue;
 }
 memset(p, 0, len);
 strcat(p, "Hello World!");
 if (0 != flag_it_is_time_to_print) { // print only if required
 printf("%s", p);
 }
 free(p);
 }
 }
 

, меня охватывает тихое отчаяние.

Здесь нет ошибок. Я не могу тыкнуть пальцем в конкретную строку, которую надо бы изменить. И я не могу (по этическим причинам) посоветовать автору удалить файл и убиться об стену. Я просто не знаю, что писать в review.