Discussion :

[AntoineCompagnie]

Bonjour!

Je cherche à savoir comment recréer la commande > en unix.

C'est dans le cas d'une fonction commande appelée par une main qui exécute les commandes en fonction des délimiteurs "&;<>|".
Ces délimiteurs sont différenciés par une fonction parsing qui renvoit un chiffre en fonction du délimiteur et remplit un tableau avec la commande

Nous traitons ici le cas 4, la redirection >

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
	case 4: // >
		close(1); // on ferme l'écran
		int fichierSortie = open(respP[0],O_CREAT|O_RDWR|O_TRUNC,0,640);// pourquoi on ouvre le tableau?
 
	if(pid>0){
 
		        close(1);
			printf("Dauphine>");
			fflush(stdout);
 
		} else {
			close(1);// on ferme l'écran
			dup(fichierSortie);
			execvp(respP[0],respP);
		}
		break;
	}

Pourquoi doit on fermer deux fois l'écran (la sortie standard) Ca me fait une erreur sinon...

[Médinoc]

Utilise plutôt dup2() que dup(), c'est moins hasardeux que de fermer puis réouvrir en priant pour que tous les descripteurs précédents soient pris...

De plus, plutôt que d'utiliser des nombres magiques pour test descripteurs, tu devrais utiliser les constantes POSIX: STDIN_FILENO, STDOUT_FILENO...

[Sve@r]

Code :

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

1
2
	case 4: // >
		int fichierSortie = open(respP[0],O_CREAT|O_RDWR|O_TRUNC,0,640);// pourquoi on ouvre le tableau?

Pourquoi doit on fermer deux fois l'écran (la sortie standard) Ca me fait une erreur sinon...

Bonjour

J'aime bien ce genre de code. On sent de suite que tu sais ce que tu fais et pourquoi tu le fais (style je mets de tout en espérant que ça fasse un truc). Donc tu "n'ouvres pas le tableau", tu ouvres le fichier dont le nom est stocké dans la case 0 de ton tableau !!!
Toutefois, la fonction open() ne prend que 3 paramètres or moi j'en compte 4 (resp[0], O_CREAT|O_RDWR|O_TRUNC, 0 (???) et 640). Là encore on sent que tu sais à quoi servent ces valeurs et ce qu'elles veulent dire...

Et sinon tu dois fermer deux fois l'écran parce que tu as probablement un peu plus haut pid=fork() (super ces programmes incomplets qu'il faut deviner !!!) et que tu as donc deux processus. Donc en fait tu ne fermes pas 2 fois l'écran, c'est chaque processus qui ferme une fois son écran perso (le processus fils a hérité des descripteurs de son père dont l'écran !!!)
Evidemment tu aurais fermé l'écran avant le fork(), tu ne l'aurais écrit l'instruction qu'une seule fois...

[Médinoc]

Des points bonus pour le 640 auquel il manque le préfixe pour préciser qu'il est exprimé en octal... et de toute façon il est passé en 4e paramètre de open() qui n'en attend que trois ou plus, le fichier est donc créé sans aucun droit pour personne.

[BufferBob]

Des points bonus pour le 640 auquel il manque le préfixe pour préciser qu'il est exprimé en octal... et de toute façon il est passé en 4e paramètre de open() qui n'en attend que trois ou plus, le fichier est donc créé sans aucun droit pour personne.

voire même ça compile pas ^^

en fait je pense qu'y a une virgule qui s'est glissée entre le 0 et le 640, typo malheureuse

[Médinoc]

Je pense que ça compile, parce que pour des raisons historiques la fonction open() tend à être déclarée variadique: int open(const char *path, int oflag, ... );.

À l'époque, quand on disait qu'un paramètre était optionnel (enfin pour être précis, il n'est pris en compte que quand O_CREAT est spécifié), on ne plaisantait pas.

[BufferBob]

Je pense que ça compile, parce que pour des raisons historiques la fonction open() tend à être déclarée variadique: int open(const char *path, int oflag, ... );.

c'est vrai qu'il y a un wrapper libc pour open, on atterit pas directement sur le syscall je m'en souvenais plus du tout, bien vu ^^

[AntoineCompagnie]

Oui, merci beaucoup, c'était bien une typo stupide!
Merci pour votre analyse!