Discussion :

[PyNub]

Bonjour tout le monde,

J'ai un arrêt brutal de mon programme avec ce message d'erreur suivi d'un rapport sur l'état de la mémoire :

Code :

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

*** glibc detected *** ./miksh: double free or corruption (fasttop): 0x00000000000603010***

Apparemment je libère deux fois une allocation mais je ne vois rien dans le programme qui cloche...

si quelqu'un sait ce que ça veut dire il est le bienvenu...

Merci d'avance pour les réponses

[gerald3d]

...Apparemment je libère deux fois une allocation...

Tu as tout dit . Maintenant sans code à nous mettre sous la dent on ne peut pas faire grand chose...

[PyNub]

En fait je libère une variable que j'ai créée auparavant sur la pile du coup c'est sûr ça ne marche pas....

En fait je suis en train de faire un shell. J'ai résolu le problème de la libération.
Mais j'ai des problèmes avec certains enchainements de commande.
Par exemple un : cat fichier
apres un : ps
Cela vient de ma fonction lecture que j'ai fait un peu vite... Il faut que je revois ça...
le code :

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
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
#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <sys/wait.h>
#define MAX_SAIS 50
#define TRUE 1
 
void prompt();
void lecture(char * cmd,char ** params);
 
char * cmd;
char ** params;
 
void clean_stdin(void){
	int c;
	do{
		c=getchar();
	} while(c!='\n' && c!=EOF);
}
 
int main(){
	int statut;
	int i=0;
	pid_t child_pid;
	while (TRUE){
		params=(char**)malloc(10);
		cmd=(char*)malloc(MAX_SAIS);
		prompt();
		lecture(cmd,params);
		child_pid=fork();
		if (child_pid !=0){
			printf("%d\t%d\n",getpid(),child_pid);
			waitpid(-1,&statut,0);
		}
		else {
			printf("%d\n",getpid());
			execvp(cmd,params);
			printf("%s\n",cmd);
			i=-1;
			do {
				i++;
				printf("%s\t",params[i]);
			}while(params[i]!=NULL);
			printf("\n");
		}
		free(cmd);
		i=0;
		do{	
			free(*(params+i));
			i++;
		}while(*(params+i)!=NULL);
		free(params);
		i=0;
		clean_stdin();
	}
}
 
void prompt(){
	printf("MiKaShEll>> ");
}
 
void lecture(char * cmd, char ** params){
	int i=0;
	int j=0;
	int k=0;
	char tmp[MAX_SAIS];
	scanf("%[^\n]",tmp);
	printf("%s\n",tmp);
	while((tmp[i]!='\0')&&(tmp[i]!='\n')&&(tmp[i]!=' ')&&(i<MAX_SAIS)){
		*(cmd+i)=*(tmp+i);
		i++;
	}
	*(cmd+i)='\0';
	*(params+j)=(char*)malloc(MAX_SAIS);
	while((cmd[k]!='\0')&&(cmd[k]!='\n')&&(cmd[k]!=' ')&&(k<MAX_SAIS)){
		*(*(params+j)+k)=*(cmd+k);
		k++;
	}
	k=0;
	j++;
	if (*(tmp+i)==' ') i++;
	while((tmp[i]!='\0')&&(tmp[i]!='\n')&&(i<MAX_SAIS)){
		*(params+j)=(char*)malloc(MAX_SAIS);
		while((tmp[i]!='\0')&&(tmp[i]!='\n')&&(tmp[i]!=' ')&&(i<MAX_SAIS)){
			*(*(params+j)+k)=*(tmp+i);
			i++;
			k++;
		}
		*(*(params+j)+k)='\0';
		if (tmp[i]=='\n') break;
		i++;
		j++;
		k=0;
	}
	*(params+j)=(char*)malloc(MAX_SAIS);
	*(params+j)=NULL;
}

Merci !

[PyNub]

Bon j'ai un peu modifié le code :

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
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
104
105
106
107
108
109
#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <sys/wait.h>
#define MAX_SAIS 50
#define TRUE 1
 
void prompt();
void lecture(char * cmd,char ** params);
 
char * cmd;
char ** params;
 
void clean_stdin(void){
	int c;
	do{
		c=getchar();
	} while(c!='\n' && c!=EOF);
}
 
