Commande linux dans un code C

Version imprimable

22/03/2012, 16h04
chercheur111

Commande linux dans un code C
Bonjour,

Mon code est le suivant:
Code:

1 2 3 4 5 6 #include <stdio.h> void main() { system("cd rep"); system("ls"); }
Lors de l'exécution, il m'affiche la liste des fichiers et dossiers qui sont dans le répertoire courant et non pas ce qui est dans le répertoire rep.
Est-ce que vous une idée sur ce problème?

Merci.
22/03/2012, 16h39
messinese

Bonjour,

Tout d'abord pour information la fonction "system" est a proscrire, essaye de regarder du coté des fonctions

Code:

exec* ( execve,execl.. )

Sinon pour répondre facilement a ta question c'est tout a fait normal tu peux bypasser ce "probleme" (qui n'en ai pas un) de la maniere la moins élégante qui soit mais puisque c'est ton choix, de la maniere suivante:

Code:

system("ls rep/");

je m'excuse d'avance auprés du reste des membres de ce forum..

Cordialement.
22/03/2012, 17h53
troumad

Citation:

Envoyé par messinese

Code:

system("ls rep/");

ou

Code:

system("cd rep/;ls");

Non ?

Citation:

Envoyé par messinese

je m'excuse d'avance auprés du reste des membres de ce forum.

? Tu veux dire quoi ?

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
#include  <stdio.h>
#include  <stdlib.h>
#include  <sys/types.h>
#include  <unistd.h>
/* Créer un processus fils exécutant un nouveau programme. PROGRAM est le nom 
   du programme à exécuter ; le programme est recherché dans le path.
   ARG_LIST est une liste terminée par NULL de chaînes de caractères à passer 
   au programme comme liste d'arguments. Renvoie l'identifiant du processus
   nouvellement crée. */
int
spawn (char *program, char **arg_list)
{
  pid_t child_pid;
  /* Duplique ce processus. */
  child_pid = fork ();
  if (child_pid != 0)
    /* Nous sommes dans le processus parent. */
    return child_pid;
  else
    {
      /* Exécute PROGRAM en le recherchant dans le path. */
      execvp (program, arg_list);
      /* On ne sort de la fonction execvp uniquement si une erreur survient. */
      fprintf (stderr, "une erreur est survenue au sein de execvp\n");
      abort ();
    }
}
 
int
main ()
{
  /* Liste d'arguments à passer à la commande "ls". */
  char *arg_list[] = {
    "ls",                       /* argv[0], le nom du programme. */
    "-l",
    "/",
    NULL                        /* La liste d'arguments doit se terminer par NULL.  */
  };
  /* Crée un nouveau processus fils exécutant la commande "ls". Ignore
     l'identifiant du processus fils renvoyé. */
  spawn ("ls", arg_list);
  printf ("Fin du programme principal\n");
  return 0;
}

si ça peut aider j'ai retrouvé ça dans un de mes vieux cours de C

23/03/2012, 08h45
messinese

Citation:

Envoyé par troumad

ou

Code:

system("cd rep/;ls");

Non ?

? Tu veux dire quoi ?

Salut,
je veux juste dire que c'est trés crade niveau code :lol: je pense que tu sera d'accord ;) ...
23/03/2012, 09h33
chercheur111

Merci pour vos réponses.

Citation:

Envoyé par messinese

je veux juste dire que c'est trés crade niveau code :lol: je pense que tu sera d'accord ;) ...

moins élégante, crade !! explique nous pourquoi c'est crade, c'est mieux ;)
23/03/2012, 10h25
messinese

Citation:

Envoyé par messinese

Bonjour,

Tout d'abord pour information la fonction "system" est a proscrire, essaye de regarder du coté des fonctions

Code:

exec* ( execve,execl.. )

Alors, la différence premiere (bien qu'un peu inutile ici) est que execve prend le meme PID que le processus qui l'aura lancé et arrete ce meme processus a la fin alors que system() cree un nouveau processus puis une fois son exécution finie il revient juste apres son appel pour poursuivre son prologue.

En effet ici sur un code si léger aucun soucis mais par exemple tu pourrais aussi tout a fait utiliser ce type de code et etre confronté a un probleme de sécurité (faille de type "system") qui permettrai alors l'execution de code avec les droits attribués a ce binaire.

Une des façons les plus fiable est donc l'emploi d'une fonction de type "exec" afin de palier a ce probleme de sécurité et de prendre de bonne habitude dés le développement de tes codes.

Pense aussi a inclure

Code:

#include <unistd.h>

afin de les utiliser.

J'espere t'avoir éclairé .
23/03/2012, 17h11
troumad

Citation:

Envoyé par messinese

je veux juste dire que c'est trés crade niveau code :lol: je pense que tu sera d'accord ;) ...

Je suis bien d'accord sur le fait qu'il vaux mieux éviter. Mais parfois, pour un petit programme, c'est tellement plus simple !
23/03/2012, 22h42
chercheur111

Merci pour vos réponses.
24/03/2012, 09h20
ok.Idriss
Bonjour.

Au passage, main() doit être de type int et retourner un int (0 en cas de succès, 1 en cas d’échec). Des macros sont d'ailleurs prévues pour ça : EXIT_SUCCESS et EXIT_FAILURE.

Exemple :
Code:

1 2 3 4 5 #include <stdlib.h> int main(void){ /* ... */ return EXIT_SUCCESS; }
Autre question : l'exercice imposes t-il de passer par les commandes systèmes ou bien de simplement lister le contenu d'un répertoire ?

Cordialement,
Idriss
24/03/2012, 19h52
chercheur111

Oui, passer par les commandes systèmes. J'ai utilisé "ls" au début juste pour tester la fonction system ...