stop setting local pntrs NULL when globals needed (tmp commented out)

all *_clear functions get a pointer that should be freed by that function.
after freeing they are set to NULL, but thats only in local scope, i.e:
the global pointer stays NOT NULL

void sink_list_clear(sink_info** sink_list, ...) {
  [..]
  free(sink_list);
  sink_list = NULL;
}

so if it is really intended, all clear functions need to be changed to:
void sink_list_clear(sink_info ***sink_list,...) {
  [..]
  free(*sink_list);
  *sink_list = NULL;
}

1 files changed, 7 insertions, 2 deletions

diff --git a/src/sink.c b/src/sink.c
index 3622f74..01f0e04 100644
--- a/src/sink.c
+++ b/src/sink.c
@@ -35,7 +35,7 @@ void sink_clear(sink_info* sink) {
 	sink_input_list_clear(sink->input_list, &sink->input_max);
 
 	free(sink);
-	sink = NULL;
+	//sink = NULL;
 }
 
 void sink_check(sink_info** sink) {
@@ -97,5 +97,10 @@ void sink_list_clear(sink_info** sink_list, uint32_t* max, uint32_t* counter) {
 	(*counter) = 0;
 	
 	free(sink_list);
-	sink_list = NULL;
+	// sink_list = NULL; 
+
+	/* TODO: for all *_clear:
+	 * setting local parameter to NULL doesnt do want is wanted
+	 * pointer to sink_list would be needed here...
+	 */
 }