Fonction system() avec commandes unix dans boucle

Version imprimable

24/09/2009, 17h04
vaderetro33

Fonction system() avec commandes unix dans boucle
J'ai un petit souci, la commande system("ma commande unix"); ne fonctionne plus lorsque je l'insère dans une boucle.

par exemple :
Code:

1 2 3 4 5 6 7 8 9 10 char command[256]; int a; for (a =0; a < 5; ++a) { sprintf(command, "echo %s >> test.txt", a); system(command); }
Merci de votre aide.
24/09/2009, 17h27
sofiane80

ton a est un int alors change le %s par %d

Code:

sprintf(command, "echo %d >> test.txt", a);

Voici mon code, la première utilisation de system("find......"); fonctionne, mais la deuxieme avec system("sed...) ne fonctionne pas, alors que lorsque je tape le code retourné par le printf la commande s'exécute bien et supprime bien la ligne à l'intérieur du fichier log. Je n'arrive a exécuter aucune commande unix dans la boucle où est le second system(), même pas un simple echo.

Voici le code :

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
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
 while(a==1)
    {
	    FILE * logindex;
		char index_name[64];
		strcpy(index_name,"logindex3.log");
        logindex = fopen("logindex3.log","r+");
        if(file_exists_non_empty("logindex3.log")==0)
        {
            char files_rep[64];
			strcpy(files_rep, "./logwork/3");
 			char command[256];
				    sprintf(command, "find %s -type f -name \"*.log\" >>allresults.txt && sort allresults.txt >>sortedresult.txt && rm allresults.txt && head -n 100 sortedresult.txt >>%s && rm sortedindex.txt;", files_rep, index_name);
					    system(command);
 
 
			printf("%s", "je rentre dans la premiere boucle");
 
 
        }
		else //on vérifie que logindex contienne des lignes !
        {
					printf("%s", "je rentre dans la deuxieme boucle !");
                    char * pathlog;
                    fscanf(logindex,"%[^\n]", pathlog);    //on copie le chemin vers le fichier log à traiter
                    FILE * log;
                    log = fopen(pathlog, "r+");
					printf("%s", pathlog);
 
                    	/////////// Varibales de récupération des données du fichier log ////////////////
                        char  timestamp[256];
                        char  ip[256];
                        char  port[256];
                        char  eventtype[256];
                        char  id_box[256];
                        char  event[256];
						///////////////////////////////////////////////////////////////////////////////
						char chaine[200];
                        if(file_exists_non_empty(pathlog)==0)
                        {
 
                            file_line_delete(pathlog,"logindex3.log"); //Suppression du log de l'index
                            file_delete(pathlog);   // Suppression du fichier log
 
                        }// fin if
 
                        else
                        {
                            while(log !=NULL)
                            {
							//	fgets(chaine,125, log); 
                            // fgetc(log); // ligne par ligne
 
                               	char ligne[256];
		                        fscanf(log,"%[^\n]", &ligne);
								printf("%s\n", ligne);
                                fscanf(log,"%[^;];%[^;];%[^;];%[^;];%[^;];%s", &timestamp, &ip, &port, &eventtype, &id_box, &event);
 
								printf("%s\n %s\n %s\n %s\n %s\n %s\n", timestamp, ip, port, eventtype, id_box, event);
                            	printf("%s", pathlog);
								fclose(log);
								char line[64];
								strcpy(line, ligne);
								char filename[64];
								strcpy(filename, pathlog);
								char command[256];
								sprintf(command, "sed -i '/%s/d' %s", line, filename);
								system(command);
								log = fopen(pathlog, "r+");
								printf("\n%s\n", command);	
 
                                char query[500];
                                sprintf(query, "INSERT INTO mf_boxes_datalogs(dtlog_added_date,bx_ip,evt_type,bx_id,bx_data) VALUES ( '%s','%s', '%s', '%s', '%s')", ip, ip, eventtype, id_box, event);
 
							   	res = PQexec(conn, query);
 
								testco = PQresultStatus(res);
 
								status = PQresStatus(testco);
								printf("\n%s\n", status);
//	 							PQfinish(conn); // fermeture de la connection
 
                                ///////// Effacement de la ligne dans le fichier log /////////////
								//char delete[256];
    	                       // sprintf(delete, "sed '/%s/d' %s ;", ligne, pathlog);
 
                        	    }// fin while
 
                        }// fin else
 
        }//fin else if
 
 
 
    }// fin boucle principale

Merci de votre aide.

En doublan,t les caractères \ ?
Code:

1 2 3 fscanf(logindex,"%[^\\n]", pathlog); ... fscanf(log,"%[^\\n]", &ligne);

25/09/2009, 09h18
vaderetro33

En doublant les caractère j'obtient une erreur de segmentation, le programme fonctionne comme cela, lorsque je fait un printf de la chaine command (exécutée dans le system()) elle est correcte car en la collant dans le shell elle s'exécute. Le problème c'est que le deuxième system() n'exécute rien, même pas un echo salut !
25/09/2009, 09h27
ram-0000

Arf !!

Code:

fscanf(log,"%[^\\n]", &ligne);

Je pense que le caractère & est superflu, ligne est déjà une adresse

Autre chose, tu es sûr de ton format (%[^\n]). Il manque pas un truc genre "%[^\n]s". (Sur ce coup, je ne suis pas sûr de moi, je ne sais pas).
25/09/2009, 09h32
vaderetro33

Les fscanf fonctionnent, j'affiche les varibales retournées avec des printf et elles sont correctes, la chaine de commande dans le system() est aussi correcte, mais la fonction system ne s'exécute pas.
Les bibliothèques sont bien incluses.
25/09/2009, 09h41
vaderetro33

Le gros souci c'est de savoir pourquoi je ne peut pas exécuter la fonction system à son deuxième appel avec une commande toute simple.
25/09/2009, 09h41
ram-0000

tu pourrais peut être tester et afficher le code de retour de la fonction system()

et aussi essayer de mieux indenter/présenter ton code par la même occasion :aie:
25/09/2009, 09h43
vaderetro33

Comment je peux faire cela ?
25/09/2009, 09h59
ram-0000
faire quoi ?
tester le code retour de system() ?
Code:

1 2 int ret = system(....) printf("Code ret de system() = %d\n", ret);
mieux présenter/indenter ?
En éditant ton post initial (ou en postant ton nouveau code) et en essayant d'aligner les blocs de code de manière à rendre plus lisible la structure pour faciliter la lecture.

Tout d'abord la fonction system ne renvoi rien, ensuite voici le code réindenté :

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
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
while(a==1)
    {
	    FILE * logindex;
		char index_name[64];
		strcpy(index_name,"logindex3.log");
        logindex = fopen("logindex3.log","r+");
        if(file_exists_non_empty("logindex3.log")==0)	//Si logindex3 n'existe pas ou ne contient pas de lignes on le créé
        {
            char files_rep[64];
			strcpy(files_rep, "./logwork/3");
 			char command[256];
			sprintf(command, "find %s -type f -name \"*.log\" >>allresults.txt && sort allresults.txt >>sortedresult.txt && rm allresults.txt && head -n 100 sortedresult.txt >>%s && rm sortedindex.txt;", files_rep, index_name);
			system(command);
 
			printf("%s", "je rentre dans la premiere boucle");
 
 
        }	//Fin du if
 
		else 
        {
					printf("%s", "je rentre dans la deuxieme boucle !");
 
                    char * pathlog;
                    fscanf(logindex,"%[^\n]", pathlog);    //on copie le chemin vers le fichier log à traiter
                    FILE * log;
                    log = fopen(pathlog, "r+");
 
					printf("%s", pathlog);
 
                    /////////// Varibales de récupération des données du fichier log ////////////////
                    char  timestamp[256];
                    char  ip[256];
                    char  port[256];
                    char  eventtype[256];
                    char  id_box[256];
                    char  event[256];
					//////////////////////////////////////////////////////////////////////////////
					char chaine[200];
 
					if(file_exists_non_empty(pathlog)==0)	//Si le fichier log n'existe pas ou ne contient plus de ligne on le supprime
                        {
 
                            file_line_delete(pathlog,"logindex3.log"); //Suppression du log de l'index
                            file_delete(pathlog);   // Suppression du fichier log
 
                        }// fin if
 
                        else
                        {
                            while(log !=NULL)
                            {
 
                            // fgetc(log); // ligne par ligne
 
                               	char ligne[256];
		                        fscanf(log,"%[^\n]", ligne);
 
								printf("%s\n", ligne);
                                fscanf(log,"%[^;];%[^;];%[^;];%[^;];%[^;];%s", timestamp, ip, port, eventtype, id_box, event);
 
								printf("%s\n %s\n %s\n %s\n %s\n %s\n", timestamp, ip, port, eventtype, id_box, event);
                            	printf("%s", pathlog);
								fclose(log);
 
								///////// Effacement de la ligne dans le fichier log /////////////
								fflush(stdin);
								char line[64];
								strcpy(line, ligne);
								char filename[64];
								strcpy(filename, pathlog);
								char command2[256];
								sprintf(command2, "sed -i '/%s/d' %s", line, filename);
								system(command2);
 
								int ret;
								ret = system(command2);
								printf("Code ret de system() = %d\n", &ret);
								/////////////////////////////////////////////////////////////////
 
								log = fopen(pathlog, "r+");
								printf("\n%s\n", command2);	
 
                                char query[500];
                                sprintf(query, "INSERT INTO mf_boxes_datalogs(dtlog_added_date,bx_ip,evt_type,bx_id,bx_data) VALUES ( '%s','%s', '%s', '%s', '%s')", ip, ip, eventtype, id_box, event);
 
							   	res = PQexec(conn, query);
 
								testco = PQresultStatus(res);
 
								status = PQresStatus(testco);
								printf("\n%s\n", status);
 
 
                        	}// fin while
 
                       }// fin else
 
        }//fin else
 
 
 
    }// fin boucle principale

25/09/2009, 10h07
vaderetro33

Pardon la fonction system renvoi -915223760
25/09/2009, 10h09
ram-0000

comment cela, elle ne retourne rien ?

Fait l'effort de copier ce que je te donne
Ce que je t'avais proposé

Code:

printf("Code ret de system() = %d\n", ret);

Ce que tu as écrit et ce n'est pas la même chose, cela qui introduit un crash potentiel de ton application

Code:

printf("Code ret de system() = %s\n", &ret);
25/09/2009, 10h14
vaderetro33

Avec ton code j'obtient une erreur de segmentation à l'exécution.
25/09/2009, 10h32
ram-0000

Je pense surtout que mon code révèle un problème qui existait déjà avant dans ton code.

As tu essayé de debugger pour voir quelle est la ligne de code qui produit le problème ? De cette manière, tu auras aussi les paramètres utilisés à cette ligne de code pour aider à comprendre le problème.
25/09/2009, 10h34
vaderetro33

La ligne de code qui produit cette erreur c'est le printf de ton code.
25/09/2009, 12h28
vaderetro33

Je pense que le problème vient de l'utilisation de la fonction system dans un while, lorsque je la place en début du programme elle fonctionne, dans le while elle ne s'exécute pas.
Si vous avez des solutions...

Citation:

Envoyé par vaderetro33

Je pense que le problème vient de l'utilisation de la fonction system dans un while,
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 #include <stdio.h> #include <stdlib.h> int main (int argc, char *argv[]) { int i =0; int ret ; while (i<5){ ret = system("ls ./"); printf("%d %d\n", ret, i); i++; } return 0; }
Le problème est ailleurs comme le dit ram-0000

Uitlises-tu gdb ?
Si oui tu devrais lancé ton prog avec, et nous retourner ici, après la seg fault le résultat.

25/09/2009, 13h52
gl
Citation:
Envoyé par ram-0000

En doublan,t les caractères \ ?

Code:

1 2 3 fscanf(logindex,"%[^\\n]", pathlog); ... fscanf(log,"%[^\\n]", &ligne);
Citation:

Envoyé par ram-0000

Autre chose, tu es sûr de ton format (%[^\n]). Il manque pas un truc genre "%[^\n]s". (Sur ce coup, je ne suis pas sûr de moi, je ne sais pas).

Non, la syntaxe est correct, il ne faut pas doubler les \ ni rajouter de s. "%[^\n]" permet de lire tout les caractères jusqu'au \n.
Par contre il n'y a aucune vérification sur la taille lue et donc un potentiel débordement de buffer.

Il est préférable d'utiliser ici fgets() + recherche/suppression du \n afin de gérer la taille du buffer.
Citation:
Envoyé par vaderetro33

Code:

1 2 3 char * pathlog; fscanf(logindex,"%[^\n]", pathlog); //on copie le chemin vers le fichier log à traiter
pathlog n'est pas un pointeur valide ici.
25/09/2009, 14h38
gl
En regardant plus attentivement le code, il y a plusieurs autres problèmes potentiels :
- Il manque les tests des codes retour des différentes fonctions, en particulier de fopen().
- Il n'y a pas de vérifications des formats et de la taille des buffer. A la moindre erreur de format dans un des fichiers et le programme part en sucette.
- Pourquoi ouvrir les fichiers en mode "r+", en général c'est source de problème. Si tu veux seulement les lire, "r" suffit.
- Certains fichiers ouverts dans la boucle ne sont pas refermés (mais réouverts au passage suivant), par exemple "logindex3.log".
- Le comportement de fflush(stdin) n'est pas défini par la norme. En outre je ne vois pas ce que tu essaies de faire ici avec cet appel.
- Tu manipules les fichiers à la fois directement dans ton code (appel à fopen, fscanf, etc.) et au travers d'autres fonctions qui manipules le nom de fichier (file_line_delete, file_delete, etc.). Es-tu certain qu'il n'y a pas d'interférence possible entre les deux ? En général ce n'est pas une bonne idée de mélanger les styles de la sorte.
- L'écriture
  Code:
  
  1 2 char files_rep[64]; strcpy(files_rep, "./logwork/3");
  est une façon compliquée d'écrire
  
  Code:
  
  char files_rep[] = "./logwork/3";
  
  En outre je ne vois pas l'intérêt de passer par une variable temporaire ici, autant l'utiliser directement dans la chaîne de commande.
- Beaucoup de valeur numérique et de chaîne de caractère en dur dans le code. Il serait plus simple de les passer en globales "static const" ou en define.
- Pourquoi créer et initialiser index_name pour ensuite utiliser la chaîne en dur dans le code et pas la variable ?
- Les commentaires // et la déclaration de variable au milieu du code sont autorisé en C99 pas en C90.
- Certaines opérations et déclarations de variable gagneraient probablement à être sortie de la boucle.
- Certains appels à fscanf() pourrait avantageusement être remplacés par des appels à fgets().
- Puisque tu utilises visiblement un compilateur C99, tu devrais remplacer sprintf() par snprintf().
07/10/2009, 10h33
vaderetro33

J'ai corrigé mon code suivant tes indication et ca fonctionne, t'es un vrai chef, merci !