std::bad_alloc :'(

Version imprimable

21/03/2006, 00h08
mister3957

std::bad_alloc :'(
Bonjour à tous,

J'instiancie une structure. Ca passe une fois, et à la deuxième, ça plante...
Code:

1 2 Unhandled exception at 0x7c81eb33 in Battle Kart.exe: Microsoft C++ exception: std::bad_alloc at memory location 0x0012f2f4..
Est-ce que ça pourrait provenir de la stdport 5.0? J'ai pas testé sans...

Merci,

A bientôt
21/03/2006, 00h47
Matthieu Brucher

Si tu pouvais expliciter un peu...
21/03/2006, 02h32
Invité

Re: std::bad_alloc :'(

Citation:

Envoyé par mister3957

Est-ce que ça pourrait provenir de la stdport 5.0? J'ai pas testé sans...

stlport?

Alors en gros j'ai ça:
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 44 45 struct MaStructure { float a1, b1, c1, d1; float a2, b2, c2, d2; float a3, b3, c3, d3; float a4, b4, c4, d4; float a5; }; class class1 { public: class1(); ~class1(); private: MaStructure* m_maStructure; }; class1::class1() { m_maStructure = new MaStructure(); }; class1::~class1() { delete m_maStructure; }; class cPrincipale { public: //constructeur // destructeur private: std::vector<class1*> m_vector; void uneFonction(); }; cPrincipale::uneFonction() { class1* c = new class1(); m_vector.push_back(c); }
Voilà, en résumé, j'ai un truc similaire à ça. Donc au premier appel de "uneFontion" ça passe bien, au seconde ça me fait une erreur:
Code:

1 2 Unhandled exception at 0x7c81eb33 in Battle Kart.exe: Microsoft C++ exception: std::bad_alloc at memory location 0x0012f4bc..
en s'arrêtant là:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 void *__CRTDECL operator new(size_t size) _THROW1(_STD bad_alloc) { // try to allocate size bytes void *p; while ((p = malloc(size)) == 0) if (_callnewh(size) == 0) { // report no memory static const std::bad_alloc nomem; _RAISE(nomem); } return (p); }
J'ai remarqué que la première fois que class1 est instanciée, ça ne rentre pas dans le while. Alors qu'à la suivante, ça y rentre et ça plante.

J'ai d'abord pensé que c'était la stlport (librairie standart) mais nan c'est pas ça.

j'ai essayé également:
Code:

1 2 3 class1* c = NULL; c = new class1();
et c'est toujours pareil, par contre
Code:

1 2 3 4 class1* c = NULL; ZeroMemory(&c, sizeof(class1)); c = new class1();
avec ça ça passe, mais ça me refait la même erreur plus tard dans le code. Donc j'en rajoute encore un et ça le refait à une autre instanciation etc... Je vais quand même pas faire des ZeroMemory() à chaque fois que je construit un objet, si?

Et pourquoi il n'arrive pas à faire de malloc(size) la deuxième fois?

Merci :)

A bientôt

21/03/2006, 10h11
Matthieu Brucher

Punaise, c'est sale...
Donne nous plus d'infos sur la pile lors de l'arrêt du programme, on se doute que c'est dans un new, mais à quel niveau ?

Bon, ensuite, pour corriger : on fait un new class1 et non un new class1() car class1() peut être considéré comme une déclaration de fonction, et non de variable.
Ensuite, ne fais JAMAIS ce que tu as écrit comme dernier exemple. Ca sent la personne qui essaie sans savoir ce qu'elle fait. Regarde bien ce que tu as écrit. Tu prends un pointeur nul, et tu écris des zéros à cet endroit. Mais attends, c'est où cet endroit ?? C'est à l'adresse 0x00000000, là où tu as les vecteurs d'interruption, ... tout ce qui permet à ton système de fonctionner. En gros, tu le fous en l'air. En plus, tu fais un new dessus par la suite, les 2 premières lignes sont donc clairement inutiles.
21/03/2006, 10h22
mister3957

