Discussion :

[mimou__]

Bonjour,

j'essaie de faire un programme pour obtenir les information sur une commande passée en paramètre.
J'ai essayé ce bout de code mais j'ai un peu de mal.

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
 
#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <string.h>
 
int main()
{
	char *manu ="man " ;
	char *commande;
	scanf("%s", &commande);
 
	strcat(manu, commande);
	system(manu);
	return 0;
}

Merci d'avance.

[xdelatour]

Bonsoir,

Je vois au moins trois erreurs :

• Ligne 9 : manu est une chaîne de taille fixe puisqu'initialisée avec "man ", du coup il est impossible de la concaténer avec autre chose ligne 13
• Ligne 10 :commande est un pointeur sur char, mais il n'est pas initialisé, donc il pointe nulle part
• Ligne 11 : &commande est passé en paramètre de scanf. scanf attend un pointeur, pas l'adresse d'un pointeur. Et même si c'était correct, comme commande ne pointe nulle part, la saisie irait stocker les caractères n'importe où

[Sve@r]

Bonjour

La variable "manu" est un pointeur (une simple adresse). Si tu la fais pointer sur une chaine statique (ici "man") alors c'est ok mais ça n'empêche pas la chaine en question d'être et rester statique donc tu ne peux pas la modifier et encore moins lui rajouter du texte.
La variable "commande" est aussi un pointeur (une simple adresse) toutefois elle, elle n'est pas initialisée donc pointe vers n'importe où dans la mémoire. As-tu le droit alors d'aller à cet emplacement et le remplir avec du texte? Règle fondamentale d'un pointeur (ex pt): ne jamais regarder (et encore moins modifier) l'emplacement vers lequel il pointe (*pt) si tu n'écris pas auparavant pt=adresse_valide. C'était le cas pour "manu" (même s'il pointe vers une zone statique non modifiable tu peux quand-même regarder son contenu), ce n'est pas le cas pour "commande".
La fonction scanf(), si on lui passe "%s", veut en contrepartie un "char étoile" en paramètre. Tu lui passes &commande. La variable "commande" étant un "char étoile", son adresse est donc un "char étoile étoile". Or un "char étoile étoile" ce n'est pas un "char étoile".

Bref comme il est clair que passer par des pointeurs sans précautions n'est pas vraiment une bonne idée, alors on les supprime.

Code c :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <string.h>
 
int main() {
	char commande[255+1];
	scanf("%s", commande);
 
	char syst[255+4+1]; 
	sprintf(syst, "man %s", commande)
	system(syst);
	return 0;
}

Ce n'est pas un code super blindé (rien n'interdit par exemple de saisir une commande plus grande que 255 caractères) mais ça permet d'avoir un point de départ. Ensuite si tu veux un code blindé il y aura d'autres possibilités soit avec une limitation à la saisie (fgets), soit avec une allocation dynamique pour être sûr d'avoir assez de place (malloc).

[mimou__]

Re,

Oui je comprends mieux maintenant.
Merci beaucoup pour vos reponses.