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 ?)

[mister3957]

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

Config.GetCount() n'est plus appelé.

En fait, j'ai réduit le scope du problème, et :

- La DLL réceptionne un pointeur void*, du style "UserData", l'enregistre et renvoie un identifiant.
- L'exécutable envoie ses données à la DLL, et gère les identifiants.

Pour gérer les identifiants, il me fallait un conteneur à la fois capable de retrouver la donnée via l'identifiant, que l'identifiant via la donnée. Ainsi, j'ai utilisé le multi_index_container.

Je me suis rendu compte que selon la volumétrie passée à ce conteneur, ça crachait à différents endroits (dans une instruction côté EXE comme DLL).

Au lieu d'utiliser ça, j'ai changer mon implémentation en utilisant deux map, et là ça fonctionne très bien. Donc je pense que c'est soit le multi_index_container qui déconne, soit l'utilisation que j'en fais.

Dans ce multi_index_container, j'utilise trois index (sequentiel, par identifiant, par données). Avec une version de boost, ça crachait sauf si je ne laissais que l'index séquentiel (toujours des problèmes de mémoire), et avec une autre version, c'est le problème ci dessus.

Alors je suis en train de recompiler une version de boost, la dernière, et si ça se reproduit, je ferai un autre sujet, spécifique à ce conteneur.

En tout cas merci pour votre écoute

A bientôt