[C] Debug Error!

[spiderontheweb]

Ciao a tutti.

Ho compilato un programma con il Visual C++ senza avere errori.
Questo programma legge da file, inizializza una matrice di caratteri e fa alcune operazioni in modo corretto.
Quando provo ad eseguirlo funziona correttamente quasi fino alla fine quando voglio liberare la memoria allocata precedentemente:

for (i=0; i< nr;i++)
   free(m[i]);
ree(m);

Mi appare la finestra di Windows con pallino rosso e crce bianca:
____________________________________________
|MIcrosoft Visual C++ Debug Library
|___________________________________________
| Debug Error!
|Program:C:\prova\Debug\prova.exe
|DAMAGE: after Normal block (#41) at 0x00441F90
|___________________________________________

A cosa pùò essere dovuto?

[oregon]

E' un problema legato alla allocazione/liberazione della memoria.

Controlla, passo passo, quando si verifica (se nel ciclo o per l'ultima free).

[spiderontheweb]

Si verifica sulla prima free

free(m[i]);

Se gli dico di ignorare mi ripete lo stesso errore su tutte le free tranne su free(m) (fuori dal for)

Ho provato a mettere prima del ciclo for

free(m[0]);

e si pianta proprio su questa.

[andbin]

Originariamente inviato da spiderontheweb
A cosa pùò essere dovuto?

Come hai dichiarato quel 'm'?? Come fai la allocazione della matrice?? Senza saperlo, è difficile capire dove sta il problema.

[spiderontheweb]

Premesso che il programma funziona correttamente fino alla free:

//prototipo
char **MallocChar(int nr, int nc);

main()
{
   char **m; //dichiara m
   ...
   m=MallocChar(nr, nc); //chiamata alla funzione
   ...
   
   for (i=0; i< nr;i++) 
      free(m[i]); //qua si pianta!!!!!!!
   free(m);
   
   return 0;
}

//funzione per allocare la memoria
char **MallocChar(int nr, int nc)
{
   int i;
   char **m;
   m = (char **)malloc(nr * sizeof(char *));
   for (i=0; i< nr;i++)
      m[i] = (char *)malloc(nc * sizeof(char));
   return m;
}

Mi sembra ok...

[oregon]

Probabilmente e' qualcosa che fai tra la malloc e la free ... qualcosa che sta nei "puntini" che hai indicato ...

[spiderontheweb]

Questo è il codice. Non l'ho scritto io.

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

#define N_MOSSE 8

int r[N_MOSSE][2] = {
	{ 1, 1},
	{ 1, 0},
	{ 1,-1},
	{ 0,-1},
	{-1,-1},
	{-1, 0},
	{-1, 1},
	{ 0, 1}
};

int nr, nc, superficie;
float altezza_media;
int totale;

void cerca_elemento_matrice(int x, int y, char **m);
char **MallocChar(int nr, int nc);


main() 
{
	char **m;
	FILE *f;
	int  i=0, c;
	int conta_isole=0;
	char riga[30];
	
	f = fopen("dati.txt", "r");
	if (f== NULL)
	{
		printf("Errore apertura file\n");
		exit(1);
	}
	else 
		fscanf(f, "%d %d ", &nr, &nc);
	m=MallocChar(nr, nc);
	while(fscanf(f, "%s",riga )!=EOF)
	{
		//m[r] = malloc (nc * sizeof (char));
		strcpy( m[i],riga); 
		printf("%s\n", m[i]);
		i++;
	}
	for(i=0;i<nr;i++)
		for(c=0;c<nc;c++)
		{
			if(m[i][c]!='#' && m[i][c]!='0')
			{
				totale=0;
				altezza_media=0.0;
				superficie=0;
				conta_isole++;
				cerca_elemento_matrice( i ,c , m );
				altezza_media=(totale/(float)superficie);
				printf("%d %f\n", superficie, altezza_media);
			}	
		}   
        
        getchar();
        free(m[0]);     
		for (i=0; i< nr;i++)
		{
			free(m[i]);
        }
		free(m);
		return 0;		
}

void cerca_elemento_matrice(int x, int y, char **m)
{
	if(x >= 0 && x < nr && y >= 0 && y < nc && m[x][y]!='#' && m[x][y]!='0')
	{
		int i;
		totale+=(m[x][y]-'0');
		superficie++;
		m[x][y]='#';
		for( i = 0; i < N_MOSSE; i++ )
			cerca_elemento_matrice( x + r[i][0], y + r[i][1], m);
	}
	return;
}

char **MallocChar(int nr, int nc)
{
	int i;
	char **m;
	m = (char **)malloc(nr * sizeof(char *));
	for (i=0; i< nr;i++)
		m[i] = (char *)malloc(nc * sizeof(char));
	return m;
}

[andbin]

Originariamente inviato da spiderontheweb
Mi sembra ok...

Sì, confermo che allocazione e de-allocazione sono OK.

Posso solo supporre che da qualche parte nel programma si va a "sovrascrivere" delle aree di memoria dove non si dovrebbe .... danneggiando ad es. i puntatori e/o l'heap.