Discussion :

[melleb]

Bonjour,

voilà, j'ai un ptit souci avec mon appli que je suis en train de développer en C++, sous visual studio 2003.

ça compile bien,
mais en cours d'exécution, à un moment,
j'ai une erreur :

Run-Time Check Failure #2 - Stack around the variable 'mess' was corrupted

Je vois à peu près d'où, ça vient
mais je sais pas trop comment le résoudre...

Voilà comment ça se présente :

j'ai une structure :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
 
struct Message {
    int longueur;
    char *data;
    }

j'ai une variable statique :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

static vector<Message> fileAttente;

et voici ce que je fais dans une méthode :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
Message mess;
 
CreerMessage(&mess);    // appel à une méthode qui me permet d'élaborer mon message
 
// je place mon message dans la file d'attente...
fileAttente.push_back(mess);
 
// j'envoie le message...
EnvoyerMessage();

Ma méthode 'EnvoyerMessage' se charge d'envoyer les messages qui sont dans la file d'attente
elle les supprime de la file d'attente une fois qu'ils sont envoyés...

Voici donc, je pense le point du problème...

Donc comment je dois m'y prendre pour faire ça correctement??...

Merci

[loicounet]

Run-Time Check Failure #2 - Stack around the variable 'mess' was corrupted

ca arrive quand?
dans cette methode:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

fileAttente.push_back(mess);

?

[melleb]

Non,

en fait, tout ça :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
Message mess;
 
CreerMessage(&mess);    // appel à une méthode qui me permet d'élaborer mon message
 
// je place mon message dans la file d'attente...
fileAttente.push_back(mess);
 
// j'envoie le message...
EnvoyerMessage();

est dans une même méthode
et c'est en sortant de cette méthode que l'erreur tombe...

[ram-0000]

Peut être que la pile est corrompue parce que quelqu'un jardine en dehors de ses plate bandes.

Que fait le constructeur de mess, que fait la fonction CreerMessage(), que fait le constructeur de copie de Message, que fait la fonction EnvoyerMessage()

[melleb]

Il n'y a pas de constructeur pour Message

La méthode CreerMessage affecte une valeur aux champs de la structure...
Et EnvoyerMessage envoie les messages qui sont dans la file d'attente et les supprime ensuite de la liste (avec un fileAttente.erase )
je pense que c'est le fait de supprimer le message de la file d'attente qui fait péter l'erreur
nan?

[Arzar]

Bonjour,
Est-ce que tu pourrais nous montrer plus de code, comme la définition complète de la struct Message, CreerMessage() et EnvoyerMessage() ?

Personellement, je commencerais pas modifier la struct Message en la remplacant par :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
 
struct Message
{ 
   std::vector<char> data; // si c'est ce n'est pas du texte
   // ou 
   std::string data; // si c'est du texte.
};

justement, pour m'épargner ce genre d'ennui.

[melleb]

Bonjour,
Est-ce que tu pourrais nous montrer plus de code, comme la définition complète de la struct Message, CreerMessage() et EnvoyerMessage() ?

Personellement, je commencerais pas modifier la struct Message en la remplacant par :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
 
struct Message
{ 
   std::vector<char> data; // si c'est ce n'est pas du texte
   // ou 
   std::string data; // si c'est du texte.
};

justement, pour m'épargner ce genre d'ennui.

La struct message est complète

Et justement, j'ai besoin d'un 'char *'
car le message est destiné à une librairie en C qui attend du char*

[loicounet]

Et justement, j'ai besoin d'un 'char *'

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

data.c_str()

ca te donne un const char*

[Arzar]

La struct message est complète

C'est un peu ce que je craignais
Il faudrait voir CreerMessage() et EnvoyerMessage() pour débusquer l'erreur, mais elle est probablement dans une mauvaise utilisation du char* data.

Si c'est possible, std::vector<char> est réellement la solution la plus simple pour faire un buffer d'octet. La librairie C qui attends un char* peut toujours recevoir &vec[0].