PLANSCAN : SCANNEUR DE FICHIER ET CONFIGURATION PC

Commentaire de BruNews le 18/10/2006 02:15:49 administrateur CS

strncpy (cMacAdress, "000000000000", 12);
La version 'N' de strcpy sert à quoi ici ???

void CharConcat (char *cOutput, char *cConcat, int iOutputSize)
{
  int iSize;
  iSize = strlen (cOutput) + strlen(cConcat) + 1;
  if (iSize <= iOutputSize)
    strncat(cOutput, cConcat, strlen(cConcat) + 1);
}
Tu es certain d'avoir parcouru cConcat suffisamment ???

void CharConcat (char *cOutput, char *cConcat, int iOutputSize)
{
  int iSize, srclen;
  srclen = strlen (cConcat);
  iSize = strlen(cOutput) + srclen + 1;
  if (iSize <= iOutputSize)
    strncat(cOutput, cConcat, srclen + 1);
}
N'allait pas aussi bien ??? (c'est peu dire).
Encore que strNcat... verif deja faite avant, strcat faisait idem en plus court.

Pour finir, la même que la tienne en conservant le même controle mais avec 1 seul parcours de chaque chaine:
void CharConcat (char *cOutput, char *cConcat, int iOutputSize)
{
  int iSize, srclen;
  srclen = strlen (cConcat);
  iSize = strlen(cOutput);
  cOutput += iSize; // OUST strcat, 1 SEUL PARCOURS
  if((iSize + srclen + 1) <= iOutputSize) strcpy(cOutput, cConcat);
}

Je suis au courant que dans certaines boites, il faut que ce soit mal codé pour faire 'sécurisé'.
Par contre, quand on n'a plus l'oeil du chef par dessus son épaule, il faut tout de même garder l'habitude de coder correctement.