[PATCH] Variable in sdap_sudo_rules_refresh_send could be used, uninitialised.
by Michal Židek
Another one liner. If no rules were passed to
sdap_sudo_rules_refresh_send (if rules[0] was NULL) and no error
occurred, then variable ret was used uninitialized. This is not very
likely to happen, but it is safer to have it initialized.
Maybe we should consider adding initialization of all "ret" variables to
our coding guidelines to avoid these "unlikely but possible" errors in
new code.
Patch attached.
Michal
11 years, 6 months
[PATCHES] sss_seed related patches
by Michal Židek
Patch 01: The debug_level variable was set before the parameters were
processed, so the default debug level was used no matter what number was
specified with --debug.
Patch 02: When interactive input fails, error message should be displayed.
https://fedorahosted.org/sssd/ticket/1549
Patch 03: When file is used to specify a password in sss_seed, then only
first line of this file is used. Also empty passwords are treated as
errors. (My local test environment is broken so I have not tested this,
please test it when reviewing.)
https://fedorahosted.org/sssd/ticket/1548
Patches are in attachment.
Michal
11 years, 6 months
OpenSSH-LPK support
by Jan Cholasta
Hi,
now that SSSD supports OpenSSH formatted public keys, the OpenSSH-LPK
<http://code.google.com/p/openssh-lpk/> schema is automatically
supported as well in the LDAP provider. All you have to do is to set the
ldap_user_ssh_public_key option in your LDAP domain section of sssd.conf
to sshPublicKey.
Currently there is no default value for ldap_user_ssh_public_key in LDAP
domains. Does it make sense to make sshPublicKey the default value for it?
Honza
--
Jan Cholasta
11 years, 6 months