cleanup: addons/51degrees/51d.c: add missing var initialization

chipitsine · chipitsine · commit 91b2d9699a27 · 2026-05-12T11:14:12.000+02:00
addons/51degrees/51d.c:1076:8: error: Uninitialized variable: _51d_prop_name.name [uninitvar]
diff --git a/addons/51degrees/51d.c b/addons/51degrees/51d.c
@@ -1053,7 +1053,7 @@ static int init_51degrees(void)
 
 static void deinit_51degrees(void)
 {
-	struct _51d_property_names *_51d_prop_name, *_51d_prop_nameb;
+	struct _51d_property_names *_51d_prop_name = NULL, *_51d_prop_nameb = NULL;
 
 #if defined(FIFTYONEDEGREES_H_PATTERN_INCLUDED) || defined(FIFTYONEDEGREES_H_TRIE_INCLUDED)
 	free(global_51degrees.header_names);
@@ -1073,6 +1073,7 @@ static void deinit_51degrees(void)
 	ha_free(&global_51degrees.data_file_path);
 	list_for_each_entry_safe(_51d_prop_name, _51d_prop_nameb, &global_51degrees.property_names, list) {
 		LIST_DELETE(&_51d_prop_name->list);
+		free(_51d_prop_name->name);
 		free(_51d_prop_name);
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -1053,7 +1053,7 @@ static int init_51degrees(void)`
`1053`	`1053`
`1054`	`1054`	`static void deinit_51degrees(void)`
`1055`	`1055`	`{`
`1056`		`- struct _51d_property_names _51d_prop_name, _51d_prop_nameb;`
	`1056`	`+ struct _51d_property_names _51d_prop_name = NULL, _51d_prop_nameb = NULL;`
`1057`	`1057`
`1058`	`1058`	`#if defined(FIFTYONEDEGREES_H_PATTERN_INCLUDED) \|\| defined(FIFTYONEDEGREES_H_TRIE_INCLUDED)`
`1059`	`1059`	`free(global_51degrees.header_names);`
`@@ -1073,6 +1073,7 @@ static void deinit_51degrees(void)`
`1073`	`1073`	`ha_free(&global_51degrees.data_file_path);`
`1074`	`1074`	`list_for_each_entry_safe(_51d_prop_name, _51d_prop_nameb, &global_51degrees.property_names, list) {`
`1075`	`1075`	`LIST_DELETE(&_51d_prop_name->list);`
	`1076`	`+ free(_51d_prop_name->name);`
`1076`	`1077`	`free(_51d_prop_name);`
`1077`	`1078`	`}`
`1078`	`1079`