Plantage sur deletion de std::vector

Version imprimable

20/02/2012, 13h05
uriotcea

Plantage sur deletion de std::vector

Bonjour,

Ca fait plsuieurs heures que je galére là dessus sans succés. Alors quelquechose m'échape surement, je vais essayé de vous expliquer, mais le code etre trop complexe pour etre simplifé.

J'ai une class "Data" contenant des double, float.. et une variable vector "V"
Je crée un tableau dynamique (100 valeurs) avec new de la class "Data". J'initiale le contenu des double et je mets aussi quelques valeur dans V. Je ne rempli par nécessairement toutes "V" des 100 "Data"
A la fin je detruit mon tableau et j'ai un plantage au destructeur de la class "Data" au titre que je fait (vue avec valgrind) un delete interdit sur la class vector.

Je sais pas facile de comprendre avec du blabla, mais si quelquechose vous saute au yeux, je suis preneur

Merci
20/02/2012, 13h28
koala01

Salut,

Pourrais tu nous présenter un code minimal compilable (structure + fonctions) qui reproduise le problème, que nous puissions nous faire une idée :question:

Merci d'avance ;)

voila a quoi ca ressemble

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
 
class Data
{
  double a,b;
  float c;
  std::vector V;
  public:
  Data() {
    a=ab=0;
    c=0;
    V.clear();
  }
};
 
void set_data(std::vector &v)
{
   for (int i=0;i<=9;i++) {
     ab=i;
   }
   v.append(10)
   v.append(10)
   v.append(10)
}
 
void main()
{
   Data *d=new Data[100];
   set_data(d);
 
   delete[] d;
  // plantage au moment du delete
   set_data(d);
 
}

20/02/2012, 19h06
Arzar

"un code minimal compilable qui reproduise le problème"

Le code ci-dessus est très loin de compiler. Il ne fait pas assez sens pour qu'on puisse deviner l'origine réelle du problème.

Désolé, je reconnais que c'était vraiment n'importe quoi.

voici le code épuré qui compile cette fois et qui, à mon grand désespoir, snif...ne plante pas comme dans mon code originale, sur les delete.

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
36
37
38
39
40
41
42
43
 
class Data
{
  public:
  double a,b;
  float c;
  std::vector<double> V;
  Data() {
    a=b=0;
    c=0;
    V.clear();
  }
};
 
void set_data(Data *d);
 
void set_data(Data *d)
{
   for (int i=0;i<=9;i++) {
     d->a=d->b=i;
   }
   d->V.push_back(10);
   d->V.push_back(12);
   d->V.push_back(14);
   return;
}
 
 
int main(int argc, char *argv[])
{
   Data *d=new Data[100];
   set_data(d);
 
   delete[] d;
   set_data(d);
 
   delete[] d;
   set_data(d);
 
   delete[] d;
   set_data(d);
  return(0);
}

20/02/2012, 20h09
uriotcea
Par contre je me rends compte que j'ai oublié d'y inclure un opérateur vraiment suspect. Il ne fait pas planter mais il y quelque chose de malsain dedans, mais je ne sais pas comment le réécrire proprement.
Code:

1 2 3 4 5 void Data::operator=(const Data &d0) { Data *d=this; memcpy(d0,&d,sizeof(Data)); }
20/02/2012, 20h15
JolyLoic

C'est effectivement très malsain.

C'est un comportement indéfini. Qui en pratique va conduire à des doubles suppressions de mémoire. A virer au plus vite.
20/02/2012, 20h30
Arzar

Edit :
Oula en effet avant toute chose il faut déjà supprimer au plus vite cet opérateur d'affectation complètement foireux :aie:. Surtout que si la classe Data ne contient effectivement que quelques double, float et std::vector l'opérateur d'affectation généré par le compilateur est suffisant.
[Fin Edit]

Ok merci, c'est plus clair.

Le dernier code ne plante pas mais c'est un coup de chance. :aie: L'utilisation d'un tableau après qu'il est était détruit (après un appel à delete[]) est un comportement indéterminé. Cela peut crasher immédiatement comme cela peut donne l'impression de marcher (jusqu'au jour ou quelqu’un change une option de compilation ou modifie ou peu le code et boum ça plante à nouveau).

D'ailleurs, je ne sais pas quel est ton compilateur, mais chez moi en debug, le débuggeur s'arrête à la ligne set_data(d) juste après le premier delete[] avec le message suivant :

Citation:

Windows has triggered a breakpoint in TestCpp.exe.

This may be due to a corruption of the heap, which indicates a bug in TestCpp.exe or any of the DLLs it has loaded.

Le reste du code ne présente pas de problème.

Si c'est possible remplacer dans l'appli originale Data *d=new Data[100]; par std::vector<Data> d(100); et faire les modif qui vont avec devrait très probablement corriger le problème.
20/02/2012, 21h09
uriotcea

Ok, merci pour ces corrections qui s'imposaient visiblement. Je vais mettre tout ca en oeuvre.