added output_buffer_free()

To do proper cleanup before exiting, we have to provide a destructor for OutputBuffer. One day, valgrind will not complain about memory leaks! git-svn-id: https://svn.musicpd.org/mpd/trunk@7315 09075e82-0dd4-0310-85a5-a0d7c8717e4f

added output_buffer_free()
f2cdac6e · Max Kellermann · Eric Wong · b6881345 · f2cdac6e · f2cdac6e
Commit f2cdac6e authored Apr 12, 2008 by Max Kellermann Committed by Eric Wong Apr 12, 2008
Hide whitespace changes
Inline Side-by-side

Showing with 9 additions and 0 deletions

outputBuffer.c src/outputBuffer.c +6 -0

outputBuffer.h src/outputBuffer.h +2 -0

playerData.c src/playerData.c +1 -0

No files found.
--- a/src/outputBuffer.c
+++ b/src/outputBuffer.c
@@ -36,6 +36,12 @@ void initOutputBuffer(OutputBuffer * cb, unsigned int size)
 	cb->currentChunk = -1;
 }

+void output_buffer_free(OutputBuffer * cb)
+{
+	assert(cb->chunks != NULL);
+	free(cb->chunks);
+}
+
 void clearOutputBuffer(OutputBuffer * cb)
 {
 	cb->end = cb->begin;

--- a/src/outputBuffer.h
+++ b/src/outputBuffer.h
@@ -62,6 +62,8 @@ typedef struct _OutputBuffer {

 void initOutputBuffer(OutputBuffer * cb, unsigned int size);

+void output_buffer_free(OutputBuffer * cb);
+
 void clearOutputBuffer(OutputBuffer * cb);

 void flushOutputBuffer(OutputBuffer * cb);

--- a/src/playerData.c
+++ b/src/playerData.c
@@ -116,5 +116,6 @@ void freePlayerData(void)
 	 * access playerData_pd and we need to keep it available for them */
 	waitpid(-1, NULL, 0);

+	output_buffer_free(&playerData_pd.buffer);
 	free(playerData_pd.audioDeviceStates);
 }