Fix removing the config and sending a SIGUSR1 results in segfault

[monky] / src / xmms2.c

diff --git a/src/xmms2.c b/src/xmms2.c
index 57b951d..3dde9aa 100644 (file)
--- a/src/xmms2.c
+++ b/src/xmms2.c
@@ -1,4 +1,7 @@
-/* Conky, a system monitor, based on torsmo
+/* -*- mode: c; c-basic-offset: 4; tab-width: 4; indent-tabs-mode: t -*-
+ * vim: ts=4 sw=4 noet ai cindent syntax=c
+ *
+ * Conky, a system monitor, based on torsmo
  *
  * Any original torsmo code is licensed under the BSD license
  *
@@ -6,7 +9,7 @@
  *
  * Please see COPYING for details
  *
- * Copyright (c) 2005-2009 Brenden Matthews, Philip Kovacs, et. al.
+ * Copyright (c) 2005-2010 Brenden Matthews, Philip Kovacs, et. al.
  *     (see AUTHORS)
  * All rights reserved.
  *
@@ -22,8 +25,6 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  *
- * vim: ts=4 sw=4 noet ai cindent syntax=c
- *
  */
 
 #include "conky.h"
@@ -86,14 +87,14 @@ static void xmms_alloc(struct information *ptr)
 void connection_lost(void *p)
 {
        struct information *ptr = p;
-        ptr->xmms2.conn_state = CONN_NO;
+       ptr->xmms2.conn_state = CONN_NO;
 
-        fprintf(stderr,"XMMS2 connection failed. %s\n", xmmsc_get_last_error(xmms2_conn));
+       fprintf(stderr,"XMMS2 connection failed. %s\n", xmmsc_get_last_error(xmms2_conn));
 
-        xmms_alloc(ptr);
-        strncpy(ptr->xmms2.status, "Disocnnected", text_buffer_size - 1);
-        ptr->xmms2.playlist[0] = '\0';
-        ptr->xmms2.id = 0;
+       xmms_alloc(ptr);
+       strncpy(ptr->xmms2.status, "Disocnnected", text_buffer_size - 1);
+       ptr->xmms2.playlist[0] = '\0';
+       ptr->xmms2.id = 0;
 }
 
 
@@ -112,7 +113,7 @@ int handle_curent_id(xmmsv_t *value, void *p)
        if (xmmsv_get_error(value, &errbuf)) {
                fprintf(stderr,"XMMS2 server error. %s\n", errbuf);
                return TRUE;
-               }
+       }
 
        if (xmmsv_get_int(value, &current_id) && current_id > 0) {
 
@@ -137,10 +138,10 @@ int handle_curent_id(xmmsv_t *value, void *p)
 
                if (xmmsv_dict_get(infos, "title", &dict_entry) && xmmsv_get_string(dict_entry, &charval)) 
                        strncpy(ptr->xmms2.title, charval, text_buffer_size - 1);
-       
+
                if (xmmsv_dict_get(infos, "album", &dict_entry) && xmmsv_get_string(dict_entry, &charval)) 
                        strncpy(ptr->xmms2.album, charval, text_buffer_size - 1);
-               
+
                if (xmmsv_dict_get(infos, "genre", &dict_entry) && xmmsv_get_string(dict_entry, &charval))
                        strncpy(ptr->xmms2.genre, charval, text_buffer_size - 1);
 
@@ -152,7 +153,7 @@ int handle_curent_id(xmmsv_t *value, void *p)
 
                if (xmmsv_dict_get(infos, "date", &dict_entry) && xmmsv_get_string(dict_entry, &charval))
                        strncpy(ptr->xmms2.date, charval, text_buffer_size - 1);
-               
+
 
 
                if (xmmsv_dict_get(infos, "tracknr", &dict_entry) && xmmsv_get_int(dict_entry, &intval))
@@ -213,19 +214,19 @@ int handle_playback_state_change(xmmsv_t *value, void *p)
        }
 
        if (xmmsv_get_int(value, &pb_state)) {
-       switch (pb_state) {
-               case XMMS_PLAYBACK_STATUS_PLAY:
-                       strncpy(ptr->xmms2.status, "Playing", text_buffer_size - 1);
-                       break;
-               case XMMS_PLAYBACK_STATUS_PAUSE:
-                       strncpy(ptr->xmms2.status, "Paused", text_buffer_size - 1);
-                       break;
-               case XMMS_PLAYBACK_STATUS_STOP:
-                       strncpy(ptr->xmms2.status, "Stopped", text_buffer_size - 1);
-                       break;
-               default:
-                       strncpy(ptr->xmms2.status, "Unknown", text_buffer_size - 1);
-       }
+               switch (pb_state) {
+                       case XMMS_PLAYBACK_STATUS_PLAY:
+                               strncpy(ptr->xmms2.status, "Playing", text_buffer_size - 1);
+                               break;
+                       case XMMS_PLAYBACK_STATUS_PAUSE:
+                               strncpy(ptr->xmms2.status, "Paused", text_buffer_size - 1);
+                               break;
+                       case XMMS_PLAYBACK_STATUS_STOP:
+                               strncpy(ptr->xmms2.status, "Stopped", text_buffer_size - 1);
+                               break;
+                       default:
+                               strncpy(ptr->xmms2.status, "Unknown", text_buffer_size - 1);
+               }
        }
        return TRUE;
 }
@@ -316,7 +317,7 @@ void update_xmms2(void)
                if (xmmsc_io_want_out(xmms2_conn))
                        xmmsc_io_out_handle(xmms2_conn);
 
-               }
        }
+}