xfnw/plan9fox
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

authsrv: don't hash in hostowner key for keyseed

Browse source 
aiju → i don't like it, it's more bullshit ways to expose the key :)
aiju → if someone can grab /adm/keyseed, they can also grab /adm/users and /adm/keys
This commit is contained in:
cinap_lenrek 2017-02-26 22:44:47 +01:00
parent da343924f4
commit 6de804b578
1 changed files with 0 additions and 18 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

18
sys/src/cmd/auth/authsrv.c
View file

@ -1005,36 +1005,18 @@ getraddr(char *dir)
void
initkeyseed(void)
{
static char info[] = "PRF key for generation of dummy user keys";
char k[DESKEYLEN], *u;
int fd;
genrandom(keyseed, sizeof(keyseed));
u = getuser();
if(!finddeskey(KEYDB, u, k)){
syslog(0, AUTHLOG, "initkeyseed: user %s not in keydb", u);
return;
}
if((fd = create("/adm/keyseed", OWRITE|OEXCL, 0600)) >= 0){
write(fd, keyseed, sizeof(keyseed));
} else if((fd = open("/adm/keyseed", OREAD)) >= 0){
read(fd, keyseed, sizeof(keyseed));
} else{
syslog(0, AUTHLOG, "initkeyseed: no seed file: %r");
memset(k, 0, sizeof(k));
return;
}
close(fd);
hkdf_x( keyseed, sizeof(keyseed),
(uchar*)info, sizeof(info)-1,
(uchar*)k, sizeof(k),
keyseed, sizeof(keyseed),
hmac_sha2_256, SHA2_256dlen);
memset(k, 0, sizeof(k));
}
void