Heu ouais exact, c'est complètement débile mais bon, ne sachant que faire, j'ai mal recopié un exemple sur le net... M'enfin, ça résoud rien quand même.

Ce soir je vais essayer de recopier des infos sur la pile (là je suis au travail j'ai pas mon projet ici).

Je vais également essayer de le compiler sous dev-c++, juste pour voire.

Merci :)

A bientôt
21/03/2006, 10h50
Matthieu Brucher

Tu as enlevé les parathèses que je t'ai indiqué ? Ca peut jouer - peut-être -
21/03/2006, 11h06
mister3957

Bah je suis pas chez moi, je suis au travail, mais j'essaye ça dès que je rentre, j'espère ce midi.

Je te tiens au courant tout à l'heure, et je te remercie beaucoup :)

A bientôt
21/03/2006, 12h32
Zen68

Il y a une erreur dans ta declaration
cPrincipale::uneFonction()
{
....
}
il manque void cPrincipale::uneFonction(){ ...}
21/03/2006, 12h37
Matthieu Brucher

Effectivement... Tu as dû avoir une liste de warning, pense à les éliminer d'abord.
21/03/2006, 13h08
Médinoc

Citation:

Envoyé par Miles

Ensuite, ne fais JAMAIS ce que tu as écrit comme dernier exemple. Ca sent la personne qui essaie sans savoir ce qu'elle fait. Regarde bien ce que tu as écrit. Tu prends un pointeur nul, et tu écris des zéros à cet endroit. Mais attends, c'est où cet endroit ?? C'est à l'adresse 0x00000000, là où tu as les vecteurs d'interruption, ... tout ce qui permet à ton système de fonctionner. En gros, tu le fous en l'air. En plus, tu fais un new dessus par la suite, les 2 premières lignes sont donc clairement inutiles.

