Re: [9front] new user

[Alex Musolino <alex@musolino.id.au>]

> I created a new user on another system. Its running hjfs
> 
> I noticed permission had sys?
> 
> cpu% ls -l
> d-rwxrwxr-x M 184 glenda glenda 0 Feb 26 14:46 glenda
> d-rwxrwxr-x M 184 www    sys    0 Feb 26 16:08 www
> 
> This does not seem right?

This is a bug in hjfs that has been uncovered by changeset
e1907b41d32441e79e8cc3db26afc5f0c4cdcef9.  New directories *are*
supposed to inherit their group from the parent directory.  However,
this is not what we want when creating home directories for new users.

I think the following ought to fix it.  Can you give this a try?

diff facb0e757ac63f763bd942a2714f979538b99eb0 uncommitted
--- a/sys/src/cmd/hjfs/auth.c
+++ b/sys/src/cmd/hjfs/auth.c
@@ -366,13 +366,18 @@
 createuserdir(Fs *fs, char *name, short uid)
 {
 	Chan *ch;
+	Dir di;
 
 	ch = chanattach(fs, CHFNOPERM);
 	if(ch == nil)
 		return;
 	ch->uid = uid;
-	if(chanwalk(ch, "usr") > 0)
+	if(chanwalk(ch, "usr") > 0){
 		chancreat(ch, name, DMDIR | 0775, OREAD);
+		nulldir(&di);
+		di.gid = name;
+		chanwstat(ch, &di);
+	}
 	chanclunk(ch);
 }