cleanup logging, need to be careful with SECURE and DEBUG

src/interface.c

@@ -156,18 +156,32 @@ void openAInterface(int fd, struct sockaddr * addr) {
 		SECURE("interface %i: opened from ",i);
 		switch(addr->sa_family) {
 		case AF_INET:
-			SECURE("%s\n",inet_ntoa(
+			{
+				char * host = inet_ntoa(
 					((struct sockaddr_in *)addr)->
-					sin_addr));
+					sin_addr);
+				if(host) { 
+					SECURE("%s\n",host);
+				}
+				else {
+					SECURE("error getting ipv4 address\n");
+				}
+			}
 			break;
 #ifdef HAVE_IPV6
 		case AF_INET6:
 			{
 				char host[INET6_ADDRSTRLEN+1];
 				memset(host,0,INET6_ADDRSTRLEN+1);
-				SECURE("%s\n",inet_ntop(AF_INET6,(void *)
+				if(inet_ntop(AF_INET6,(void *)
 					&(((struct sockaddr_in6 *)addr)->
-					sin6_addr),host,INET6_ADDRSTRLEN));
+					sin6_addr),host,INET6_ADDRSTRLEN)) 
+				{
+					SECURE("%s\n",host);
+				}
+				else {
+					SECURE("error getting ipv6 address\n");
+				}
 			}
 			break;
 #endif

src/log.h

@@ -27,22 +27,16 @@
 
 extern int logLevel;
 
-#define ERROR(x, arg...) do { \
-		myfprintf(stderr, x , ##arg); \
-	} while(0);
+#define ERROR(x, arg...) myfprintf(stderr, x , ##arg)
 
-#define LOG(x, arg...) do { \
-		myfprintf(stdout, x , ##arg); \
-	} while(0); 
+#define LOG(x, arg...) myfprintf(stdout, x , ##arg)
 
-#define SECURE(x, arg...) do { \
-		if(logLevel>=LOG_LEVEL_SECURE) myfprintf(stdout, x , ##arg); \
-	} while(0);
+#define SECURE(x, arg...) if(logLevel>=LOG_LEVEL_SECURE) \
+				myfprintf(stdout, x , ##arg)
 		
 
-#define DEBUG(x, arg...) do { \
-		if(logLevel>=LOG_LEVEL_DEBUG) myfprintf(stdout, x , ##arg); \
-	} while(0);
+#define DEBUG(x, arg...) if(logLevel>=LOG_LEVEL_DEBUG) \
+				myfprintf(stdout, x , ##arg)
 
 void initLog();
 

src/playerData.c

@@ -103,7 +103,7 @@ void initPlayerData() {
 	playerData_pd->playerControl.queueState = PLAYER_QUEUE_BLANK;
 	playerData_pd->playerControl.queueLockState = PLAYER_QUEUE_UNLOCKED;
 	playerData_pd->playerControl.seek = 0;
-	memset(playerData_pd->playerControl.file,0,MAXPATHLEN);
+	memset(playerData_pd->playerControl.file,0,MAXPATHLEN+1);
 	playerData_pd->playerControl.crossFade = crossfade;
 	playerData_pd->playerControl.softwareVolume = 100;
 	playerData_pd->playerControl.totalPlayTime = 0;
@@ -113,7 +113,7 @@ void initPlayerData() {
 	playerData_pd->decoderControl.state = DECODE_STATE_STOP;
 	playerData_pd->decoderControl.seek = 0;
 	playerData_pd->decoderControl.error = DECODE_ERROR_NOERROR;
-	memset(playerData_pd->decoderControl.file,0,MAXPATHLEN);
+	memset(playerData_pd->decoderControl.file,0,MAXPATHLEN+1);
 }
 
 PlayerData * getPlayerData() {