Fix possible memory leak

	- see realloc man :
The realloc() function returns a pointer to the newly allocated memory, which is suitably aligned for any kind of variable and may be different from ptr, or NULL if the request fails.  If size was equal to 0, either NULL or a pointer suitable to be passed to free() is returned.  If realloc() fails the original block is left untouched; it is not freed or moved.

1 files changed, 5 insertions, 2 deletions

diff --git a/vconf.c b/vconf.c
index 11bd47e..45ebd15 100755
--- a/vconf.c
+++ b/vconf.c
@@ -1574,6 +1574,7 @@ static int _vconf_get_key_elektra_format(keynode_t *keynode, FILE *fp)
 	int err_no = 0;
 	char err_buf[100] = { 0, };
 	int func_ret = VCONF_OK;
+	char tmp;
 
 	INFO("_vconf_get_key_elektra_format start");
 
@@ -1620,11 +1621,13 @@ static int _vconf_get_key_elektra_format(keynode_t *keynode, FILE *fp)
 	{
 		if(value) {
 			value_size = value_size + strlen(file_buf);
-			value = (char *) realloc(value, value_size);
-			if(value == NULL) {
+			tmp = (char *) realloc(value, value_size);
+			if(!tmp) {
+				free(value);
 				func_ret = VCONF_ERROR_NO_MEM;
 				break;
 			}
+			value = tmp;
 			strncat(value, file_buf, strlen(file_buf));
 		} else {
 			value_size = strlen(file_buf) + 1;