help-shishi
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: shisa API for setting password in plaintext


From: Elrond
Subject: Re: shisa API for setting password in plaintext
Date: Thu, 23 Mar 2006 12:16:53 +0100
User-agent: Mutt/1.5.9i

On Thu, Mar 23, 2006 at 11:51:38AM +0100, Simon Josefsson wrote:
[...]
> Oops, agreed.
> 
> > I don't know, what the right way out of this is.
> > Maybe Shishi_key should know the salt, maybe
> > shishi_key_from_name should output the salt. I don't know,
> > really.
> 
> It seems better to separate this, I've created:
> 
> int
> shishi_derive_default_salt (Shishi * handle,
>                           const char *name,
>                           char **salt)
> 
> Which shishi_key_from_name and shisa.c now use.
> 
> As a result, shisa.c doesn't use shishi_key_from_name any more, but
> that API may be useful anyway, so I'll keep it.
> 
> What do you think?

Looks mostly good. :)

shishi_derive_default_salt() should free(principal);
free(realm); at the end. :-)

shisa.c hasn't yet made it into viewcvs, will look at it
later then.


    Elrond




reply via email to

[Prev in Thread] Current Thread [Next in Thread]