diff options
| author | Simo Sorce <simo@redhat.com> | 2013-08-31 14:21:22 -0400 |
|---|---|---|
| committer | Simo Sorce <simo@redhat.com> | 2013-09-05 09:09:42 -0400 |
| commit | 150050d68d33aa337e4596e4b471064c7e3f9271 (patch) | |
| tree | 957c96f45fe4dbb82fec4c85206fc5d0ff5febb2 /src/python | |
| parent | 7618870571b3f74c97c89d8ae1114ff8b7b4879d (diff) | |
| download | sssd-150050d68d33aa337e4596e4b471064c7e3f9271.tar.gz sssd-150050d68d33aa337e4596e4b471064c7e3f9271.tar.xz sssd-150050d68d33aa337e4596e4b471064c7e3f9271.zip | |
krb5: Add file/dir path precheck
Add a precheck on the actual existence at all of the file/dir ccname
targeted (for FILE/DIR types), and bail early if nothing is available.
While testing I found out that without this check, the krb5_cc_resolve()
function we call as user to check old paths would try to create the
directory if it didn't exist.
With a ccname of DIR:/tmp/ccdir_1000 saved in the user entry this would
cause two undesirable side effects:
First it would actually create a directory with the old name, when it
should not.
Second, because for some reason the umask is set to 0127 in sssd_be, it
would create the directory with permission 600 (missing the 'x' traverse
bit on the directory. If the new ccache has the same name it would cause
the krb5_child process to fal to store the credential cache in it.
Related:
https://fedorahosted.org/sssd/ticket/2061
Diffstat (limited to 'src/python')
0 files changed, 0 insertions, 0 deletions
