Discussion :

[Zizou_le_vrai]

Bonjour,

Je d�veloppe une application console avec vS2012 update4 en C++.

suite � l'appel du bout de code suivant :

*this = ResultInter

qui appelle l'op�rateur =

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
6
7
8
9
BOOL_STRUCT& BOOL_STRUCT::operator =(const BOOL_STRUCT&  Op)
{
	OpOk = false;
	// Controle A = A ??? impossible/inutile/erreur
	if(this == &Op || !Op.OpOk || (Op.nbbits <= NULL) || (Op.boolfield == NULL)) return *this;
	
	if(boolfield != NULL) delete [] boolfield;    // erreur entr�e dans _unlock
........
}

d�clenche une erreur et fais appel � la fct ci-dessous se trouvant dans mlock.c

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
/***
* _unlock - Release multi-thread lock
*
*Purpose:
*       Note that it is legal for a thread to aquire _EXIT_LOCK1
*       multiple times.
*
*Entry:
*       locknum = number of the lock to release
*
*Exit:
*
*Exceptions:
*
*******************************************************************************/
 
void __cdecl _unlock ( int locknum )
{
        /*
         * leave the critical section.
         */
        LeaveCriticalSection( _locktable[locknum].lock );
}

pourtant le pointeur existe et bien initialis� avec une adresse (tableau de bool).

pourtant je fais appel � cet op�rateur de multiples fois sans erreur.

Quelqu'un peut il m'aider ?
Merci.

[ternel]

En fait, la faute, c'est de lib�rer un pointeur sans le mettre � NULL alors qu'il continue � �tre accessible.

Du coup, au passage suivant, le pointeur n'est toujours pas nul, donc on le delete encore.
Ceci est un comportement non d�fini (undefined behaviour) provoque une fin du monde parmi toutes celles possibles, dont:

• plantage violent
• cassage de donn�e arbitraire
• formatage du disque dur (pourquoi pas, le comportement n'�tant pas d�fini)
• rien pour le moment

Des quatre, la pire �tant la derni�re, et la premi�re �tant ton symptome.


Au passage, delete ([] ou non) ne fait rien sur un pointeur nul. (que ce soit parce qu'il vaut 0 ou NULL, ou parce qu'il vaut std::nullptr).

Si possible, passe � nullptr, c'est mieux que NULL (car ce dernier est une MACRO)

[prgasp77]

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
if(boolfield != NULL)
{
   delete [] boolfield;
   bootfield = NULL;
}

Je pense que tu passes plusieurs fois par ton delete[], du coup tu lib�res la m�moire une seconde fois (ce qui est ill�gal) ; �a doit lever une exception ou faire un abort() qui te sort de ton contexte et d�truit tes mutex donc appelle unlock(). Pour rappel, delete NULL et delete[] NULL sont autoris�s ; toujours affecter NULL � un pointeur apr�s avoir lib�r� sa m�moire. Ou mieux, ne pas utiliser de pointeurs nus.

[Zizou_le_vrai]

Re Bonjour,

Merci pour vos r�ponses qui m'ont largement aiguill� sur la solution ou plut�t bug (avant la fin du monde).

Le pointeur que je veux 'deleter' a �t� modifi� auparavant en y inscrivant une valeur plus qui occupe une plus grande place m�moire avec memcpy().
L'erreur s'est produite plus loin dans l'op�rateur = au moment de supprimer le pointeur (violation du heap local).

J'ai donc corrig� r�ussi � corriger le probl�me.

Merci encore BEAUCOUP pour vos r�ponses et votre aide.

[Bousk]

memcpy est � proscrire pour une class !
A n'utiliser qu'avec des POD pour �viter toute surprise.