shithub: gefs

Info  •  Files  •  Log  •  Branches

fs: deref uid/gid after nil check

--- a/fs.c

+++ b/fs.c

@@ -800,10 +800,10 @@

 	in = 0;

 	u = uid2user(uid);

 	g = uid2user(gid);

-	if(u->id == g->id)

-		in = 1;

 	if(u != nil && g != nil)

-		for(i = 0; i < g->nmemb; i++)

+		if(u->id == g->id)

+			in = 1;

+		else for(i = 0; i < g->nmemb; i++)

 			if(u->id == g->memb[i])

 				in = 1;

 	runlock(&fs->userlk);