delete [] mavariable impossible ?!...... le retour!

Version imprimable

Bonjour,

Je suis en train de me casser les dents sur un probleme qui je pense doit être tout bête...
En gros je crée une chaine (char*),
Je la remplie
et je souhaite liberer la memoire, et ca me sort une erreur:

Debug Error!
Program: ......monprogramme.exe
DAMAGE: after Normal block (#3167) at 0x00AB71C0

Voila le code:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 char* tempTimeSt; if ( TimeSt != NULL ) { unsigned long length = WideCharToMultiByte(CP_ACP,0,TimeSt,SysStringLen(TimeSt),NULL,0,NULL,NULL); tempTimeSt = new char[length]; length = WideCharToMultiByte (CP_ACP,0,TimeSt,SysStringLen(TimeSt),tempTimeSt,length,NULL,NULL); tempTimeSt[length] = '\0'; } delete [] tempTimeSt;
L'erreur se déclenche lors de l'éxecution du delete[]
Est ce que quelqu'un saurait pourquoi??

06/11/2006, 12h26
Médinoc

Tu as du avoir un dépassement de buffer ("damage after normal block") à un certain moment entre l'allocation et le delete...
06/11/2006, 12h47
Micromalice

C'est trouvé....

Merci pour cette réponse si rapide..

En fait je pense que l'erreur vient du fait que je rajoute le caractère '\0' a la position machaine[longueur de machaine] or le dernier élement est machaine[longueur de machaine-1]..

En ajoutant un élement a ma chaine, ca marche....

merci encore.

delete [] mavariable impossible .... le retour!

Re-bonjour à tous,

J'ai a nouveau un probleme de base sur le delete de pointeurs...
et je réutilise ce post pour ne pas faire double emploi.

voila le code:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 char** MonTableau = NULL; MonTableau= new char*[20]; for (i=0; i<9; i++) { MonTableau[i] = new char[255]; [...traitement...] } for (i=0; i<9; i++) delete [] MonTableau[i];
Le 1er élement est bien supprimé mais lorsqu'on arrive au 2e, je n'ai aucune erreur par contre:
1/ lorsque je regarde au debuggeur: MonTableau[1]= "n'importe quoi..."
2/ le programme sort de la fonction...

Quelqu'un saurait-il pourquoi?

07/11/2006, 10h37
Médinoc

Sans le traitement, on ne peut pas t'aider...

Au fait: pourquoi utiliser des char* et non des std::string ?

Toujours aussi rapide (merci...;) )
Donc avec le traitement ca donne:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
 
{
   char** MonTableau = NULL;
   char* tempTimeSt;
 
[...On remplie tempTimeSt (voire 1ere fonction du Post)...]
 
   MonTableau= new char*[20];
 
   for (i=0; i<20; i++)
   {
      //On rempli les tableaux de char*
      stdanstab(TabTimeSt,tempTimeSt);
   }
 
   for (i=0; i<20; i++)
     delete [] MonTableau[i];
}
 
 
//remplie un tableau de chaine depuis une chaine de caracteres avec separateur
char CDatabaseMgr::stdanstab(char **tab, char* buffer)
{
	int i=0;
	if (tab[i]==NULL) 
		tab[i] = new char[255];
	tab[i]=strtok(buffer,";");
	while(tab[i])
	{
		i++;
		if (i== ProfondeurDef) break;
		tab[i]=strtok(0,";");
	}
	return i;
}

J'utilise des char* car la fonction strtok travaille avec des char*, je trouvais ca plus simple...
Par contre désolé mais je n'arrive pas à faire une indentation correct, bien qu'en édition de message ce soit OK :?

07/11/2006, 10h59
Médinoc

Utilise CODE et non QUOTE...
07/11/2006, 11h00
Médinoc
Code:

1 2 tab[i] = new char[255]; tab[i]=strtok(buffer,";");
Et une fuite de mémoire pour la table 42! Une!
07/11/2006, 11h14
Micromalice

Autant pour moi....

c'est une faute de frappe, qui n'existe plus...:?

Par contre, je ne pense pas que cela vienne du traitement car lorsque je regarde au débuggeur, lors du delete [] de MonTableau[0], le tableau est bien remplie comme je le souhaite (pour MonTableau[1], ..[2] etc...)
07/11/2006, 11h23
Médinoc

La fuite mémoire est toujours là, par contre.
Et l'explication de l'erreur du delete également: Tu fais un delete[] sur un résutlat de strtok() et non un résultat de new[]...
07/11/2006, 11h46
Micromalice
Pour etre sûr d'avoir tout compris...

Le new char[255] est inutil car je récupère un pointeur depuis strtok (la fuite mémoire est là???)
donc je n'ai pas besoin de le supprimer...

Par contre il faut que je supprime MonTableau?

et donc faire un
Code:

1 2 delete [] MonTableau;
07/11/2006, 11h49
Médinoc

C'est ça.
Et il faudra sans doute aussi un delete[] buffer au même moment, si nécessaire.
Attention, buffer[] ne doit être ni détruit ni modifié, puisque les cases de tableau[] pointent dessus...
07/11/2006, 14h07
Micromalice

merci pour tout...

c'est tout bon.:D