Discussion :

[mister3957]

Bonjour,

J'ai une DLL qui a besoin d'allouer un tableau d'entier, dont le nombre d'élément est variable selon les cas.

Lorsque ça tente d'allouer un tableau de 0 ou 1 entier, ça crash :

Code :

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

1
2
3
4
5
6
7
Windows has triggered a breakpoint in myApp.exe.
 
This may be due to a corruption of the heap, which indicates a bug in myApp.exe or any of the DLLs it has loaded.
 
This may also be due to the user pressing F12 while myApp.exe has focus.
 
The output window may have more diagnostic information.

En revanche, pour 2 ou plus, ça ne crash pas..

Qu'est-ce que je peux faire pour gérer ça ?

Merci
A bientôt

[Bousk]

Bonjour,

quel est le code en question de l'allocation ?

[mister3957]

De ce genre là :

Code :

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

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
 
struct DataSet
{
	DataSet();
	~DataSet();
 
	void Reserve(int iCount);
	int AddData(void* pUserData);
 
	SUserData* m_pData;
	int m_iCount;
	int m_iCapacity;
	int m_iBuffer;
	void* m_pCaps[10];
};
 
struct Context
{
    DataSet mSets[7];
};
 
struct ADataType
{
   ADataType();
   ~ADataType();
   int* m_Tab;
};
 
// Dans un .cpp :
ADataType::ADataType()
{
   int iCount = Config.GetCount();
   m_Tab = new int[iCount ];
}
 
ADataType::~ADataType()
{
   delete[] m_Tab;
}

Dans un .h, je définis la valeur par défaut d'un "ADataType" ainsi :
const ADataType ADataTypeDefault = ADataType();

Lorsque ma DLL se charge, le "Config.GetCount()" renvoie 0, et ça fonctionne.

Plus tard, lorsque j'initialise mes données :

Code :

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

1
2
3
 
Context myContext;
myContext.mSets[0].m_pCaps[1] = new ADataType[1];

Et c'est là que ça crash, le new ADataType[1] appelle le constructeur de ADataType, et lorsque la valeur de Config.GetCount() est de 0 ou 1, supérieure à 1, ça ne crash pas.

Y a-t-il quelque chose de bizarre dans ce code ?

Merci
A bientôt

[Bousk]

A priori oui, il y a des trucs qui me paraissent bizarre

Code :

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

void* m_pCaps[10];

Pourquoi un void* ?

Code :

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

ADataType** m_pCaps[10];

Faire un new avec une taille de 0, ça crashera toujours, il faut gérer cette valeur particulière.
Une valeur de 1 ne devrait pas entraîner de crash. Es-tu sûr qu'il s'agit de 1 et non 0 ?

[mister3957]

La valeur de zéro ne provoque pas de crash dans le cas où ça initialise la DLL :

Code :

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

1
2
 
const ADataType ADataTypeDefault = ADataType();

Et oui je suis sûr que la valeur 1 crash.

Par ailleurs, j'ai juste mis ceci :

Code :

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

ADataType data

Avec en dure "1" dans le constructeur :

Code :

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

m_Tab = new int[1];

Et ça crash quand même

[dragonjoker59]

Par ailleurs, j'ai juste mis ceci :

Code :

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

ADataType data

Avec en dure "1" dans le constructeur :

Code :

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

m_Tab = new int[1];

Et ça crash quand même

Du coup la fonction Config.GetCount() fait quoi ? (est-elle toujours appelée ?)