malloc free Ошибка.

[BeNN]

Добрый день. Проблема заключается следующем: в функцию передается указатель на динамический массив, функция создает новый динамический массив(с большей размерностью) и копирует содержимое старого. Дальше по идее надо освободить память, выделенную под старый массив - вот тут и возникает ошибка. Что я не так делаю? Подскажите пожалуйста. Пишу на си, visual 6.0

int* yvel(int *a, int kolvo, int kol) 
{
	int r;
	int *w;
	w = (int *) malloc(sizeof(int) kolvo+kol);		
	for (r = 0;r < kolvo;r++)						
		w[r] = a[r];
		free(a);		//<<<<<			
	for (r = kolvo; r < kolvo+kol;r++)				 
		w[r] = 0;									 
	return &w[0];
}

[Blade]

Что за ошибка то? И приведите часть когда, где создается переменная a и где вызывается функция

[profi]

free(a);

Вообще тут ничего не нужно удалять.

Вместо этого:

return &w[0]

можно написать так:

return w;

И получаем вот это:

int* yvel(int *a, int kolvo, int kol)  
{ 
    int r; 
    int *w; 
    w = (int *) malloc(sizeof(int) kolvo+kol);         
    for (r = 0;r < kolvo;r++)                         
        w[r] = a[r];          
    for (r = kolvo; r < kolvo+kol;r++)                  
        w[r] = 0;                                      
    return w; 
}

[BeNN]

Да, а как же быть с массивом а? он больше не нужен и память нужно освобождать, не так ли?

[BeNN]

#include <stdio.h>
#include <malloc.h>


int* vid(int kolvo)		
{
	int *z;
	z = (int *) malloc(sizeof(int) kolvo);
	return &z[0];	
}

int* zap(int *a, int kolvo)		
{
	int b;
	for (b = 0;b < kolvo;b++)
		a[b] = b+1;
	return &a[0];
}

int* yvel(int *a, int kolvo, int kol) 
{
    int r;
    int *w;
    w = (int *) malloc(sizeof(int) kolvo+kol);        
    for (r = 0;r < kolvo;r++)                        
        w[r] = a[r];
        free(a);        //<<<<<            
    for (r = kolvo; r < kolvo+kol;r++)                 
        w[r] = 0;                                     
    return &w[0];
}  

main() {
int kolvo=5;	 	
int kol=10;
int *a, *w;

a = vid(kolvo);		
zap (a, kolvo);
w = yvel(a,kolvo,kol);
}

Выдает Debug error!

[profi]

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

Вообще-то это все так:

#include <vcl.h>
#include <stdio.h>

int* vid(int kolvo)
{
	int *z;
	z = new int[kolvo];
	return z;
}

int* zap(int *a, int kolvo)
{
	int b;
	for (b = 0;b < kolvo;b++)
		a[b] = b+1;
	return a;
}

int* yvel(int *a, int kolvo, int kol)
{
    int r;
    int *w;
    w = new int[kolvo+kol];
    for (r = 0;r < kolvo;r++)
        w[r] = a[r];

    for (r = kolvo; r < kolvo+kol;r++)
        w[r] = 0;
    return w;
}

int main()
{
int kolvo=5;
int kol=10;
int *a, *w;

a = vid(kolvo);
zap (a, kolvo);
w = yvel(a,kolvo,kol);

for (int i=0;i<kolvo;i++)
printf("w[%d]=%d\n",i,w[i]);

delete []a;
delete []w;

return 0;
}

Сравните этот код со своим.

[BeNN]

А возможно все же написать через malloc и free и чтобы удаление происходило в функции yvel ?

[profi]

Возможно через malloc, но разницы не вижу. Просто через new короче и проще. Нельзя очистить память под указатель *a так как вы под него в функции yvel её не выделяли.

[BeNN]

Ясно, а если через malloc - то просто заменить new на malloc? Получается нельзя в одной функции удалить память, выделенную в другой? только в main'е ?

[profi]

Ясно, а если через malloc - то просто заменить new на malloc?

Вот такая конструкция функции new (под массив):

указатель = new int[размер_массива];

олучается нельзя в одной функции удалить память, выделенную в другой? только в main'е ?

Тут много "приколов". Долго объяснять, почитайте лучше книгу Стивена Прата, статью про передачу указателей функциям.