<feed xmlns='http://www.w3.org/2005/Atom'>
<title>sssd.git/src/providers/ldap/sdap_sudo.c, branch simo</title>
<subtitle>Unnamed repository; edit this file to name it for gitweb.</subtitle>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/sbose/public_git/sssd.git/'/>
<entry>
<title>Add domain arguments to sysdb sudo functions</title>
<updated>2013-01-11T15:43:14+00:00</updated>
<author>
<name>Simo Sorce</name>
<email>simo@redhat.com</email>
</author>
<published>2013-01-08T06:25:47+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/sbose/public_git/sssd.git/commit/?id=f323cc33b275f7f05d9946a0bbd4566902f11b04'/>
<id>f323cc33b275f7f05d9946a0bbd4566902f11b04</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>sudo smart refresh: fix debug message</title>
<updated>2013-01-07T20:47:24+00:00</updated>
<author>
<name>Pavel Březina</name>
<email>pbrezina@redhat.com</email>
</author>
<published>2013-01-07T10:13:40+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/sbose/public_git/sssd.git/commit/?id=7ed683eba94b5bcdc4fe3f9037d80d0fedcddfd0'/>
<id>7ed683eba94b5bcdc4fe3f9037d80d0fedcddfd0</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>sudo smart refresh: do not include usn in filter if no valid usn is known</title>
<updated>2013-01-07T20:47:24+00:00</updated>
<author>
<name>Pavel Březina</name>
<email>pbrezina@redhat.com</email>
</author>
<published>2013-01-07T08:56:18+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/sbose/public_git/sssd.git/commit/?id=023ebc3d7e76978cfe7952480e0d7d88a2e1f690'/>
<id>023ebc3d7e76978cfe7952480e0d7d88a2e1f690</id>
<content type='text'>
https://fedorahosted.org/sssd/ticket/1736

When there are no rules during first refresh, we don't have valid
USN value. We use 0 in this case, but it turned out that OpenLDAP
takes it as invalid time format (if modifyTimestamp is used instead
of USN) and thus returns no records.

Now we don't include USN/modifyTimestamp attribute in the filter
if such situasion occurs.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
https://fedorahosted.org/sssd/ticket/1736

When there are no rules during first refresh, we don't have valid
USN value. We use 0 in this case, but it turned out that OpenLDAP
takes it as invalid time format (if modifyTimestamp is used instead
of USN) and thus returns no records.

Now we don't include USN/modifyTimestamp attribute in the filter
if such situasion occurs.
</pre>
</div>
</content>
</entry>
<entry>
<title>LDAP: initialize refresh function handler</title>
<updated>2013-01-04T15:34:23+00:00</updated>
<author>
<name>Ondrej Kos</name>
<email>okos@redhat.com</email>
</author>
<published>2012-12-19T09:25:39+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/sbose/public_git/sssd.git/commit/?id=d091342880477358cf7317111abce05fb5802aab'/>
<id>d091342880477358cf7317111abce05fb5802aab</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>Add default section to switch statement</title>
<updated>2012-12-19T17:42:58+00:00</updated>
<author>
<name>Sumit Bose</name>
<email>sbose@redhat.com</email>
</author>
<published>2012-12-19T09:19:01+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/sbose/public_git/sssd.git/commit/?id=767a679fd5ac2f409476180c0dbcd8ecbe991503'/>
<id>767a679fd5ac2f409476180c0dbcd8ecbe991503</id>
<content type='text'>
switch statements should always have a default section. In this
particular case gcc gave a "'send_fn' may be used uninitialized in this
function" warning.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
switch statements should always have a default section. In this
particular case gcc gave a "'send_fn' may be used uninitialized in this
function" warning.
</pre>
</div>
</content>
</entry>
<entry>
<title>sudo: do full refresh when data provider is back online</title>
<updated>2012-12-18T16:15:22+00:00</updated>
<author>
<name>Pavel Březina</name>
<email>pbrezina@redhat.com</email>
</author>
<published>2012-12-14T13:19:44+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/sbose/public_git/sssd.git/commit/?id=dcb712782ed81cd19fc5d9a5dab37e50294d6f2e'/>
<id>dcb712782ed81cd19fc5d9a5dab37e50294d6f2e</id>
<content type='text'>
https://fedorahosted.org/sssd/ticket/1689