int main(){
	int statut;
	int i=0;
	pid_t child_pid;
	while (TRUE){
		params=(char**)malloc(10);
		printf("alloue params : %p\n",params);
		cmd=(char*)malloc(MAX_SAIS);
		printf("alloue cmd : %p\n",cmd);
		prompt();
		lecture(cmd,params);
		child_pid=fork();
		if (child_pid !=0){
			printf("%d\t%d\n",getpid(),child_pid);
			waitpid(-1,&statut,0);
		}
		else {
			printf("exécute : %d\n",getpid());
			execvp(cmd,params);
			printf("%s\n",cmd);
			i=-1;
			do {
				i++;
				printf("%s\t",params[i]);
			}while(params[i]!=NULL);
			printf("\n");
		}
		i=0;
		do{	
			printf("libère %p : %s\n",*(params+i),*(params+i));
			free(*(params+i));
			printf("libéré\n");
			i++;
		}while(*(params+i)!=NULL);
		printf("libère NULL : %p\n",*(params+i));
		free(*(params+i));
		printf("libéré\n");
		printf("Libère cmd : %p\n",cmd);
		free(cmd);
		printf("libéré\n");
		printf("libère params : %p\n",params);
		free(params);
		printf("libéré\n");
		i=0;
		clean_stdin();
	}
}
 
void prompt(){
	printf("MiKaShEll>> ");
}
 
void lecture(char * cmd, char ** params){
	int i=0;
	int j=0;
	int k=0;
	char tmp[MAX_SAIS];
	scanf("%[^\n]",tmp);
	printf("%s\n",tmp);
	*(params+j)=(char*)malloc(MAX_SAIS);
	while((tmp[i]!='\0')&&(tmp[i]!='\n')&&(tmp[i]!=' ')&&(i<MAX_SAIS)){
		*(cmd+i)=*(tmp+i);
		*(*(params+j)+i)=*(tmp+i);
		i++;
	}
	*(cmd+i)='\0';
	*(*(params+j)+i)='\0';
	k=0;
	j++;
	if (*(tmp+i)==' ') i++;
	while((tmp[i]!='\0')&&(tmp[i]!='\n')&&(i<MAX_SAIS)){
		*(params+j)=(char*)malloc(MAX_SAIS);
		printf("params[%d] alloué %p\n",j,*(params+j));
		while((tmp[i]!='\0')&&(tmp[i]!='\n')&&(tmp[i]!=' ')&&(i<MAX_SAIS)){
			*(*(params+j)+k)=*(tmp+i);
			i++;
			k++;
		}
		*(*(params+j)+i)='\0';
		printf("params[%d]:\t%s\n",j,*(params+j));
		if (tmp[i]=='\n') break;
		i++;
		j++;
		k=0;
	}
	*(params+j)=(char*)malloc(MAX_SAIS);
	*(params+j)=NULL;
	printf("params[%d]:\t%s\n",j,*(params+j));
}

J'ai fait en sorte que les allocations et libération mémoire apparaissent. Le programme fonctionne bien avec une commande dès que l'on en place trois c'est la cata... Voici ce que j'obtiens en sortie du programme :

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
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
alloue params : 0x603010
alloue cmd : 0x603030
MiKaShEll>> ls -l
ls -l
params[1] alloué 0x6030b0
params[1]:      -l
params[2]:      (null)
7982    7984
exécute : 7984
total 44
-rwxr-xr-x 1 mik users 16625 nov. 20 01:51 miksh
-rw-r--r-- 1 mik users  2179 nov. 20 01:51 miksh.c
-rwxr-xr-x 1 mik users 12142 sept. 12 23:21 p109
-rw-r--r-- 1 mik users   842 nov. 12 17:31 p109.c
-rw-r--r-- 1 mik users   481 nov.  3 19:04 te.c
libère 0x603070 : ls
libéré
libère 0x6030b0 : -l
libéré
libère NULL : (nil)
libéré
Libère cmd : 0x603030
libéré
libère params : 0x603010
libéré
alloue params : 0x603010
alloue cmd : 0x603030
MiKaShEll>> cp p109.c kk
cp p109.c kk
params[1] alloué 0x603070
params[1]:      p109.c
params[2] alloué 0x603130
params[2]:      kk
params[3]:      (null)
7982    7986
exécute : 7986
libère 0x6030b0 : cp
libéré
libère 0x603070 : p109.c
libéré
libère 0x603130 : kk
libéré
libère NULL : (nil)
libéré
Libère cmd : 0x603030
*** glibc detected *** ./miksh: free(): invalid pointer: 0x0000000000603030 ***
======= Backtrace: =========
etc ...

