<feed xmlns='http://www.w3.org/2005/Atom'>
<title>sssd.git/src/responder, branch python_api</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>Refactoring: remove duplicated code in nss responder</title>
<updated>2013-04-21T09:23:43+00:00</updated>
<author>
<name>Sumit Bose</name>
<email>sbose@redhat.com</email>
</author>
<published>2013-04-15T08:58:05+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/sbose/public_git/sssd.git/commit/?id=b3e247cef1f1c81a24ae7759903c11289744e94c'/>
<id>b3e247cef1f1c81a24ae7759903c11289744e94c</id>
<content type='text'>
Different user and group lookup requests used nearly identical code,
this patch unifies some of the related code paths.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Different user and group lookup requests used nearly identical code,
this patch unifies some of the related code paths.
</pre>
</div>
</content>
</entry>
<entry>
<title>Fix and rename get_my_domain_data()</title>
<updated>2013-04-21T08:22:36+00:00</updated>
<author>
<name>Sumit Bose</name>
<email>sbose@redhat.com</email>
</author>
<published>2013-04-19T15:44:26+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/sbose/public_git/sssd.git/commit/?id=d29d5d9e0e9ee0396a46f4375092644f29024f25'/>
<id>d29d5d9e0e9ee0396a46f4375092644f29024f25</id>
<content type='text'>
The task of get_my_domain_data() is to read some information about the
configured domain from the cache. While the sysdb interface was
redesigned some changes changed the behaviour so that the data of the
domain of the current request was read. If this domain is a sub-domain
the wrong data was read. As a result group-memberships of the configured
domain were not taken into account.

The original code didn't made it easy to see that always the parent
domain should be used here, because there was no comment indication this
and the function name get_my_domain_data() didn't made it clear either.
Additionally to fixing the issue this patch also adds a comment and
rename the function to get_parent_domain_data().

Fixes https://fedorahosted.org/sssd/ticket/1888
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The task of get_my_domain_data() is to read some information about the
configured domain from the cache. While the sysdb interface was
redesigned some changes changed the behaviour so that the data of the
domain of the current request was read. If this domain is a sub-domain
the wrong data was read. As a result group-memberships of the configured
domain were not taken into account.

The original code didn't made it easy to see that always the parent
domain should be used here, because there was no comment indication this
and the function name get_my_domain_data() didn't made it clear either.
Additionally to fixing the issue this patch also adds a comment and
rename the function to get_parent_domain_data().

Fixes https://fedorahosted.org/sssd/ticket/1888
</pre>
</div>
</content>
</entry>
<entry>
<title>Inform about function duplication.</title>
<updated>2013-04-17T09:39:57+00:00</updated>
<author>
<name>Michal Zidek</name>
<email>mzidek@redhat.com</email>
</author>
<published>2013-04-16T11:14:51+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/sbose/public_git/sssd.git/commit/?id=714ba5f50551a42df324714358dc379b351d4a53'/>
<id>714ba5f50551a42df324714358dc379b351d4a53</id>
<content type='text'>
sss_mc_set_recycled is a static function, that should not
be used outside nsssrv_mmap_cache.c. The sss_cache tool
is an exception, because in the case when sssd is not running,
sss_cache must invalidate the memory cache file. That is why
sss_mc_set_recycled was copied to the tools_mc_util.c
(as helper function for sss_memcache_invalidate function).
It was duplicated to allow this function to remain static
(and invisible to any .h files), so that it is not used anywhere else.
Wrong usage of this function might cause race conditions and corrupt
the cache.

I'll add comments about the duplication to the code.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
sss_mc_set_recycled is a static function, that should not
be used outside nsssrv_mmap_cache.c. The sss_cache tool
is an exception, because in the case when sssd is not running,
sss_cache must invalidate the memory cache file. That is why
sss_mc_set_recycled was copied to the tools_mc_util.c
(as helper function for sss_memcache_invalidate function).
It was duplicated to allow this function to remain static
(and invisible to any .h files), so that it is not used anywhere else.
Wrong usage of this function might cause race conditions and corrupt
the cache.

I'll add comments about the duplication to the code.
</pre>
</div>
</content>
</entry>
<entry>
<title>Allow using flatname for subdomain home dir template</title>
<updated>2013-04-10T11:38:44+00:00</updated>
<author>
<name>Jakub Hrozek</name>
<email>jhrozek@redhat.com</email>
</author>
<published>2013-04-04T11:03:23+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/sbose/public_git/sssd.git/commit/?id=6fc4702a3037d9bb5b27bcb58f70edf1802b7b19'/>
<id>6fc4702a3037d9bb5b27bcb58f70edf1802b7b19</id>
<content type='text'>
https://fedorahosted.org/sssd/ticket/1609
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
https://fedorahosted.org/sssd/ticket/1609
</pre>
</div>
</content>
</entry>
<entry>
<title>Allocate PAM DP request data on responder context</title>
<updated>2013-04-08T21:47:23+00:00</updated>
<author>
<name>Jakub Hrozek</name>
<email>jhrozek@redhat.com</email>
</author>
<published>2013-04-05T15:02:20+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/sbose/public_git/sssd.git/commit/?id=a28391f68a09eaedbe1a515f513ca56bbb94d7a5'/>
<id>a28391f68a09eaedbe1a515f513ca56bbb94d7a5</id>
<content type='text'>
https://fedorahosted.org/sssd/ticket/1869

