From 31ce005ef33bd0296d890251b14941be5a847a90 Mon Sep 17 00:00:00 2001
From: Karl Heyes <karl@xiph.org>
Date: Wed, 29 Aug 2007 02:01:28 +0000
Subject: [PATCH] small const update, add NULL check, NULL shouldn't occur here
 but it's better to play safe

svn path=/icecast/trunk/icecast/; revision=13649
---
 src/stats.c | 14 ++++++++------
 src/stats.h |  2 +-
 2 files changed, 9 insertions(+), 7 deletions(-)

diff --git a/src/stats.c b/src/stats.c
index 473e8a15..34e1ae1d 100644
--- a/src/stats.c
+++ b/src/stats.c
@@ -87,8 +87,8 @@ static int _compare_source_stats(void *a, void *b, void *arg);
 static int _free_stats(void *key);
 static int _free_source_stats(void *key);
 static void _add_event_to_queue(stats_event_t *event, event_queue_t *queue);
-static stats_node_t *_find_node(avl_tree *tree, char *name);
-static stats_source_t *_find_source(avl_tree *tree, char *source);
+static stats_node_t *_find_node(avl_tree *tree, const char *name);
+static stats_source_t *_find_source(avl_tree *tree, const char *source);
 static void _free_event(stats_event_t *event);
 static stats_event_t *_get_event_from_queue (event_queue_t *queue);
 
@@ -289,7 +289,7 @@ void stats_event_args(const char *source, char *name, char *format, ...)
     stats_event(source, name, buf);
 }
 
-static char *_get_stats(char *source, char *name)
+static char *_get_stats(const char *source, const char *name)
 {
     stats_node_t *stats = NULL;
     stats_source_t *src = NULL;
@@ -313,7 +313,7 @@ static char *_get_stats(char *source, char *name)
     return value;
 }
 
-char *stats_get_value(char *source, char *name)
+char *stats_get_value(const char *source, const char *name)
 {
     return(_get_stats(source, name));
 }
@@ -358,7 +358,7 @@ void stats_event_dec(const char *source, const char *name)
 /* note: you must call this function only when you have exclusive access
 ** to the avl_tree
 */
-static stats_node_t *_find_node(avl_tree *stats_tree, char *name)
+static stats_node_t *_find_node(avl_tree *stats_tree, const char *name)
 {
     stats_node_t *stats;
     avl_node *node;
@@ -385,7 +385,7 @@ static stats_node_t *_find_node(avl_tree *stats_tree, char *name)
 /* note: you must call this function only when you have exclusive access
 ** to the avl_tree
 */
-static stats_source_t *_find_source(avl_tree *source_tree, char *source)
+static stats_source_t *_find_source(avl_tree *source_tree, const char *source)
 {
     stats_source_t *stats;
     avl_node *node;
@@ -626,6 +626,8 @@ static void *_stats_thread(void *arg)
             event = _get_event_from_queue (&_global_event_queue);
             thread_mutex_unlock(&_global_event_mutex);
 
+            if (event == NULL)
+                continue;
             event->next = NULL;
 
             thread_mutex_lock(&_stats_mutex);
diff --git a/src/stats.h b/src/stats.h
index 46b5b229..6df316af 100644
--- a/src/stats.h
+++ b/src/stats.h
@@ -95,7 +95,7 @@ void stats_callback (client_t *client, void *notused);
 void stats_transform_xslt(client_t *client, const char *uri);
 void stats_sendxml(client_t *client);
 void stats_get_xml(xmlDocPtr *doc, int show_hidden, const char *show_mount);
-char *stats_get_value(char *source, char *name);
+char *stats_get_value(const char *source, const char *name);
 
 #endif  /* __STATS_H__ */