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

[polin11]

Подскажите, программа состоит из 2 функций (main и fun), программа меняет значение указателя на символьный массив созданный в динамически распределяемой памяти.
Программ меняет символьный массив "abc" на "cba" ошибок нет. Вопрос в другом, нужно ли удалять указатель из кучи в функции main или fun, не произойдет ли утечка памяти,
не становиться ли указатель str_f из функции fun "диким" при завершении функции fun??? При попытке удалить указатель delete str_f или delete[] str_f, при выполнении программы выходит ошибка.

#include "stdafx.h"
#include <iostream>
#include <stdlib.h>
#include <conio.h>
#include <stdio.h>
#include <windows.h>

using namespace std;
char* fun()
{
   char* str_f=new char[5];
   ZeroMemory(str_f, 5);
   str_f="cba";
   /* delete str_f;  нужно ли удалять указатель массив char из динамически распределяемой памяти. При выполнении выходит ошибка */
   return str_f; 
}
int _tmain(int argc, _TCHAR* argv[])
{
	
	int n=5;
	char* str=new char[5];
	ZeroMemory(str, 5);
	str="abc";
	cout<<str <<endl;
	str=fun();
	cout<<str <<endl;
	/* delete[] str; нужно ли удалять указатель массив char из динамически распределяемой памяти. При выполнении выходит ошибка */
	cout<<str;
	getch();
	return 0;
	
}

[Alex11223]

не произойдет ли утечка памяти

произойдет конечно.

А ошибка потому что тут

str_f="cba";

создается новый строковой литерал и указатель на него присваивается в str_f. То есть в str_f после этого уже нет указателя на то, что new выделено было.

Тут strcpy надо использовать скорее всего. А еще лучше std::string раз С++.

Ну и если у вас функция возвращает указатель, то очевидно нельзя освобождать эту память в этой функции.

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

То есть примерно так:

void fun(char* str_f)
{
   strcpy(str_f, "cba");
}

int main()
{
    char* str_f = new char[5];

    fun(str_f);

    cout << str_f;

    delete str_f;
    str_f = nullptr;