Currently the private data passed to the PAM request is a structure
allocated on the client context. But in the odd case where the back end
would be stopped or stuck until the idle timeout hits, the DP callback
would access data that were freed when the client timed out.

This patch introduces a new structure allocated on responder context,
whose only purpose is to live as long as the request is active.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
https://fedorahosted.org/sssd/ticket/1869

Currently the private data passed to the PAM request is a structure
allocated on the client context. But in the odd case where the back end
would be stopped or stuck until the idle timeout hits, the DP callback
would access data that were freed when the client timed out.

This patch introduces a new structure allocated on responder context,
whose only purpose is to live as long as the request is active.
</pre>
</div>
</content>
</entry>
<entry>
<title>Check for correct variable name</title>
<updated>2013-04-03T11:26:41+00:00</updated>
<author>
<name>Jakub Hrozek</name>
<email>jhrozek@redhat.com</email>
</author>
<published>2013-04-03T09:18:31+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/sbose/public_git/sssd.git/commit/?id=274fe6a4f8bcb23e31929430110c0b52e9ce233a'/>
<id>274fe6a4f8bcb23e31929430110c0b52e9ce233a</id>
<content type='text'>
https://fedorahosted.org/sssd/ticket/1864
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
https://fedorahosted.org/sssd/ticket/1864
</pre>
</div>
</content>
</entry>
<entry>
<title>Making the authtok structure really opaque.</title>
<updated>2013-04-02T15:01:08+00:00</updated>
<author>
<name>Lukas Slebodnik</name>
<email>lslebodn@redhat.com</email>
</author>
<published>2013-03-14T08:10:39+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/sbose/public_git/sssd.git/commit/?id=9acfb09f7969a69f58bd45c856b01700541853ca'/>
<id>9acfb09f7969a69f58bd45c856b01700541853ca</id>
<content type='text'>
Definition of structure sss_auth_token was removed from header file
authtok.h and there left only declaration of this structure.
Therefore only way how to use this structure is to use accessory function from
same header file.

To creating new empty authotok can only be used newly created function
sss_authtok_new(). TALLOC context was removed from copy and setter functions,
because pointer to stuct sss_auth_token is used as a memory context.

All declaration of struct sss_auth_token variables was replaced with
pointer to this structure and related changes was made in source code.

Function copy_pam_data can copy from argument src which was dynamically
allocated with function create_pam_data() or zero initialized struct pam_data
allocated on stack.

https://fedorahosted.org/sssd/ticket/1830
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Definition of structure sss_auth_token was removed from header file
authtok.h and there left only declaration of this structure.
Therefore only way how to use this structure is to use accessory function from
same header file.

To creating new empty authotok can only be used newly created function
sss_authtok_new(). TALLOC context was removed from copy and setter functions,
because pointer to stuct sss_auth_token is used as a memory context.

All declaration of struct sss_auth_token variables was replaced with
pointer to this structure and related changes was made in source code.

Function copy_pam_data can copy from argument src which was dynamically
allocated with function create_pam_data() or zero initialized struct pam_data
allocated on stack.

https://fedorahosted.org/sssd/ticket/1830
</pre>
</div>
</content>
</entry>
<entry>
<title>Reusing create_pam_data() on the other places.</title>
<updated>2013-04-02T15:01:08+00:00</updated>
<author>
<name>Lukas Slebodnik</name>
<email>lslebodn@redhat.com</email>
</author>
<published>2013-03-14T08:44:41+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/sbose/public_git/sssd.git/commit/?id=53b58615fbc13eddcd6e2f28066b67cb5f16b6d3'/>
<id>53b58615fbc13eddcd6e2f28066b67cb5f16b6d3</id>
<content type='text'>
Function create_pam_data() should be only one way how to create new
struct pam_data, because it also initialize destructor to created
object.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Function create_pam_data() should be only one way how to create new
struct pam_data, because it also initialize destructor to created
object.
</pre>
</div>
</content>
</entry>
<entry>
<title>coding style fix</title>
<updated>2013-03-20T12:29:29+00:00</updated>
<author>
<name>Pavel Březina</name>
<email>pbrezina@redhat.com</email>
</author>
<published>2012-10-19T10:23:17+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/sbose/public_git/sssd.git/commit/?id=3c6b579b14bbf24acd4e8151673fc9c17c9f8615'/>
<id>3c6b579b14bbf24acd4e8151673fc9c17c9f8615</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>change responder contexts hierarchy</title>
<updated>2013-03-20T12:29:29+00:00</updated>
<author>
<name>Pavel Březina</name>
<email>pbrezina@redhat.com</email>
</author>
<published>2012-10-19T09:43:47+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/sbose/public_git/sssd.git/commit/?id=7c9fe57ad82747a32721ca0a08c5569282f3e0c4'/>
<id>7c9fe57ad82747a32721ca0a08c5569282f3e0c4</id>
<content type='text'>
https://fedorahosted.org/sssd/ticket/1575

The hierarchy is now:
main_ctx -&gt; responder_ctx -&gt; specific_ctx

where specific_ctx is one of sudo, pam, etc.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
https://fedorahosted.org/sssd/ticket/1575

The hierarchy is now:
main_ctx -&gt; responder_ctx -&gt; specific_ctx

where specific_ctx is one of sudo, pam, etc.
</pre>
</div>
</content>
</entry>
</feed>