Pas exactement, en fait, il transforme son pointeur NULL en pointeur égal à 0x00000000 au niveau binaire (ce qui selon l'architecture, peut être différent) et évidemment, il déborde derrière.
Mais c'est à ne pas faire aussi.
21/03/2006, 13h53
Matthieu Brucher

Oui, effectivement, il y a des cas particuliers, mais ça marche comme ça au moins sur l'archi x86, non ? - çà moins que je confonde avec 68k;.. -
21/03/2006, 13h59
swirtel

Il serait intéressant d'apprendre les bases du C++ et de la programmation et surtout la gestion mémoire.

Recopier un code sans le comprendre NE SERT à RIEN.

concernant le std::bad_alloc, tu peux regarder le contenu de la méthode what(), qui te dira quel est le problème.
21/03/2006, 14h22
mister3957

ben d'habitude j'essaye de comprendre, et j'y arrive, mais là comme ça me soulait ce problème, j'ai été cherché des bouts de codes à droite à gauche, j'ai recopié pour voir pi voilà. Ca reste rare, et une fois qu'en faisant comme ça ça fonction, alors je me penche dessus pour comprendre au final ce que j'ai fait, et pour pouvoir nettoyer et optimiser par la suite.

Mais je l'accorde, c'est pas bien.

J'ai pas pu rentrer chez moi ce midi, mais ce soir je vais faire cette méthode what() (jamais entendu parler) et j'vais recopier un peu la pile plus en profondeur en espérant que ça vous sera utile pour m'aider :)

A bientôt
21/03/2006, 14h47
swirtel

si Visual C++ utilise les specs de la STL , tu as une méthode what dans toutes exceptions du namespace std.

std::bad_alloc contient la méthode const char * what() throw();
21/03/2006, 17h01
mister3957

Ok, merci beaucoup, j'regarde ça ce soir et je vous dit quoi :)

A ce soir

alors,

ça me renvoi une bad_alloc. Sa description est... "bad allocation". J'ai cherché des infos sur google, la cause principale du déclenchement de cette exception est un lorsque l'on essaye d'alouer trop de mémoire et qu'on en a pas assez... Dans mon cas, je fais rien d'extra ordinaire, et ce que j'essaye d'instancier c'est la structure D3DMATERIAL9:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 
typedef struct _D3DMATERIAL9 {
    D3DCOLORVALUE   Diffuse;        /* Diffuse color RGBA */
    D3DCOLORVALUE   Ambient;        /* Ambient color RGB */
    D3DCOLORVALUE   Specular;       /* Specular 'shininess' */
    D3DCOLORVALUE   Emissive;       /* Emissive color RGB */
    float           Power;          /* Sharpness if specular highlight */
} D3DMATERIAL9;
 
typedef struct _D3DCOLORVALUE {
    float r;
    float g;
    float b;
    float a;
} D3DCOLORVALUE;

Sinon je vais recopier un peu la pile des appels jusqu'à l'erreur:

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
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
 
m_dxMaterial = new D3DMATERIAL9(); <--
 
---------------------------------------------
 
void *__CRTDECL operator new(size_t size) _THROW1(_STD bad_alloc)
        {       // try to allocate size bytes
        void *p;
        while ((p = malloc(size)) == 0) <--
                if (_callnewh(size) == 0)
                {       // report no memory
                static const std::bad_alloc nomem;
                _RAISE(nomem);
                }
 
        return (p);
        }
 
----------------------------------------------
 
extern "C" _CRTIMP void * __cdecl malloc (
        size_t nSize
        )
{
        void *res = _nh_malloc_dbg(nSize, _newmode, _NORMAL_BLOCK, NULL, 0); <--
 
        RTCCALLBACK(_RTC_Allocate_hook, (res, nSize, 0));
 
        return res;
}
 
--------------------------------------------------
 
extern "C" void * __cdecl _nh_malloc_dbg (
        size_t nSize,
        int nhFlag,
        int nBlockUse,
        const char * szFileName,
        int nLine
        )
{
        void * pvBlk;
 
        for (;;)
        {
            /* do the allocation
             */
            pvBlk = _heap_alloc_dbg(nSize, nBlockUse, szFileName, nLine); <--
 
            if (pvBlk)
            {
                return pvBlk;
            }
            if (nhFlag == 0)
            {
                errno = ENOMEM;
                return pvBlk;
            }
 
            /* call installed new handler */
            if (!_callnewh(nSize))
            {
                errno = ENOMEM;
                return NULL;
            }
 
            /* new handler was successful -- try to allocate again */
        }
}
 
-------------------------------------------------------------------------
 
// après on passe dans une grande fonction ou il fait plusieurs tests.
// Puis il récupère la taille de l'objet:
blockSize = _ROUND2(blockSize, _GRANULARITY);
// et il l'envoi à
pHead = (_CrtMemBlockHeader *)_heap_alloc_base(blockSize);
 
--------------------------------------------------------------------------
 
__forceinline void * __cdecl _heap_alloc (size_t size)
 
{
#ifndef _WIN64
    void *pvReturn;
#endif  /* _WIN64 */
 
    if (_crtheap == 0) {
        _FF_MSGBANNER();    /* write run-time error banner */
        _NMSG_WRITE(_RT_CRT_NOTINIT);  /* write message */
        __crtExitProcess(255);  /* normally _exit(255) */
    }
 
#ifdef _WIN64
    return HeapAlloc(_crtheap, 0, size ? size : 1);
#else  /* _WIN64 */
    if (__active_heap == __SYSTEM_HEAP) {
        return HeapAlloc(_crtheap, 0, size ? size : 1); <--
    } else
    if ( __active_heap == __V6_HEAP ) {
        if (pvReturn = V6_HeapAlloc(size)) {
            return pvReturn;
        }
    }
#ifdef CRTDLL
    else if ( __active_heap == __V5_HEAP )
    {
        if (pvReturn = V5_HeapAlloc(size)) {
            return pvReturn;
        }
    }
#endif  /* CRTDLL */
 
    if (size == 0)
        size = 1;
 
    size = (size + BYTES_PER_PARA - 1) & ~(BYTES_PER_PARA - 1);
 
    return HeapAlloc(_crtheap, 0, size);
 
#endif  /* _WIN64 */
}
 
et HeapAlloc(_crtheap, 0, size ? size : 1); a renvoyé NULL. à partir de là ça remonte jusqu'à ma ligne "new" avec l'exception...

Je pense que je vais désormais éviter les pointeurs :s

Miles: j'ai essayé new D3DMATERIAL9 (sans les parenthèses) et ça ne change rien ;)

A bientôt

21/03/2006, 23h31
Matthieu Brucher

Efface une fois tous les fichiers .o et recommence la compilation, parce que c'est louche cette histoire...
21/03/2006, 23h47
mister3957

j'ai effacé tout les .obj (en espérant que c'est les .o dont tu parlais) et j'ai fait un rebuild all.