A l'aide !

[gerald3d]

Puisque tu limites la longueur de "cmd" tu peux très bien t'éviter une allocation dynamique. Je ferais ce genre de chose:

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
int
main(int argc, char *argv[])
{
  int statut;
  int i=0;
  pid_t child_pid;
  char cmd[MAX_SAIS];  // Déclaration en locale d un tableau de char.

  while (TRUE)
  {
    params=(char**)calloc(10, sizeof(char));
    printf("alloue params : %p\n",params);

    // nettoyage de cmd
    for (i=0;i<MAX_SAIS; i++) cmd[i]=' ';

//    cmd=(char*)calloc(MAX_SAIS, sizeof(char));  // Allocation inutile
//    printf("alloue cmd : %p\n",cmd);
    prompt();
    lecture(cmd,params);  // La transmission du tableau se fait toujours par le pointeur du premier élément.
...

Tu supprimes le free(cmd); devenu lui aussi inutile et le tour et joué .

[PyNub]

Oui je peux essayer, c'est ce que j'ai fait en premier... Mais je ne pense pas que ça résolve mon problème. Le fait d'allouer dynamiquement me permet par la suite de ne pas limiter justement la saisie. Ce que je ne comprends pas c'est que ça marche très bien avec une commande et un argument alors que deux arguments me déclenche une erreur.

Merci

[PyNub]

C'est bien ce que je pensais ça ne marche pas...
Le code modifié :

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
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
104
105
106
107
108
109
#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <sys/wait.h>
#define MAX_SAIS 50
#define TRUE 1
 
void prompt();
void lecture(char * cmd,char ** params);
 
char cmd[MAX_SAIS];
char ** params;
 
void clean_stdin(void){
	int c;
	do{
		c=getchar();
	} while(c!='\n' && c!=EOF);
}
 
int main(){
	int statut;
	int i=0;
	pid_t child_pid;
	while (TRUE){
		params=(char**)malloc(10);
		printf("alloue params : %p\n",params);
//		cmd=(char*)malloc(MAX_SAIS);
//		printf("alloue cmd : %p\n",cmd);
		prompt();
		lecture(cmd,params);
		child_pid=fork();
		if (child_pid !=0){
			printf("%d\t%d\n",getpid(),child_pid);
			waitpid(-1,&statut,0);
		}
		else {
			printf("exécute : %d\n",getpid());
			execvp(cmd,params);
			printf("%s\n",cmd);
			i=-1;
			do {
				i++;
				printf("%s\t",params[i]);
			}while(params[i]!=NULL);
			printf("\n");
		}
		i=0;
		do{	
			printf("libère %p : %s\n",*(params+i),*(params+i));
			free(*(params+i));
			printf("libéré\n");
			i++;
		}while(*(params+i)!=NULL);
		printf("libère NULL : %p\n",*(params+i));
		free(*(params+i));
		printf("libéré\n");
//		printf("Libère cmd : %p\n",cmd);
//		free(cmd);
//		printf("libéré\n");
		printf("libère params : %p\n",params);
		free(params);
		printf("libéré\n");
		i=0;
		clean_stdin();
	}
}
 
void prompt(){
	printf("MiKaShEll>> ");
}
 
void lecture(char * cmd, char ** params){
	int i=0;
	int j=0;
	int k=0;
	char tmp[MAX_SAIS];
	scanf("%[^\n]",tmp);
	printf("%s\n",tmp);
	*(params+j)=(char*)malloc(MAX_SAIS);
	while((tmp[i]!='\0')&&(tmp[i]!='\n')&&(tmp[i]!=' ')&&(i<MAX_SAIS)){
		*(cmd+i)=*(tmp+i);
		*(*(params+j)+i)=*(tmp+i);
		i++;
	}
	*(cmd+i)='\0';
	*(*(params+j)+i)='\0';
	k=0;
	j++;
	if (*(tmp+i)==' ') i++;
	while((tmp[i]!='\0')&&(tmp[i]!='\n')&&(i<MAX_SAIS)){
		*(params+j)=(char*)malloc(MAX_SAIS);
		printf("params[%d] alloué %p\n",j,*(params+j));
		while((tmp[i]!='\0')&&(tmp[i]!='\n')&&(tmp[i]!=' ')&&(i<MAX_SAIS)){
			*(*(params+j)+k)=*(tmp+i);
			i++;
			k++;
		}
		*(*(params+j)+i)='\0';
		printf("params[%d]:\t%s\n",j,*(params+j));
		if (tmp[i]=='\n') break;
		i++;
		j++;
		k=0;
	}
	*(params+j)=(char*)malloc(MAX_SAIS);
	*(params+j)=NULL;
	printf("params[%d]:\t%s\n",j,*(params+j));
}

La sortie toujours pareil :

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
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
mik@linux-8ufy:/home/dev/system/Tanenbaun> ./miksh
alloue params : 0x603010
MiKaShEll>> ls -l
ls -l
params[1] alloué 0x603070
params[1]:      -l
params[2]:      (null)
8435    8436
exécute : 8436
total 44
-rwxr-xr-x 1 mik users 16625 nov. 20 22:44 miksh
-rw-r--r-- 1 mik users  2197 nov. 20 22:43 miksh.c
-rwxr-xr-x 1 mik users 12142 sept. 12 23:21 p109
-rw-r--r-- 1 mik users   842 nov. 12 17:31 p109.c
-rw-r--r-- 1 mik users   481 nov.  3 19:04 te.c
libère 0x603030 : ls
libéré
libère 0x603070 : -l
libéré
libère NULL : (nil)
libéré
libère params : 0x603010
libéré
alloue params : 0x603010
MiKaShEll>> cp p109.c gg
cp p109.c gg
params[1] alloué 0x603030
params[1]:      p109.c
params[2] alloué 0x6030f0
params[2]:      gg
params[3]:      (null)
8435    8439
exécute : 8439
libère 0x603070 : cp
libéré
libère 0x603030 : p109.c
*** glibc detected *** ./miksh: free(): invalid pointer: 0x0000000000603030 ***
======= Backtrace: =========
/lib64/libc.so.6(+0x73286)[0x7f34b2b1e286]
/lib64/libc.so.6(cfree+0x6c)[0x7f34b2b2302c]
./miksh[0x400a12]
/lib64/libc.so.6(__libc_start_main+0xfd)[0x7f34b2ac9b7d]
./miksh[0x400799]
======= Memory map: ========
00400000-00402000 r-xp 00000000 08:07 392687                             /home/dev/system/Tanenbaun/miksh
00601000-00602000 r--p 00001000 08:07 392687                             /home/dev/system/Tanenbaun/miksh
00602000-00603000 rw-p 00002000 08:07 392687                             /home/dev/system/Tanenbaun/miksh
00603000-00624000 rw-p 00000000 00:00 0                                  [heap]
7f34ac000000-7f34ac021000 rw-p 00000000 00:00 0 
7f34ac021000-7f34b0000000 ---p 00000000 00:00 0 
7f34b2895000-7f34b28aa000 r-xp 00000000 08:06 138195                     /lib64/libgcc_s.so.1
7f34b28aa000-7f34b2aa9000 ---p 00015000 08:06 138195                     /lib64/libgcc_s.so.1
7f34b2aa9000-7f34b2aaa000 r--p 00014000 08:06 138195                     /lib64/libgcc_s.so.1
7f34b2aaa000-7f34b2aab000 rw-p 00015000 08:06 138195                     /lib64/libgcc_s.so.1
7f34b2aab000-7f34b2c02000 r-xp 00000000 08:06 137372                     /lib64/libc-2.11.2.so
7f34b2c02000-7f34b2e01000 ---p 00157000 08:06 137372                     /lib64/libc-2.11.2.so
7f34b2e01000-7f34b2e05000 r--p 00156000 08:06 137372                     /lib64/libc-2.11.2.so
7f34b2e05000-7f34b2e06000 rw-p 0015a000 08:06 137372                     /lib64/libc-2.11.2.so
7f34b2e06000-7f34b2e0b000 rw-p 00000000 00:00 0 
7f34b2e0b000-7f34b2e2a000 r-xp 00000000 08:06 132781                     /lib64/ld-2.11.2.so
7f34b3003000-7f34b3006000 rw-p 00000000 00:00 0 
7f34b3026000-7f34b3029000 rw-p 00000000 00:00 0 
7f34b3029000-7f34b302a000 r--p 0001e000 08:06 132781                     /lib64/ld-2.11.2.so
7f34b302a000-7f34b302b000 rw-p 0001f000 08:06 132781                     /lib64/ld-2.11.2.so
7f34b302b000-7f34b302c000 rw-p 00000000 00:00 0 
7fff1f89c000-7fff1f8bd000 rw-p 00000000 00:00 0                          [stack]
7fff1f9f9000-7fff1f9fa000 r-xp 00000000 00:00 0                          [vdso]
ffffffffff600000-ffffffffff601000 r-xp 00000000 00:00 0                  [vsyscall]
Abandon

Merci qd même...

[PyNub]

tu peux très bien t'éviter une allocation dynamique.

C'est vrai que qd on sait pas les utiliser...

Code :

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

params=(char**)malloc(10);

Franchement je me demande ou je suis allé chercher ça...

Voilà le code rectifié avec l'utilisation correcte de malloc...

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
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
104
105
106
107
108
109
110
111
112
113
114
#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <sys/wait.h>
#define MAX_SAIS 50
#define TRUE 1
 
void prompt();
int lecture(char * cmd,char ** params);
 
char * cmd;
char ** params;
 
void clean_stdin(void){
	int c;
	do{
		c=getchar();
	} while(c!='\n' && c!=EOF);
}
 
int main(){
	int statut;
	int i=0;
	int nbp;
	pid_t child_pid;
	while (TRUE){
		params=malloc(10*sizeof(char*));
//		printf("alloue params : %p\n",params);
		cmd=malloc(MAX_SAIS*sizeof(char));
//		printf("alloue cmd : %p\n",cmd);
		prompt();
		nbp=lecture(cmd,params);
		child_pid=fork();
		if (child_pid !=0){
			printf("%d\t%d\n",getpid(),child_pid);
			waitpid(-1,&statut,0);
		}
		else {
			printf("exécute : %d\n",getpid());
			execvp(cmd,params);
			printf("%s n'est pas un programme\n",cmd);
			i=-1;
			do {
				i++;
//				printf("%s\t",params[i]);
			}while(params[i]!=NULL);
//			printf("\n");
		}
		i=nbp;
		do{	
//			printf("libère %p : %s\n",*(params+i),*(params+i));
			free(*(params+i));
//			printf("libéré param %d\n",i);
			i--;
		}while(i>=0);
//		printf("libère NULL : %p\n",*(params+i));
//		free(*(params+i));
//		printf("libéré\n");
//		printf("Libère cmd : %p\n",cmd);
		free(cmd);
//		printf("libéré\n");
//		printf("libère params : %p\n",params);
		free(params);
//		printf("libéré\n");
		i=0;
		clean_stdin();
	}
}
 
void prompt(){
	printf("MiKaShEll>> ");
}
 
int lecture(char * cmd, char ** params){
	int i=0;
	int j=0;
	int k=0;
	char tmp[MAX_SAIS];
	scanf("%[^\n]",tmp);
//	printf("%s\n",tmp);
	*(params+j)=malloc(MAX_SAIS*sizeof(char));
//	printf("params[%d] alloué %p\n",j,*(params+j));
	while((tmp[i]!='\0')&&(tmp[i]!='\n')&&(tmp[i]!=' ')&&(i<MAX_SAIS)){
		*(cmd+i)=*(tmp+i);
		*(*(params+j)+i)=*(tmp+i);
		i++;
	}
	*(cmd+i)='\0';
	*(*(params+j)+i)='\0';
//	printf("params[%d]:\t%s\n",j,*(params+j));
	k=0;
	j++;
	if (*(tmp+i)==' ') i++;
	while((tmp[i]!='\0')&&(tmp[i]!='\n')&&(i<MAX_SAIS)){
		*(params+j)=malloc(MAX_SAIS*sizeof(char));
//		printf("params[%d] alloué %p\n",j,*(params+j));
		while((tmp[i]!='\0')&&(tmp[i]!='\n')&&(tmp[i]!=' ')&&(i<MAX_SAIS)){
			*(*(params+j)+k)=*(tmp+i);
//			printf("%d\t%c\t%c\n",k,*(tmp+i),*(*(params+j)+k));
			i++;
			k++;
		}
		*(*(params+j)+k)='\0';
//		printf("params[%d]:\t%s\n",j,*(params+j));
		if (tmp[i]=='\n') break;
		i++;
		j++;
		k=0;
	}
	*(params+j)=malloc(MAX_SAIS*sizeof(char));
	*(params+j)=NULL;
//	printf("params[%d]:\t%s\n",j,*(params+j));
	return (j);
}

Merci pour ton aide