Discussion :

[jaimelagalette]

Bonjour tout le monde,
je suis en train de me remettre au C et j'ai un petit problème :

je veux lire une ligne entière sur l'entrée standard, j'utilise pour ca la fonction gets (même si elle à l'air dépréciée ...)

cette fonction a marchée toute la journée sur d'autre programme, et depuis quelque heures elle ne veux plus : rien que le simple programme suivant me renvoit une erreur de segmentation dès que je rentre und chaîne ...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
#include<unistd.h>
#include<stdio.h>
#include<stdlib.h>
#include<string.h>
 
int main(int argc, char **argv) {
	char* phrase;
	gets(phrase);
	exit(0);
}

Si quelqu'un a une idée de où ça pourrait venir ça m'arrangerait bien,
merci d'avance !!

[ram-0000]

D'abord tu l'as dit toi même gets() est dépréciée, il faut lui préférer fgets().

Ceci dit, ton bug vient du faut que tu ranges la chaine lue par gets dans un buffer aléatoire.

char* phrase est un pointeur qui n'est pas initialisé et qui pointe n'importe ou.

Code :

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

1
2
3
4
5
int main(int argc, char **argv) {
	char phrase[1024];
	gets(phrase);
	exit(0);
}

devrait corriger ton problème.

La limitation de gets(), c'est que si l'utilisateur est vicieux, il saisi plus de 1023 caractères et ton beau programme part en live car il n'y a rien pour dire à gets() de ne lire que 1023 caractères parce que le buffer fait 1024 caractères

[the-death]

La limitation de gets(), c'est que si l'utilisateur est vicieux, il saisi plus de 1023 caractères et ton beau programme part en live car il n'y a rien pour dire à gets() de ne lire que 1023 caractères parce que le buffer fait 1024 caractères

Je rajouterai qu'au mieux c'est le bug mais au pire c'est des failles de sécurité relativement simples à exploiter. Même si lorsque l'on débute on ne pense pas forcément à la sécurité, celle-ci ne doit pas être négligée.