summaryrefslogtreecommitdiff
AgeCommit message (Collapse)AuthorFilesLines
2013-07-24Every time release allocated memory in function py_sss_getgrouplistLukas Slebodnik1-0/+2
Coverity: 11922
2013-07-24Lower timeout to contact DNS serverMichal Zidek1-1/+1
c-ares timeout to wait for response from DNS server before moving to next DNS server is lowered from 5s to 2s. Partially solves https://fedorahosted.org/sssd/ticket/1966
2013-07-24Set default DNS resolution timeout to 6 seconds.Michal Zidek2-3/+3
Partially solves ticket: https://fedorahosted.org/sssd/ticket/1966 To avoid the problem mentioned in the ticket above, option dns_discovery_domain must be set properly.
2013-07-24LDAP: Use domain-specific name where appropriateJakub Hrozek5-112/+159
The subdomain users user FQDN in their name attribute. However, handling of whether to use FQDN in the LDAP code was not really good. This patch introduces a utility function and converts code that was relying on user/group names matching to this utility function. This is a temporary fix until we can refactor the sysdb API in #2011.
2013-07-23Fix possible dereference of a NULL pointer.Lukas Slebodnik1-1/+1
We check if function ipa_get_ad_id_ctx returns NULL, but function ipa_get_ad_id_ctx could never return NULL. This issue was found by scan-build.
2013-07-23pysss: prevent crashing when group is unresolvableAlexander Bokovoy1-2/+13
In unlikely case that an NSS module returns a reference to a group and we are unable to resolve it shortly after that, make sure these groups are skipped.
2013-07-23Remove unused constant.Michal Zidek1-2/+0
2013-07-23KRB5: Do not send PAC in server modeJakub Hrozek5-7/+27
The krb5 child contacts the PAC responder for any user except for the IPA native users if the PAC is configured. This works fine for the general case but the ipa_server_mode is a special one. The PAC responder is there, but since in the server mode we should be operating as AD provider default, the PAC shouldn't be analyzed either in this case.
2013-07-22SUDO: realloc with sizeof(uint32_t) when adding uint32_tJakub Hrozek1-1/+1
2013-07-22pysss: add pysss.getgrouplist(username)Alexander Bokovoy1-0/+59
getgrouplist(3) call is missing from Python older than Python 3.3 Introduce supplementary binding to provide getgrouplist as part of pysss interface. Since getgrouplist() can be run against any domain, place it at top module rather than pysss.local namespace. pysss.getgrouplist(username) -> tuple(group list as strings)
2013-07-22build: fix dependencies for pysss moduleAlexander Bokovoy1-0/+1
https://fedorahosted.org/sssd/ticket/2025
2013-07-22Fix autotols warnings: macro xyz not found in libraryLukas Slebodnik1-4/+2
2013-07-22Fix warnings: uninitialized variableLukas Slebodnik4-6/+5
2013-07-22Remove unused memory context from function unpack_authtokLukas Slebodnik1-3/+3
2013-07-19Fix the default FQDN formatJakub Hrozek2-3/+3
Commit 52ae806bd17c3c00d70bd1aed437f10f5ae51a1c changed the default FQDN format by accident to the one we only ever user internally. This commit fixes the mistake.
2013-07-19TOOLS: Update all services with sss_debuglevelOndrej Kos1-2/+7
https://fedorahosted.org/sssd/ticket/2007 PAC, SSH, SUDO and AUTOFS services' debug levels weren't modified when sss_debuglevel tool was used.
2013-07-19AD: Set the bool value same as default value in optsJakub Hrozek3-4/+74
https://fedorahosted.org/sssd/ticket/2023 When the option values are copied using dp_opt_copy_map, the .val member is used if it's not NULL. At the same time, the bool options are never NULL, unlike integers or strings that can have special NULL-like values such as NULL_STRING. This effectively means that when copying a bool option, the .val member is always used. But in the AD maps, some .val fields were set differently from the .def_val fields. The effect was that when the AD subdomain provider was initialized from IPA subdomain provider using only the defaults, some options (notably referral chasing) were set to a value that didn't make sense for the AD provider. This patch makes sure that for all boolean option, the .val is always the same as .def_val.
2013-07-19IPA: warn if full_name_format is customized in server modeJakub Hrozek3-1/+20
https://fedorahosted.org/sssd/ticket/2009 If the IPA server mode is on and the SSSD is running on the IPA server, then the server's extdom plugin calls getpwnam_r to read info about trusted users from the AD server and return them to the clients that called the extended operation. The SSSD returns the subdomain users fully-qualified, ie "user@domain" by default. The format of the fully qualified name is configurable. However, the extdom plugin returns the user name without the domain component. With this patch, when ipa_server_mode is on, warn if the full_name_format is set to a non-default value. That would prompt the admin to change the format if he changed it to something exotic.
2013-07-19Add mising argument required by format stringLukas Slebodnik2-3/+4
2013-07-19Use functionm ldb_dn_get_linearized to format struct ldb_dnLukas Slebodnik1-3/+6
2013-07-19Fix clang format string warning.Lukas Slebodnik2-2/+2
warning: format string is not a string literal (potentially insecure) [-Wformat-security]
2013-07-18Remove unused be_ctx->sigchld_ctxJakub Hrozek2-9/+0
2013-07-18KRB: Replace multiple calls with variableOndrej Kos1-6/+9
Instead of multiple calls of sss_authtok_get_type, perform the call just once and store into variable.
2013-07-18KRB: Handle empty password gracefullyOndrej Kos1-0/+11
https://fedorahosted.org/sssd/ticket/1814 Return authentication error when empty password is passed.
2013-07-18Do not try to set password when authtok_length is zeroOndrej Kos1-2/+6
https://fedorahosted.org/sssd/ticket/1814 When the authtok_length is zero, it shouldn't call sss_authtok_set_password, because it tries to determine lenght of passed string by itself and would read parts of DBus message behind boundaries of authtok.
2013-07-17MAN: Clarify the min_id/max_id limits furtherJakub Hrozek1-0/+4
https://fedorahosted.org/sssd/ticket/2005 Some users were confused by our description of min_id/max_id and thought the limits only applied to returning entries from the NSS responder. However, the limits are actually enforced on the back end side, so the entries are not even saved to cache.
2013-07-17MAN: IP addresss does not work when used for ad_serverJakub Hrozek1-1/+1
https://fedorahosted.org/sssd/ticket/1998 Currently using IP address as value of ad_server is not supported, so the man pages should not mention that as an option.
2013-07-17MAN: clarify the default access provider for ADJakub Hrozek1-0/+5
https://fedorahosted.org/sssd/ticket/1965 After we added a section that clarified what access_provider=ad did, some users were confused and thought that "ad" was also the default access provider if "id_provider=ad" was specified.
2013-07-17SIGCHLD handler: do not call callback when pvt data where freedPavel Březina5-5/+37
https://fedorahosted.org/sssd/ticket/1992
2013-07-17dyndns timeout test: catch SIGCHLD handler eventsPavel Březina1-0/+12
https://fedorahosted.org/sssd/ticket/1992 dyndns test will crash with this patch
2013-07-17RPM: Require libsss_idmap from sssd-commonJakub Hrozek1-3/+1
The NSS responder recently started using libsss_idmap in the getbysid functions. The bug itself was spotted by one of our automated QA tools.
2013-07-17Every time use permissive control in function memberof_mod.Lukas Slebodnik1-0/+7
Storing cyclic groups into sysdb can cause adding ghost members, which has already been stored. Function ldb_modify will fail with error [Attribute or value exists]. With permisive control, duplicated attributes will be skipped as if it was never added. https://fedorahosted.org/sssd/ticket/1846
2013-07-17print hint about password complexity when new password is rejectedPavel Březina2-1/+15
https://fedorahosted.org/sssd/ticket/1827
2013-07-17LDAP: When resolving a SID, search for groups first, then usersJakub Hrozek1-19/+18
https://fedorahosted.org/sssd/ticket/1997 Most of the time, the SIDs are resolved as a call coming from the PAC responder during initgroups. In that case at least, it makes sense to search for group matching that SID first, then users. We may consider making this behaviour configurable ie for the server mode where typically the users should be queried first.
2013-07-16remove unused variablePavel Březina1-4/+0
2013-07-16Remove unused function parameterLukas Slebodnik1-3/+1
2013-07-15Use conditional build for retrieving ccache.Lukas Slebodnik4-88/+78
Some krb5 functions needn't be available for retrieving ccache with principal. Therefore ifdef is used to solve this situation with older version of libkrb5. There were two functions with similar functionality in krb5_child and krb5_utils. They were merged to one universal function, which was moved to file src/util/sss_krb5.c
2013-07-15Missing space in debug messageMichal Zidek1-2/+2
2013-07-11Return right directory name for dircacheLukas Slebodnik1-6/+10
Dircache can be either file or directory. Wrong value was used when dircache was itself directory. https://fedorahosted.org/sssd/ticket/2002
2013-07-11Always set port status to neutral when resetting service.Michal Zidek1-1/+2
We did not set port status for metaservers (srv servers) in fo_reset_services(). Fixes: https://fedorahosted.org/sssd/ticket/1933
2013-07-10KRB5_CHILD: Fix handling of get_password return codeOndrej Kos1-4/+10
The switch statement was dead code due to missing case/default.
2013-07-10sss_cache: Add option to invalidate all entriesMichal Zidek2-0/+24
Option -E/--everething was added to invalide all types of entries. https://fedorahosted.org/sssd/ticket/1988
2013-07-09Do not copy special files when creating homedirOndrej Kos3-73/+10
https://fedorahosted.org/sssd/ticket/1778 When trying to copy special file, only message is logged now.
2013-07-09DB: sysdb_search_user_by_name: search by both name and aliasJakub Hrozek2-6/+36
2013-07-02Move sssd_pac binary to the IPA and AD providersStephen Gallagher1-5/+8
This will ensure that we aren't pulling in extra samba4 dependencies for the Kerberos provider.
2013-07-02BUILD: Use pkg-config to detect cmockaLukas Slebodnik2-20/+19
2013-07-02RPM: Move sssd_pac to the krb5-common subpackageJakub Hrozek1-4/+4
The PAC responder is now used by both IPA and AD providers.
2013-07-02Remove sysv->systemd upgrade routinesStephen Gallagher1-12/+0
There are no longer any Fedora platforms running SSSD with SYSV init scripts. We don't need the upgrade logic any more.
2013-07-02Move pre and post scripts to sssd-commonStephen Gallagher1-5/+5
2013-07-01ldap: only update shadowLastChange when password change is successfulJim Collins1-1/+2
https://fedorahosted.org/sssd/ticket/1999 ldap_auth.c code which was added to SSSD for updating the shadowLastChange when "ldap_chpass_update_last_change" option is enabled updates shadowLastChange even when the PAM password change status reports failure. We should only update shadowLastChange on PAM password change success or we open up a work around for users to avoid changing their passwords periodically as required by policy. The user simply attempts to change password, fails by trying to set new password which invalid (denied due to password history check) yet shadowLastChange is updated, avoiding their need to actually change the password they are using.