Posté par Cetera .
En réponse au message Sonar Qube faux positif....
Évalué à 1.
Dernière modification le 27 février 2020 à 00:05.
Que fait la fonction resetBoard() ? Elle change la taille du tableau de pointeurs de fonctions appelDeHook[] ? Mais de toute manière, comme elle ne prend pas en paramètre l’index mis en cause, il est certain qu’elle ne peut pas te prémunir du dépassement en élargissant suffisamment le tableau !
Et quand bien même, ce tableau, il faut le remplir avec des pointeurs valides, donc j’imagine que tu ne pourras pas l’étendre autant que nécessaire pour éviter le dépassement.
Bref, Sonar n’a pas tort, et il va falloir en tenir compte. Qu’est-ce que tu veux faire, exactement ? Quel est le comportement attendu si l’index a une valeur négative ou trop grande ? On ne tient pas compte de l’appel ? On exécute appelDeHook[0] avec les paramètres ? On fait encore autre chose ?
# Sonar n’a pas tort...
Posté par Cetera . En réponse au message Sonar Qube faux positif.... Évalué à 1. Dernière modification le 27 février 2020 à 00:05.
Que fait la fonction
resetBoard()? Elle change la taille du tableau de pointeurs de fonctionsappelDeHook[]? Mais de toute manière, comme elle ne prend pas en paramètre l’index mis en cause, il est certain qu’elle ne peut pas te prémunir du dépassement en élargissant suffisamment le tableau !Et quand bien même, ce tableau, il faut le remplir avec des pointeurs valides, donc j’imagine que tu ne pourras pas l’étendre autant que nécessaire pour éviter le dépassement.
Bref, Sonar n’a pas tort, et il va falloir en tenir compte. Qu’est-ce que tu veux faire, exactement ? Quel est le comportement attendu si l’index a une valeur négative ou trop grande ? On ne tient pas compte de l’appel ? On exécute
appelDeHook[0]avec les paramètres ? On fait encore autre chose ?