Discussion :

[bruce-willis]

Bonjour,

Dans le but de bien coder et �viter tout bug de mon code, comment traiter un pointeur avant de l'utiliser c'est � dire mettre un point ou un -> apr�s?

Code :

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

1
2
if (pTable->GetValue(6))
               ((CBCGPGridItem*)(pTable->GetValue(6)))->AddFieldYard(*cClass);

(ici GetValue retourne un pointeur)
ne suffit pas je pense car quelquefois les pointeurs prennent des valeurs non valides mais pas NULL: 0xCCCCCCCC, 0xCDCDCDCD, etc*

Je ne sais pas si je trop pr�voyant mais les codes open source que j'ai regard� ne pr�tent pas attention � cela!! Or je pense qu'il faut toujours tester un pointeur avant de l'utiliser, quelquefois, on a d�j� lib�r� le pointeur avant

[TheGzD]

Il faut toujours initialiser ses pointeurs. Et bien �videmment les tester avant usage pour v�rifier qu'ils ne sont pas �gaux � NULL.

[bruce-willis]

Il faut toujours initialiser ses pointeurs. Et bien �videmment les tester avant usage pour v�rifier qu'ils ne sont pas �gaux � NULL.

C'est un peu ce que j'avais dit!
Un pointeur non NULL ne veut pas dire OK !!!

[bruce-willis]

Un article en anglais relevait les 5 erreurs en programmation C++:
1. Null Pointer dereference
2. Null pointer dereference from function
3. Memory leaks
4. Array index out of bounds
5. Uninitialized variables

http://kloctalk.klocwork.com/?tag=nu...er-dereference

Ce que j'aimerais apprendre c'est comment g�rer 1. Null Pointer dereference

[ram-0000]

Un truc comme cela ? (ou alors je n'ai pas compris)

Code :

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

1
2
3
4
5
6
7
8
9
...
if(ptr == NULL)
{
   // leve une exception ou retourne une erreur (au choix)
   throw exception;
   return false;
}
int ma_variable = *ptr;
...

[Neitsa]

Bonjour,

Pour g�rer un pointeur non NULL mais pointant vers une m�thode ou un membre donn�e incorrect on pourrait �tre tenter de g�rer l'exception r�sultante (via try / catch par exemple) mais c'est une tr�s mauvaise id�e.

On trouve souvent ce genre de probl�me lors de tests d'injection de fautes sur programmes.

La solution est de prendre le probl�me � l'envers: non pas "comment je g�re ce probl�me ?" mais se demander plut�t "comment puis-je pour que cela n'arrive pas !"

Chaque probl�matique appelle une solution pr�cise.

Exemple: Dans le cas d'un "use after free" - utilisation d'un pointeur ou de donn�es en m�moire alors que le pointeur (ou le bloc m�moire) a d�j� �t� lib�r� - il faut s'assurer que justement on ne va pas tenter d'utiliser un pointeur d�j� lib�rer.

Solution: Quand un pointeur est lib�r�, on le force automatiquement � NULL et tous les tests subs�quents s'effectueront sur NULL, comme le montre l'exemple de ram-0000.

La base de donn�e des faiblesses communes en programmation s'appelle CWE (Common Weakness Enumeration), je la recommande vivement:

http://cwe.mitre.org/data/index.html

Exemple avec le CWE-416 (use after free) :

http://cwe.mitre.org/data/definitions/416.html

[bruce-willis]

Je suis entrain de lire ce tuto: http://loic-joly.developpez.com/tuto...mart-pointers/ pour voir si �a me convient?

Neitsa, je ne comprends pas bien o� veux-tu en venir?

[Gabrielly]

Si tu sais avec pr�cision que tu n'utilises plus ton pointeur tu l'initialises � NULL

Code :

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

1
2
3
4
5
6
7
 
// work work work...
if(pObject)
{
        delete pObject;     // bon j'en finis avec 
        pObject = NULL;   // et je le ramène à NULL pour prévenir.
}

[bruce-willis]

New est un op�rateur retournant un pointeur, donc � g�rer aussi!

Que diriez-vous si je fais � partir de maintenant cette transformation � chaque utilisation de NEW

Code :

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

1
2
3
4
5
6
	CLoad* pSET = new CLoad(44, NULL);
 
	if (pSET)
	{
		m_pListEntity->push_back(pSET);
	}

devient

Code :

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

1
2
3
   std::auto_ptr<CLoad> pSET(new CLoad(44, NULL));
   m_pListEntity->push_back(pSET.get());
   pSET.release();

Utilisation de auto_ptr dans l'en-t�te <memory>