Discussion :

[gnto]

Bonjour,

Ca fais plusieurs fois que je vois que la fonction scanf est dangereuse ! Alors j'ai pris la peine de regarder le man.

La conversion %s devrait toujours être accompagnée d'une longueur maximale de chaîne de caractères. En effet, il existe un risque de débordement de buffer, qui peut conduire à un trou de sécurité important dans un programme Set-UID ou Set-GID.

Est-elle dangeureuse avec une limitation de la longueur ? et un "%20s" limite-t'il la chaine a 20 caractères ?

Merci d'avance.

[Médinoc]

Même avec une limite, elle reste mal employée, car le buffer clavier n'est pas purifié.
Il y a sur le site un tutoriel scanf() qui permet de purifier le buffer clavier avec juste un scanf() et un getchar()...

[gege2061]

Le problème de scanf c'est qu'elle est souvent utilisée par des débutants mais cette fonction n'est pas simple à utiliser : Scanf démythifiée.

En plus la saisie utilisateur est tout sauf formatée. Par contre dans le cas d'un fichier ça peut être pratique.

[gnto]

Ok, merci beacoup

[Emmanuel Delahaye]

Est-elle dangeureuse avec une limitation de la longueur ?

Si le sujet est maitrisé, non.

et un "%20s" limite-t'il la chaine a 20 caractères ?

Non, à 20 + le 0. Il faut donc un tableau de 21 char. C'est horrible à maintenir

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
 
   char line[21];
 
   scanf ("%20s", line);

Alors que

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
   char line[21];
   fgets(line, sizeof line, stdin);

est clair est net.
Pour être propre :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
   char line[21];
   fgets(line, sizeof line, stdin);
   clean(line, stdin);

avec (à mettre une fois pour toutes en bibliothèque e retirant les 'static', bien sûr...)

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
 
static void purge (FILE * fp)
{
   int c;
   while ((c = fgetc (fp)) != '\n' && c != EOF)
   {
   }
}
 
static void clean (char *s, FILE * fp)
{
   /* search ... */
   char *p = strchr (s, '\n');  /* <string.h> */
   if (p != NULL)
   {
      /* ... and kill */
      *p = 0;
   }
   else
   {
      purge (fp);
   }
}