Rien y fait, c'est pareil

Entre temps, j'ai remplacé le D3DMATERIAL9* par un D3DMATERIAL9, et là il me balance la même exception sur un tableau: VERTEX* tabVertex = new VERTEX[nbVertex]; VERTEX étant une structure de 5 float.

et si on essaye de le recompiler sur une autre machine, ça peut être un test intéressant? Si oui, ben j'peux transmettre mes sources...

J'ai essayé DEV-C++... ça me semble assez galère d'y faire le portage, j'ai pas cherché longtemps
22/03/2006, 00h11
Matthieu Brucher

Tu as l'air d'avoir un autre souci, je ne sais pas. Tu utilises des bibliothèques avec ton code ?
22/03/2006, 00h19
mister3957

oui, j'utilise:

// l'ensemble collada 1.4
libcollada_integrationFull.lib
libcollada_STLDatabase.lib
libcollada_stdErrPlugin.lib
libcollada_LIBXMLPlugin.lib
libcollada_dom.lib
libcollada_dae.lib

// winsock (obligé pour collada)
ws2_32.lib

// zlib-1.2.3.win32 (obligé pour collada)
zlib.lib

// libiconv-1.9.1 (obligé pour collada)
iconv.lib

// libxml2-2.6.23 (obligé pour collada)
libxml2_a.lib

// directx (février 2006)
dxerr9.lib
dxguid.lib
d3dx9d.lib
d3d9.lib

// défaut
winmm.lib
comctl32.lib

libxml2, iconv et les libs collada, je les ai compilé moi même en /MDd et mon projet est également en /MDd
22/03/2006, 10h58
Matthieu Brucher

Et le reste est aussi compilé en MDd ?
22/03/2006, 14h07
mister3957

ben le reste je sais pas, j'ai pas réussit à recompiler la zlib, et je sais pas comment faire pour recompiler le truc winsock, winmm et comctl32 de windows ni directx (je sais pas si c'est faisable)

Je rentre chez moi ce midi, je vais regarder si ils proposent plusieurs librairie compilée chacune differemment, et mieux, si c'est possible de les recompiler soit même.

Mais pour zlib, j'ai jamais réussit à la recompiler...
22/03/2006, 14h52
Matthieu Brucher

Bizarre, j'ai toujours réussi à la compiler, celle-èlà ;)
Pour les lib de Microsoft, tu ne peux pas les recompiler, c'est normal. En revanche, il y a peut-être un conflit avec zlib, mais je ne sais vraiment pas d'où peut venir ton pb.
Essaies d'élaguer au maximum ton projet jusqu'à obtenir un mini programme avec le bug.
22/03/2006, 14h55
mister3957

Ouais je vais essayer ça, et quand ça sera assez court, je posterai tout :)

J'fais ça ce soir.

A bientôt
22/03/2006, 22h00
mister3957

Ok c'est bon,

Alors verdict: j'suis une couille. J'ai fait un débordement de tableau.

J'avais mal lu les spec de collada (un peu compliqué et pas très intuitif), et donc j'avais mal codé, et donc mon tableau de WORD dépassait...

Mais n'empêche qu'après avoir fait tant de recherche sur le std::bad_alloc, on doit être le seul forum à expliciter une cause de son déclenchement autre qu'une saturation de mémoire... C'est toujours ça d'pris :p

Aller à bientôt :)

Aurélien