[PATCH v2 1/3] libkmod-module: Do not free the list in kmod_module_info_append

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



In error case, just return NULL and let the caller free the list.
---
 libkmod/libkmod-module.c |   15 ++++++++-------
 1 file changed, 8 insertions(+), 7 deletions(-)

diff --git a/libkmod/libkmod-module.c b/libkmod/libkmod-module.c
index ae0d9c8..d7d6938 100644
--- a/libkmod/libkmod-module.c
+++ b/libkmod/libkmod-module.c
@@ -2100,16 +2100,13 @@ static struct kmod_list *kmod_module_info_append(struct kmod_list **list, const
 	struct kmod_list *n;
 
 	info = kmod_module_info_new(key, keylen, value, valuelen);
-	if (info == NULL) {
-		kmod_module_info_free_list(*list);
+	if (info == NULL)
 		return NULL;
-	}
 	n = kmod_list_append(*list, info);
-	if (n == NULL) {
+	if (n != NULL)
+		*list = n;
+	else
 		kmod_module_info_free(info);
-		kmod_module_info_free_list(*list);
-	}
-	*list = n;
 	return n;
 }
 
@@ -2171,6 +2168,10 @@ KMOD_EXPORT int kmod_module_get_info(const struct kmod_module *mod, struct kmod_
 	ret = count;
 
 list_error:
+	if (ret < 0) {
+		kmod_module_info_free_list(*list);
+		*list = NULL;
+	}
 	free(strings);
 	return ret;
 }
-- 
1.7.10.4

--
To unsubscribe from this list: send the line "unsubscribe linux-modules" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux