0

Есть указатель на массив чар, к примеру char *content;

Нужно его заполнить информацией, что будет правильнее?

  1. content = strdup("TEXT"); free(content);

  2. content = malloc(strlen("TEXT") + 1); strcpy(content, "TEXT"); free(content);

Что в первом, что во втором варианте valgrind ругается на отсутствие освобождение памяти при завершении программы.

Вариант char *content = "TEXT"; не подходит из-за того что переменная content часть структуры, которая многократно изменяется в ходе работы.

4
  • 2 - по сути просто реализация 1. Так что используйте, что вам внутренне ближе :) А проблема - лично у меня подозрение, что она связана не с применением этих функций, а с чем-то иным...
    – Harry
    14 мая 2019 в 8:17
  • 10 bytes in 2 blocks are possibly lost in loss record 31 of 1,499 at 0x483877F: malloc (vg_replace_malloc.c:299) by 0x50EA6FE: strdup (in /usr/lib/libc-2.29.so) by 0x48926A0: json_object_object_add_ex (in /usr/lib/libjson-c.so.4.0.0) by 0x489522A: json_tokener_parse_ex (in /usr/lib/libjson-c.so.4.0.0) by 0x48969E8: json_tokener_parse_verbose (in /usr/lib/libjson-c.so.4.0.0) by 0x4896A4E: json_tokener_parse (in /usr/lib/libjson-c.so.4.0.0) Вот вывод валгринда, подозреваю что это из-за отсутствия высвобождения памяти.
    – Andrey
    14 мая 2019 в 8:24
  • Ну так вы освобождаете память через free? Или нет? 14 мая 2019 в 9:04
  • В ходе выполнения программы да, но по ее завершению через ctrl+c valgrind показывает отсутствие освобождения памяти.
    – Andrey
    14 мая 2019 в 11:15

1 ответ 1

1

Второй вариант "правильнее" тем, что не использует нестандартную функцию strdup. Разумеется, имеет смысл позаботиться о том, чтобы не выписывать явно один и тот же строковый литерал два раза.

3
  • Было бы интересно узнать, как можно позаботиться о том, чтобы не выписывать явно один и тот же строковый литерал два раза 14 мая 2019 в 9:31
  • @Andrei Khotko: Объявить именованную константу, макрос, переменную и т.п. 15 мая 2019 в 17:07
  • Ааа, вы это имели ввиду. Я немного неправильно вас понял 17 мая 2019 в 8:48

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими условиями использования и подтверждаете, что прочитали и поняли наши политику конфиденциальности и нормы поведения.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.