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

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

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