diff options
| author | Dmitri Pal <dpal@redhat.com> | 2009-04-14 14:55:42 -0400 |
|---|---|---|
| committer | Simo Sorce <ssorce@redhat.com> | 2009-04-14 17:13:40 -0400 |
| commit | 9fc454c84d539cd90aed3a74a350bdc792455407 (patch) | |
| tree | a857209c49560b961c5a99d74f27e02f1e041fa3 /common/ini/ini_config_ut.c | |
| parent | 4ad7fe5e6acc87140fc29b635605af8445d2d32f (diff) | |
| download | sssd-9fc454c84d539cd90aed3a74a350bdc792455407.tar.gz sssd-9fc454c84d539cd90aed3a74a350bdc792455407.tar.xz sssd-9fc454c84d539cd90aed3a74a350bdc792455407.zip | |
Fixing memory issues in ini and collection
The read_line() function used an internal buffer allocated on stack
as temporary storage for a line read from file, then returned it.
read_line() now gets a buffer from the caller.
Fixed memory leaks in INI and Collection found by valgrind.
Diffstat (limited to 'common/ini/ini_config_ut.c')
| -rw-r--r-- | common/ini/ini_config_ut.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/common/ini/ini_config_ut.c b/common/ini/ini_config_ut.c index 6787c36c2..5441e02c0 100644 --- a/common/ini/ini_config_ut.c +++ b/common/ini/ini_config_ut.c @@ -303,7 +303,6 @@ int get_test() debug_item(item); - printf("Get item as string without duplication from NULL item.\n"); /* Get a string without duplicication */ @@ -759,9 +758,10 @@ int get_test() return -1; } - for (i=0;i<size;i++) printf("Section: [%s]\n", prop_array[i]); + for (i=0;i<size;i++) printf("Attribute: [%s]\n", prop_array[i]); free_attribute_list(prop_array); + destroy_collection(ini_config); printf("Done with get test!\n"); return EOK; } |