<feed xmlns='http://www.w3.org/2005/Atom'>
<title>sssd.git/common/ini/ini_config_ut.c, branch master</title>
<subtitle>Experimental work on SSSD - Systen Security Services Daemon</subtitle>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/simo/public_git/sssd.git/'/>
<entry>
<title>Remove common directory</title>
<updated>2010-08-19T15:15:09+00:00</updated>
<author>
<name>Stephen Gallagher</name>
<email>sgallagh@redhat.com</email>
</author>
<published>2010-08-13T18:51:16+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/simo/public_git/sssd.git/commit/?id=ad42d90b7e23978b62e36d6885d5fea0a105d6d0'/>
<id>ad42d90b7e23978b62e36d6885d5fea0a105d6d0</id>
<content type='text'>
All files formerly in common are now being built individually out
of the ding-libs repository.

git clone git://git.fedorahosted.org/git/ding-libs.git
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
All files formerly in common are now being built individually out
of the ding-libs repository.

git clone git://git.fedorahosted.org/git/ding-libs.git
</pre>
</div>
</content>
</entry>
<entry>
<title>Memory leak in case of empty value</title>
<updated>2010-06-28T12:38:14+00:00</updated>
<author>
<name>Dmitri Pal</name>
<email>dpal@redhat.com</email>
</author>
<published>2010-06-10T16:35:05+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/simo/public_git/sssd.git/commit/?id=e67e64ddefb993f7fdea4501f7efbe657cb0f164'/>
<id>e67e64ddefb993f7fdea4501f7efbe657cb0f164</id>
<content type='text'>
Addressing coverity issue. Ticket #513.
The memory was really leaked when
the configuration value was empty.
Added unit test that confirmed the bug.
MANUAL MERGE.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Addressing coverity issue. Ticket #513.
The memory was really leaked when
the configuration value was empty.
Added unit test that confirmed the bug.
MANUAL MERGE.
</pre>
</div>
</content>
</entry>
<entry>
<title>Adding support for explicit 32/64 types (attempt 2).</title>
<updated>2010-05-17T17:52:16+00:00</updated>
<author>
<name>Dmitri Pal</name>
<email>dpal@redhat.com</email>
</author>
<published>2010-05-17T17:21:56+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/simo/public_git/sssd.git/commit/?id=5c65c5a50c496b161c7532055778c9e223185c21'/>
<id>5c65c5a50c496b161c7532055778c9e223185c21</id>
<content type='text'>
This is a reworked patch to add support for explicit
32 and 64 bit values in the config files.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This is a reworked patch to add support for explicit
32 and 64 bit values in the config files.
</pre>
</div>
</content>
</entry>
<entry>
<title>Introducing a comment object</title>
<updated>2010-04-30T11:51:19+00:00</updated>
<author>
<name>Dmitri Pal</name>
<email>dpal@redhat.com</email>
</author>
<published>2010-04-22T01:16:18+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/simo/public_git/sssd.git/commit/?id=01c94127cf0fd72b6db828f1830fff1e5ed55bd1'/>
<id>01c94127cf0fd72b6db828f1830fff1e5ed55bd1</id>
<content type='text'>
Comment object will store
the comments found in the INI file.
It is based on the ref_array
interface.

Fixing review comments for comment obj.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Comment object will store
the comments found in the INI file.
It is based on the ref_array
interface.

Fixing review comments for comment obj.
</pre>
</div>
</content>
</entry>
<entry>
<title>Fix ini_config unit test</title>
<updated>2010-04-14T16:15:54+00:00</updated>
<author>
<name>Stephen Gallagher</name>
<email>sgallagh@redhat.com</email>
</author>
<published>2010-04-14T15:20:41+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/simo/public_git/sssd.git/commit/?id=b460494e9361f3dbc8da75d4d907a15c90382b49'/>
<id>b460494e9361f3dbc8da75d4d907a15c90382b49</id>
<content type='text'>
When running 'make distcheck', the entire source directory is set
to read-only, to ensure that the build process only has write
access to $builddir. As a result, this was causing the unit test
for file mode to fail, since the file it was testing resides in
the $srcdir.

This patch guarantees that the test file has the correct
permissions prior to running the access test.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
When running 'make distcheck', the entire source directory is set
to read-only, to ensure that the build process only has write
access to $builddir. As a result, this was causing the unit test
for file mode to fail, since the file it was testing resides in
the $srcdir.

This patch guarantees that the test file has the correct
permissions prior to running the access test.
</pre>
</div>
</content>
</entry>
<entry>
<title>Acess control and config change checks</title>
<updated>2010-04-14T16:15:54+00:00</updated>
<author>
<name>Dmitri Pal</name>
<email>dpal@redhat.com</email>
</author>
<published>2010-04-03T00:08:14+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/simo/public_git/sssd.git/commit/?id=bf7247298136660f512bd1a96f68be1487f425b6'/>
<id>bf7247298136660f512bd1a96f68be1487f425b6</id>
<content type='text'>
1) Fixed the issue that metadata was saved
as numbers. Was supposed to be saved as strings.
2) Added two functions. One is to check permissions
on the config file. Another to check if the file
has changed and thus the cinfiguration needs
to be reread.
3) Added unit test will sample code
and comments how to use the functions.
4) Added doxygen description in the comments.
5) Fixed couple typos and ommisions here and there.

