Comment 15 for bug 259293

Revision history for this message
Dustin Kirkland  (kirkland) wrote :

Patch attached.

I've tested this about as much as I can, throwing a number of cases at it, and it seems to do the right thing for me. It seems to handle cronjobs, multiple console logins, and multiple ssh logins properly.

I'd like to get one of our security guys to review this (jdstrand and/or kees), as well as one of our PAM guys (slangasek and/or pitti). I'll also try to get some feedback from the other ecryptfs upstream maintainers (mhalcrow and/or tyhicks).

Note that the new code does **not** run within the setuid portion of the code, so it should be low risk from a security perspective.

I'll also put a package in my PPA, any testing would be **much** appreciated.

:-Dustin