asprintf(): как освободить указатели?

Question

У меня есть такой код: я дважды назначил журнал, есть ли потенциальная утечка памяти для первого журнала &?

char *log = NULL; 
asprintf(&log, "Hello: %s", name); 
if (known_person== true){ 
    asprintf(&log, "%s, %s", log, ", my old friend."); 
} 
free (log); 

Answer 1

Да, код будет течь, поскольку asprintf не проверяет и не пытается повторно использовать предыдущий указатель. Следовательно, память просто теряется. Лучший способ избежать этой проблемы в вашем примере будет переписать код, как

char *log = NULL; 
if (known_person== true) 
    asprintf(&log, "Hello: %s, my old friend.", name); 
else 
    asprintf(&log, "Hello: %s", name); 

free (log); 

Таким образом, буфер выделяется один раз и освобождается правильно.

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

char *temp = NULL; 
asprintf(&temp, "Hello: %s", name); 

char *log = NULL; 
if (known_person== true) { 
    asprintf(&log, "%s, my old friend.", temp); 
    free(temp); 
} 
else { 
    log = temp; 
} 

free (log); 

Answer 2

Да. Вероятно, asprinf не попадет в одно и то же место памяти и не знает о предыдущих вызовах.

Answer 3

находится там [а] [...] утечку памяти

Определенно да.

Ссылка на память, выделенную на 1-ом вызове aprintf() получает перезаписаны 2 вызова aprintf(), так что нет никаких шансов на free() 1-й выделенной памяти больше, это «утечка».

Чтобы исправить это ввести 2-й (временный) указатель:

char name[] = "C"; 
char * log = NULL; 

{ 
    char * log_tmp = NULL; 

    asprintf(&log_tmp, "Hello: %s", name); 
    if (known_person == true) 
    { 
    asprintf(&log, "%s, %s", log_tmp, ", my old friend."); 
    free(log_tmp); 
    } 
    else 
    { 
    log = log_tmp; 
    } 
} 

/* Use log. */ 

free(log); 

Другой и, вероятно, дешевле (быстрее, так как некоторые вещи обрабатывать во время компиляции, но время выполнения) подход к этой проблеме было бы следующие:

#define FORMAT_STR "Hello: %s" 
#define FORMAT_SUFFIX_STR ", my old friend." 

... 

char name[] = "C"; 
char * log = NULL; 

{ 
    char format[sizeof FORMAT_STR""FORMAT_SUFFIX_STR + 1] = FORMAT_STR; 

    if (known_person == true) 
    { 
    strcat(format, FORMAT_SUFFIX_STR); 
    } 

    asprintf(&log, format, name); 
} 

/* Use log. */ 

free(log); 

Добавление проверки ошибок в системные вызовы оставляется читателю в качестве упражнения.

3-й подход к этому только с использованием функций стандарта C является:

char name[] = "C"; 
char * log = NULL; 

{ 
    char * log_tmp = NULL; 

    asprintf(&log_tmp, "Hello: %s", name); 
    if (known_person == true) 
    { 
    asprintf(&log, "%s, %s", log_tmp, ", my old friend."); 
    free(log_tmp); 
    } 
    else 
    { 
    log = log_tmp; 
    } 
} 

/* Use log. */ 

free(log); 

Другой и, вероятно, дешевле (быстрее, так как некоторые вещи обрабатывать во время компиляции, но время выполнения) подход к этой проблеме будет следующее:

#define FORMAT_STR "Hello: %s" 
#define FORMAT_SUFFIX_STR ", my old friend." 

... 

char name[] = "C"; 
char * log = NULL; 

{ 
    char format[sizeof FORMAT_STR""FORMAT_SUFFIX_STR + 1] = FORMAT_STR; 

    if (known_person == true) 
    { 
    strcat(format, FORMAT_SUFFIX_STR); 
    } 

    { 
    int s = snprintf(NULL, 0, format, name); 
    if (-1 == s) 
    { 
     /* failure */ 
    } 
    else 
    { 
     log = malloc(s + 1); 
     if (NULL == log) 
     { 
     /* failure */ 
     } 
     else 
     { 
     if (-1 == sprintf(log, format, name)) 
     { 
      /* failure */ 
     } 
     } 
    } 
    } 
} 

free(log);