#ERREUR# potentiellement pointeur non initialisé la variable 'ptrNames' utilisé

J'obtiens l'erreur "potentiellement pointeur non initialisé la variable 'ptrNames " utilisé" si j'exécute ce code:

void processHits (GLint hits, GLuint buffer[]) //Some prints
{
    unsigned int i, j;
    GLuint names, *ptr, minZ,*ptrNames, numberOfNames;
    if (hits == 0)
        noSelected = true;
    else 
        noSelected = false;
    ptr = (GLuint *) buffer;
    minZ = 0xffffffff;
    for (i = 0; i < hits; i++) {
        names = *ptr;
        ptr++;
        if (*ptr < minZ) {
            numberOfNames = names;                                                   
            minZ = *ptr;
            ptrNames = ptr+2;
        }
        ptr += names+2;
    }
    ptr = ptrNames; //Error at this line!
    for (j = 0; j < numberOfNames; j++,ptr++) {
        if (hits > 0)
           LastSelected = *ptr;
    }
}
InformationsquelleAutor float | 2013-10-29
  1. 9

    L'erreur est correcte. ptrNames n'est initialisé si le test if (*ptr < minZ) réussit.

    Le plus simple correctif peut être à l'initialisation d' ptrNames à NULL puis vérifier sa valeur après la première for boucle, le retour (car il n'existe pas de coups de processus) si elle n'a pas été mis à jour.

    void processHits (GLint hits, GLuint buffer[]) //Some prints
{
    unsigned int i, j;
    GLuint names, *ptr, minZ,*ptrNames=NULL, numberOfNames;
    ...
    if (ptrNames == NULL)
        return;
    ptr = ptrNames;
    ...
}
    • dans ce dernier cas ptrNames va encore être initialisée lors de la cession. C'est en dehors de la boucle for, qui pourrait ne pas être facilement visibles à cause de la mauvaise mise en forme.
    • Merci. J'ai mis à jour la mise en forme de la question afin de rendre cela plus clair. Je vais mettre à jour ma réponse dans un sec...
    • Merci, maintenant ça fonctionne!
    InformationsquelleAutor simonc
  2. 1

    Vous devez initialiser votre ptrNames variable à quelque chose. Le if (*ptr < minZ) peut empêcher ptrNames être quelque chose à cet effet, vous aurez un pointeur qui pointe quelque part, vous ne savez pas et potentiellement tout casser.

    InformationsquelleAutor Nico