[INI] Fixing crash detected on 64-bit system

This patch corrects original code to be
more on the safe side and check parameters
before using.
Instead of dereferencing metadata it is now
passed as reference to the next level.
It is not used there yet so no other new changes
needed so far.

[INI] Addressing review comments

[INI] Addressing comments.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
1) Fixed the issue that metadata was saved
as numbers. Was supposed to be saved as strings.
2) Added two functions. One is to check permissions
on the config file. Another to check if the file
has changed and thus the cinfiguration needs
to be reread.
3) Added unit test will sample code
and comments how to use the functions.
4) Added doxygen description in the comments.
5) Fixed couple typos and ommisions here and there.

[INI] Fixing crash detected on 64-bit system

This patch corrects original code to be
more on the safe side and check parameters
before using.
Instead of dereferencing metadata it is now
passed as reference to the next level.
It is not used there yet so no other new changes
needed so far.

[INI] Addressing review comments

[INI] Addressing comments.
</pre>
</div>
</content>
</entry>
<entry>
<title>Adding metadata interface</title>
<updated>2010-04-14T16:15:54+00:00</updated>
<author>
<name>Dmitri Pal</name>
<email>dpal@redhat.com</email>
</author>
<published>2010-03-29T20:58:29+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/simo/public_git/sssd.git/commit/?id=e4956873b0ca4ec732e009f88be43549506f7819'/>
<id>e4956873b0ca4ec732e009f88be43549506f7819</id>
<content type='text'>
This patch:
1) Adds the definition of the metadata interface
to the header file. The functions that were exposed
for no good reason are now hidden.
2) Previously exposed functions and their descriptions
are removed from the public header and placed into
the source code for now.
3) The function that reads the config file no longer
tries to close file in case of error.
4) Lines collection is still passed in into the reading
function but as a collection itself not as a pointer
to it.
5) All the parts related to processing lines are currently
ifdefed using HAVE_VALIDATION that is currently is not defined.
This is done to disable creation of the lines collection
utill it is actually needed. I did not want to blindly remove
it though and loose already done work that will be useful
in future.
6) Version of the library and interface is updated
7) New header and source modules are introduced to hold functions
related to the meta data. They are mostly stubbed out.

This is incomplete patch. It builds and make check runs.
It is created just to simplify the review a bit.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This patch:
1) Adds the definition of the metadata interface
to the header file. The functions that were exposed
for no good reason are now hidden.
2) Previously exposed functions and their descriptions
are removed from the public header and placed into
the source code for now.
3) The function that reads the config file no longer
tries to close file in case of error.
4) Lines collection is still passed in into the reading
function but as a collection itself not as a pointer
to it.
5) All the parts related to processing lines are currently
ifdefed using HAVE_VALIDATION that is currently is not defined.
This is done to disable creation of the lines collection
utill it is actually needed. I did not want to blindly remove
it though and loose already done work that will be useful
in future.
6) Version of the library and interface is updated
7) New header and source modules are introduced to hold functions
related to the meta data. They are mostly stubbed out.

This is incomplete patch. It builds and make check runs.
It is created just to simplify the review a bit.
</pre>
</div>
</content>
</entry>
<entry>
<title>Fixing type conversion in INI interface.</title>
<updated>2010-03-17T16:03:58+00:00</updated>
<author>
<name>Dmitri Pal</name>
<email>dpal@redhat.com</email>
</author>
<published>2010-03-11T17:28:40+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/simo/public_git/sssd.git/commit/?id=8834e6fe4f89d7b0227377619d96d455e1f28ebb'/>
<id>8834e6fe4f89d7b0227377619d96d455e1f28ebb</id>
<content type='text'>
Additional changes.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Additional changes.
</pre>
</div>
</content>
</entry>
<entry>
<title>Fixing verbosity and formatting of the INI unit test.</title>
<updated>2010-03-12T14:00:28+00:00</updated>
<author>
<name>Dmitri Pal</name>
<email>dpal@redhat.com</email>
</author>
<published>2010-03-11T17:20:10+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/simo/public_git/sssd.git/commit/?id=70a54fe1c527efabf0c3258a2daa669f5e2bb788'/>
<id>70a54fe1c527efabf0c3258a2daa669f5e2bb788</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>INI: Added method to get string list with empty values</title>
<updated>2010-01-21T20:02:15+00:00</updated>
<author>
<name>Dmitri Pal</name>
<email>dpal@redhat.com</email>
</author>
<published>2009-12-25T23:54:56+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/simo/public_git/sssd.git/commit/?id=526dd14cdc1461859a5dd1cdd83ee9b1268175fe'/>
<id>526dd14cdc1461859a5dd1cdd83ee9b1268175fe</id>
<content type='text'>
The original implementation was compressing the list,
throwing away empty strings.
The function that did that was pretty brain damaging.
I cleaned it up and adjusted so that it could return
list with empty values and without them.
The old function was turned into a wrapper and a new
high level function was intorduced to provide
ability to get both empty and non empty strings.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The original implementation was compressing the list,
throwing away empty strings.
The function that did that was pretty brain damaging.
I cleaned it up and adjusted so that it could return
list with empty values and without them.
The old function was turned into a wrapper and a new
high level function was intorduced to provide
ability to get both empty and non empty strings.
</pre>
</div>
</content>
</entry>
</feed>
