Stack around the variable was corrupted

Version imprimable

Bonjour,
lors de l'execution d'une fonction d'une dll que j'ai faite, j'obtiens l'erreur suivante :

Citation:

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

et cette erreur survient à la derniere ligne (sur l'accolade fermante) de la fonction suivante
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 bool EnvirDb::TreatHeader() { SQLSMALLINT ncols; std::string sHeader; // get the column count this->srRetour = SQLColAttribute(this->hStmt,0,SQL_DESC_COUNT,NULL,0,NULL,&ncols); if (this->srRetour != SQL_SUCCESS && this->srRetour != SQL_SUCCESS_WITH_INFO) { return false; } sHeader = ""; // run through the columns for (int i=0; i<ncols; i++) { if (i != 0) { sHeader += (char)254; sHeader += (char)255; } ... // j'ai enlevé tout le code ici car il ne touche pas à ncols } SendMessage(this->Onglet, WM_USER+0, 1, (LPARAM)sHeader.c_str()); return true; }
la fonction se déroule donc comme il faut mais c'est à sa sortie que j'obtiens l'erreur.

A noter que ce meme code fonctionne correctement dans un programme C++ non dll.

05/04/2006, 13h17
tsp

bon alors j'ai résolu mon probleme en mettant ma variable ncols en static :

Code:

static SQLSMALLINT ncols;

si toutefois quelqu'un a une explication du phénomène, je suis preneur.
05/04/2006, 13h37
Aurelien.Regat-Barrel

Tu n'as pas résolu l'erreur, tu as fait en sorte qu'elle ne soit plus visible.
Ce qui se passe c'est que tu as un débordement d'écriture sur la pile. Cela peut être provoqué par une mauvaise utilisation d'une variable locale, sHeader par exemple. Passes-tu quelque part un pointeur sur sHeader ou sur ncols ? (typiquement, GetString( &sHeader )).
05/04/2006, 14h20
tsp

effectivement dans la partie de code que j'ai enlevé parce qu'elle prenait trop de place, j'ai cette ligne :

Code:

if (!(this->AddInfo(i+1, SQL_DESC_AUTO_UNIQUE_VALUE, &sHeader, 'N'))) return false;

une 20aine de fois pour différentes info.
Comment dois je proceder alors ! ! reflechissons reflechissons
05/04/2006, 14h27
Aurelien.Regat-Barrel

Que fait AddInfo ? Comment est-ce qu'elle manipule la string reçue ?

en fait addinfo concatene un string à sHeader :

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
bool EnvirDb::AddInfo(SQLUSMALLINT numcol, SQLUSMALLINT info, std::string* sHeader, char cType)
{
	CHAR sTexte[MAX_NAME_LEN];
	int iEntier;
	SQLSMALLINT  col_len;
 
	switch (cType)
	{
         // pour des info numériques
		case 'N' :
			srRetour = SQLColAttribute(this->hStmt,numcol,info,NULL,0,NULL,&iEntier);
			if (srRetour != SQL_SUCCESS && srRetour != SQL_SUCCESS_WITH_INFO) 
			{
				return false;
			}
			*sHeader += to_string(iEntier);
			break;
 
          // pour des info textes
		case 'T' :
			srRetour = SQLColAttribute(this->hStmt, numcol, info, (SQLPOINTER)(&sTexte), MAX_NAME_LEN, &col_len, NULL);
			if (srRetour != SQL_SUCCESS && srRetour != SQL_SUCCESS_WITH_INFO) 
			{
				return false;
			}
			*sHeader += to_string(sTexte);
			break;
 
		default: break;
	}
 
// ajoute un separateur apres l'info
	*sHeader += (char)254;
	*sHeader += (char)255;
	return true;
}

05/04/2006, 15h15
Aurelien.Regat-Barrel

Je ne vois rien de chaquant dans l'utilisation de sHeader. Tu pourrais utiliser une référence au lieu d'un pointeur. to_string ne modifie pas la chaine reçue ?
L'erreur que tu rencontres est typiquement produite par une fausse manip sur des buffers de ce genre:

Code:

CHAR sTexte[MAX_NAME_LEN];

genre tu te trompes dans la taille donnée (plus grande que ce qui est dispo) et la fonction écrit donc plus que MAX_NAME_LEN caractères. Et ça corromp la pile.
Refait en sorte que le compilateur te provoque l'erreur, puis commente ton code de manière sélective jusqu'à trouver l'appel de fonction qui provoque le problème, et recommence jusqu'à trouve la ligne fautive...
Vérifie bien toutes les tailles de buffers local etc...
05/04/2006, 16h00
tsp

bon alors en fait, j'ai changé mon ncols de place pour le mettre dans une autre fonction et ça me mettait toujours cette erreur à la sortie de la nouvelle fonction. J'ai donc changé mon :

Code:

SQLSMALLINT ncols;

pour un :

Code:

int ncols;

et oh surprise ça marche.
Par contre ce que je ne comprends pas trop c'est qu'avec un SQLSMALLINT ça marchait dans un programme C++ qui n'etait pas une DLL.

Merci.
05/04/2006, 16h54
Aurelien.Regat-Barrel

Citation:

NumericAttributePtr
[Output]
Pointer to an integer buffer in which to return the value in the FieldIdentifier field of the ColumnNumber row of the IRD, if the field is a numeric descriptor type, such as SQL_DESC_COLUMN_LENGTH. Otherwise, the field is unused.

il attend un entier de 32 bits = il va écrire 32 bits. Si tu lui donne une variable de 16 bits, il va écrire 16 bits de plus ailleurs, en suivant probablement. Les conséquences de ce débordement sont purement aléatoire. Donc dans ton autre programme, tu as potentiellement un bug.