Add a online callback if the first full refresh fails due to the
provider beeing offline so we can perform the refresh as soon as
possible.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
https://fedorahosted.org/sssd/ticket/1689

Add a online callback if the first full refresh fails due to the
provider beeing offline so we can perform the refresh as soon as
possible.
</pre>
</div>
</content>
</entry>
<entry>
<title>sudo: schedule another full refresh in short interval if the first fails</title>
<updated>2012-12-18T16:15:22+00:00</updated>
<author>
<name>Pavel Březina</name>
<email>pbrezina@redhat.com</email>
</author>
<published>2012-12-14T11:51:33+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/sbose/public_git/sssd.git/commit/?id=5860e321d82d007a5b7f4a1ff9a3893156e22b19'/>
<id>5860e321d82d007a5b7f4a1ff9a3893156e22b19</id>
<content type='text'>
https://fedorahosted.org/sssd/ticket/1689

If the first full refresh of sudo rules fails because the data provider
is offline, we will schedule another one in 2, 4, ... minutes.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
https://fedorahosted.org/sssd/ticket/1689

If the first full refresh of sudo rules fails because the data provider
is offline, we will schedule another one in 2, 4, ... minutes.
</pre>
</div>
</content>
</entry>
<entry>
<title>check dp error in sdap_sudo_full_refresh_done()</title>
<updated>2012-12-18T16:15:22+00:00</updated>
<author>
<name>Pavel Březina</name>
<email>pbrezina@redhat.com</email>
</author>
<published>2012-12-14T11:29:50+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/sbose/public_git/sssd.git/commit/?id=e5c9834c45849cbf82ea420fa3e0efdc102cc26e'/>
<id>e5c9834c45849cbf82ea420fa3e0efdc102cc26e</id>
<content type='text'>
https://fedorahosted.org/sssd/ticket/1689
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
https://fedorahosted.org/sssd/ticket/1689
</pre>
</div>
</content>
</entry>
<entry>
<title>add sdap_sudo_schedule_refresh()</title>
<updated>2012-12-18T16:15:22+00:00</updated>
<author>
<name>Pavel Březina</name>
<email>pbrezina@redhat.com</email>
</author>
<published>2012-12-14T12:15:53+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/sbose/public_git/sssd.git/commit/?id=260366c37cafab011cf48760eaf04282098ef800'/>
<id>260366c37cafab011cf48760eaf04282098ef800</id>
<content type='text'>
Reduces amount of code duplication.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Reduces amount of code duplication.
</pre>
</div>
</content>
</entry>
<entry>
<title>sudo: don't get stuck in rules and smart refresh when offline</title>
<updated>2012-12-11T15:10:32+00:00</updated>
<author>
<name>Pavel Březina</name>
<email>pbrezina@redhat.com</email>
</author>
<published>2012-12-04T14:17:59+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/sbose/public_git/sssd.git/commit/?id=3f23f27cd77eb4dbf362a0a4fdfbe0b18a2fb714'/>
<id>3f23f27cd77eb4dbf362a0a4fdfbe0b18a2fb714</id>
<content type='text'>
https://fedorahosted.org/sssd/ticket/1682

The problem was in following code:
if (ret != EOK || state-&gt;dp_error != DP_ERR_OK || state-&gt;error != EOK) {
    tevent_req_error(req, ret);
    return;
}

In situation when data provider error occurs (e.g. when offline),
ret == EOK but dp_error != DP_ERR_OK and we take the true branch.
This results in calling tevent_req_error(req, EOK).

Unfortunately, with EOK tevent_req_error only returns false, but
does not trigger callback and this tevent request hangs forever,
because no tevent_req_done(req) is called.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
https://fedorahosted.org/sssd/ticket/1682

The problem was in following code:
if (ret != EOK || state-&gt;dp_error != DP_ERR_OK || state-&gt;error != EOK) {
    tevent_req_error(req, ret);
    return;
}

In situation when data provider error occurs (e.g. when offline),
ret == EOK but dp_error != DP_ERR_OK and we take the true branch.
This results in calling tevent_req_error(req, EOK).

Unfortunately, with EOK tevent_req_error only returns false, but
does not trigger callback and this tevent request hangs forever,
because no tevent_req_done(req) is called.
</pre>
</div>
</content>
</entry>
</feed>
