On Wed, Feb 08, 2017 at 11:58:32PM -0500, Selva Nair wrote:
> Hi,
> 
> On Wed, Feb 8, 2017 at 10:01 PM, Antonio Quartulli <a...@unstable.cc> wrote:
> 
> > On Wed, Feb 08, 2017 at 02:25:44PM -0500, selva.n...@gmail.com wrote:
> > > From: Selva Nair <selva.n...@gmail.com>
> > >
> > > - Keep the username even if auth-nocache is specified so that
> > >   any auth_token pushed by the server could be utilized
> >
> > This means that even when using no auth-token the username will be cached.
> > Can this be a security concern?
> >
> 
> I would consider username as not sensitive  material although not sure
> everyone would agree. Unfortunately there is no way to know in advance that
> auth-token may get pushed so I can't think of a good way of avoiding this.
> A not so secure approach (I considered this first) would be to delay
> clearing the username/password to post pushed-options processing, but then
> one has to handle cases like what if the push reply never arrives and so
> on.. In general its always better to clear sensitive data at the earliest.
> 
> The way out would be to do one more purge_user_pass(.., false) after push
> processing.. sigh..  will go there only if absolutely necessary.

Yesterday, while discussing the same issue on IRC, I came up with this patch:

http://bpaste.net/show/153e8d51c02d

It does indeed wait for the push-reply to come back before making a decision
about wiping the user_pass object or not.
This way also the user is wiped when nocache remains true.

Not sure which approach is better though.

Cheers,

-- 
Antonio Quartulli

Attachment: signature.asc
Description: Digital signature

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, SlashDot.org! http://sdm.link/slashdot
_______________________________________________
Openvpn-devel mailing list
Openvpn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openvpn-devel

Reply via email to