diff libaudacious/beepctrl.c @ 1766:6cbb9360e8e2 trunk

[svn] - always zero-out stack variables, as stack memory may be dirty on x86 and friends

--- a/libaudacious/beepctrl.c	Sun Oct 01 00:58:32 2006 -0700
+++ b/libaudacious/beepctrl.c	Mon Oct 02 01:46:59 2006 -0700
@@ -61,6 +61,8 @@
 {
     gpointer data = NULL;
 
+    memset(pkt_hdr, '\0', sizeof(ServerPktHeader));
+
     if (read(fd, pkt_hdr, sizeof(ServerPktHeader)) ==
         sizeof(ServerPktHeader)) {
         if (pkt_hdr->data_length) {
@@ -81,6 +83,8 @@
     gpointer data;
     ServerPktHeader pkt_hdr;
 
+    memset(&pkt_hdr, '\0', sizeof(ServerPktHeader));
+
     data = remote_read_packet(fd, &pkt_hdr);
     if (data)
         g_free(data);
@@ -93,6 +97,8 @@
 {
     ClientPktHeader pkt_hdr;
 
+    memset(&pkt_hdr, '\0', sizeof(ClientPktHeader));
+
     pkt_hdr.version = XMMS_PROTOCOL_VERSION;
     pkt_hdr.command = command;
     pkt_hdr.data_length = data_length;
@@ -172,6 +178,8 @@
     gpointer data;
     gint fd;
 
+    memset(&pkt_hdr, '\0', sizeof(ServerPktHeader));
+
     if ((fd = xmms_connect_to_session(session)) == -1)
         return ret;
     remote_send_packet(fd, cmd, NULL, 0);
@@ -193,6 +201,8 @@
     gpointer data;
     gint fd, ret = 0;
 
+    memset(&pkt_hdr, '\0', sizeof(ServerPktHeader));
+
     if ((fd = xmms_connect_to_session(session)) == -1)
         return ret;
     remote_send_packet(fd, cmd, NULL, 0);
@@ -214,6 +224,8 @@
     gint fd;
     gfloat ret = 0.0;
 
+    memset(&pkt_hdr, '\0', sizeof(ServerPktHeader));
+
     if ((fd = xmms_connect_to_session(session)) == -1)
         return ret;
     remote_send_packet(fd, cmd, NULL, 0);
@@ -234,6 +246,8 @@
     gpointer data;
     gint fd;
 
+    memset(&pkt_hdr, '\0', sizeof(ServerPktHeader));
+
     if ((fd = xmms_connect_to_session(session)) == -1)
         return NULL;
     remote_send_packet(fd, cmd, NULL, 0);
@@ -250,6 +264,8 @@
     gpointer data;
     gint fd;
 
+    memset(&pkt_hdr, '\0', sizeof(ServerPktHeader));
+
     if ((fd = xmms_connect_to_session(session)) == -1)
         return NULL;
     remote_send_packet(fd, cmd, &pos, sizeof(guint32));
@@ -606,6 +622,8 @@
     gint fd;
     gpointer data;
 
+    memset(&pkt_hdr, '\0', sizeof(ServerPktHeader));
+
     if ((fd = xmms_connect_to_session(session)) == -1)
         return;
 
@@ -737,6 +755,8 @@
     gint fd, ret = 0;
     guint32 p = pos;
 
+    memset(&pkt_hdr, '\0', sizeof(ServerPktHeader));
+
     if ((fd = xmms_connect_to_session(session)) == -1)
         return ret;
     remote_send_packet(fd, CMD_GET_PLAYLIST_TIME, &p, sizeof(guint32));
@@ -757,6 +777,8 @@
     gint fd;
     gpointer data;
 
+    memset(&pkt_hdr, '\0', sizeof(ServerPktHeader));
+
     if ((fd = xmms_connect_to_session(session)) == -1)
         return;
     remote_send_packet(fd, CMD_GET_INFO, NULL, 0);
@@ -964,6 +986,8 @@
     gint fd, ret = 0;
     guint32 p = pos;
 
+    memset(&pkt_hdr, '\0', sizeof(ServerPktHeader));
+
     if ((fd = xmms_connect_to_session(session)) == -1)
         return ret;
     remote_send_packet(fd, CMD_PLAYQUEUE_IS_QUEUED, &p, sizeof(guint32));
@@ -985,6 +1009,8 @@
     gint fd, ret = 0;
     guint32 p = pos;
 
+    memset(&pkt_hdr, '\0', sizeof(ServerPktHeader));
+
     if ((fd = xmms_connect_to_session(session)) == -1)
         return ret;
     remote_send_packet(fd, CMD_PLAYQUEUE_GET_POS, &p, sizeof(guint32));
@@ -1006,6 +1032,8 @@
     gint fd, ret = 0;
     guint32 p = pos;
 
+    memset(&pkt_hdr, '\0', sizeof(ServerPktHeader));
+
     if ((fd = xmms_connect_to_session(session)) == -1)
         return ret;
     remote_send_packet(fd, CMD_PLAYQUEUE_GET_QPOS, &p, sizeof(guint32));
@@ -1026,6 +1054,8 @@
     gint fd;
     gpointer data;
 
+    memset(&pkt_hdr, '\0', sizeof(ServerPktHeader));
+
     if (preamp)
         *preamp = 0.0;
 
@@ -1065,6 +1095,8 @@
     gpointer data;
     gfloat val = 0.0;
 
+    memset(&pkt_hdr, '\0', sizeof(ServerPktHeader));
+
     if ((fd = xmms_connect_to_session(session)) == -1)
         return val;
     remote_send_packet(fd, CMD_GET_EQ_BAND, &band, sizeof(band));