search

42shpimanmls / 42sh

Shell project at 42
3 stars 0 forks source link

Permissions with mask #112

Closed pilespin closed 7 years ago

n0izn0iz commented 7 years ago

Il faut pas juste retirer le dernier argument d'open ??

Si le umask change, les permissions doivent changer ! La c'est fixé a la compilation...

Voir le comportement d'sh:

sh-4.4$ umask -S
u=rwx,g=rx,o=rx
sh-4.4$ echo bli > file1
sh-4.4$ umask -S u=rwx,g=,o=
u=rwx,g=,o=
sh-4.4$ echo bli > file2
sh-4.4$ ls -l
total 8
-rw-r--r-- 1 n0iz n0iz 4 Apr  8 16:59 file1
-rw------- 1 n0iz n0iz 4 Apr  8 17:00 file2
sh-4.4$
n0izn0iz commented 7 years ago

Wtf, pourquoi ya pas besoin de review pour merge ??

pilespin commented 7 years ago

de branche a branche

pilespin commented 7 years ago

ACCESS GRANTED

lsimonne commented 7 years ago

non, t'es obligé d'avoir les flags de permission dans open, si tu fais un umask ça rajoute juste une couche de masque par dessus ce que tu passes à open en fait je crois

n0izn0iz commented 7 years ago

Ah ok j'ai compris ^^

lsimonne commented 7 years ago

je peux merge pour modifier les errors etc? sinon ça va faire des conflits après ˆˆ

pilespin commented 7 years ago

merge