Как очистить выделенную память адреса под адрес указателя на первый элемент строки?

Question

[Илхомжон Матазимов]

Вроде делаю все правильно, но уже 2-й день ломаю голову почему не получается освободить выделенную память:

#include <stdio.h>
#include <stdlib.h>

int	main(void)
{
	const char **text;
	const char *smth = "some";

	text = (const char**)malloc(sizeof(char*));
	if(NULL == text) return 0;

	text = &smth;
	printf("%s\n", text[0]);
	
	free(text);
	text = NULL;

	return 0;
}

Comments

[Евгений Шатунов]

Илхомжон Матазимов , давай вместе подумаем, что именно происходит в этом участке твоего кода?
text = &smth;
Сможешь мне помочь и рассказать подробно, что тут делается? Я, кажется, не совсем понимаю эту строку.

Answer 1

[Илхомжон Матазимов]

#include <stdio.h>
#include <stdlib.h>

int	main(void)
{
  const char **text;
  const char *smth;
  
  smth = "some";

  text = (const char**)malloc(sizeof(char*));
  if(NULL == text) return 0;

  *text = smth;
  printf("%s\n", text[0]);
  
  free(text);
  text = NULL;

  return 0;
}

Answer 2

[Adamos]

// выделяется память размером с указатель (4/8 байт для x32/x64), ее адрес записывается в text
text = (const char**)malloc(sizeof(char*));
// в text записывается адрес другой строки, а где там была выделена память - больше никто не знает
text = &smth;
// освобождается та память, на которую указывает text - это переменная smth. Теперь память, в которой лежит эта переменная, может в любой момент быть заменена при новом выделении памяти, и smth получит новое значение, не имеющее никакого отношения к той строке, на которую оно указывало изначально
free(text);

Comments

[Илхомжон Матазимов]

То-есть вы хотите сказать это невозможно из-за того, что эти два переменных взаимосвязаны друг с другом? Если я правильно понял ваш абстрактный ответ, то ваше рассуждение неправильное

[Adamos]

Илхомжон Матазимов, я не знаю, что вы себе нафантазировали, но в этом коде освобождается тупо не тот участок памяти, который выделяется.

[Илхомжон Матазимов]

Adamos, тогда скажите мне как присвоить адрес первого элемента переменной text, а не ее адресу

[Adamos]

Илхомжон Матазимов, так вы это и делаете в коде - text = &smth;
Осталось выкинуть нахрен весь этот салатик с malloc и free.
За ненадобностью.

[Илхомжон Матазимов]

Adamos, пренебрегать утечкой памяти лучше не стоит.

Вот решение:

#include <stdio.h>
#include <stdlib.h>

int	main(void)
{
	const char **text;
	const char *smth;
	
	smth = "some";

	text = (const char**)malloc(sizeof(char*));
	if(NULL == text) return 0;

	*text = smth;
	printf("%s\n", text[0]);
	
	free(text);
	text = NULL;

	return 0;
}

[Adamos]

Это решение - чего? Зачем в этом коде вообще ВСЕ строчки, в которых содержится переменная text?

#include <stdio.h>
#include <stdlib.h>

int	main(void)
{
  const char *smth = "some";
  printf("%s\n", smth);
  return 0;
}

[Евгений Шатунов]

Илхомжон Матазимов, мне теперь интересно, почему ты проигнорировал мое предложение?

тогда скажите мне как присвоить адрес первого элемента переменной text, а не ее адресу

Чтобы это сказать, тебе сперва следует разобраться с терминологией адреса, указателя и переменной.
const char **text; - это определение переменной text с типом модифицируемого указателя на модифицируемый указатель на константный символ.
const char *smth = "some"; - это определение переменной smth с типом модифицируемого указателя на константный символ. Переменная инициализируется адресом размещения строкового литерала "some".

text = &smth; - у переменной smth берется адрес. Это будет значение с типом const char **.
Если для тебя важно сохранить выделенную через malloc память, тебе стоит и оперировать с указателями соответствующим образом.
Значение разыменования text, т.е. *text, будет иметь тип const char *. Этот тип уже согласуется с типом smth.

*text = smth; - вот так ты запишешь адрес размещения литерала в выделенную через malloc память, которую ниже и сможешь удалить через free.

[Adamos]

Евгений Шатунов, но остается открытым вопрос, зачем вообще нужен этот цирк с конями.
В text можно записать адрес smth и работать с ним без всяких выделений динамической памяти в принципе.

[Илхомжон Матазимов]

Adamos, в одном задание нужно написать функцию, которая возвращает адрес на адрес первого элемента char, с обязательным использованием выделение памяти

[Евгений Шатунов]

Adamos, товарищ явно учится еще, развлекается в своей песочнице наверное.
Я не берусь отвечать на такие вопросы, хоть и тоже пока не вижу смысла в его действиях.

[Илхомжон Матазимов]

Евгений Шатунов, вы правы, я все ещё учусь, и буду учиться ещё долго, но на ты сразу не перехожу

[Adamos]

Илхомжон Матазимов, так поучитесь еще и не разводить на айтишных сайтах обидки из-за такой фигни, как обращение на "ты", например. Они только демонстрируют незрелость выступающего.
Не стоит заострять внимание на том, как выражается собеседник, если он говорит по делу.

[Илхомжон Матазимов]

Adamos, да это шутка, извините

Answer 3

[Евгений Залецкий]

Если память не изменяет, то нужно delete[]

Comments

[Илхомжон Матазимов]

Дело в том, что это чистый Си без плюсов

[Евгений Залецкий]

Илхомжон Матазимов, а зачем после очистки памяти вы присваиваете значение NULL? Разве не просто free(переменная)?

[Сергей Горностаев]

Илхомжон Матазимов, тогда зачем стоит тег "c++"?

[Сергей Горностаев]

Евгений Залецкий, это best practice. Лучше иметь нулевой указатель, чем указатель на неаллоцированную память.

[Евгений Залецкий]

Сергей Горностаев, я по этому дал ответ под С++. Я слаб в С, но буду теперь знать, спасибо.

[Евгений Шатунов]

Евгений Залецкий, в C++ это тоже best practice при работе с сырой памятью. Никто за тебя битый указатель занулять не станет.
Но, на будущее, в C++ для этого кода стоит использовать std::unique_ptr<const char* []> и вообще не иметь дело с new и delete.