silence several compiler warnings

6 files changed, 16 insertions, 10 deletions

diff --git a/src/client.c b/src/client.c
index ac1045a..d4a4a10 100644
--- a/src/client.c
+++ b/src/client.c
@@ -318,7 +318,7 @@ static int start_listen(struct mux_client *client)
         count = device_get_list(devs);
 
         // going to need a larger buffer for many devices
-        int needed_buffer = count * (sizeof(struct usbmuxd_device_record) + sizeof(struct usbmuxd_header)) + REPLY_BUF_SIZE;
+        uint32_t needed_buffer = count * (sizeof(struct usbmuxd_device_record) + sizeof(struct usbmuxd_header)) + REPLY_BUF_SIZE;
         if(client->ob_capacity < needed_buffer) {
                 usbmuxd_log(LL_DEBUG, "Enlarging client %d reply buffer %d -> %d to make space for device notifications", client->fd, client->ob_capacity, needed_buffer);
                 client->ob_buf = realloc(client->ob_buf, needed_buffer);
@@ -464,7 +464,7 @@ static int client_command(struct mux_client *client, struct usbmuxd_header *hdr)
 
 static void process_send(struct mux_client *client)
 {
-        int res;
+        uint32_t res;
         if(!client->ob_size) {
                 usbmuxd_log(LL_WARNING, "Client %d OUT process but nothing to send?", client->fd);
                 client->events &= ~POLLOUT;
diff --git a/src/device.c b/src/device.c
index 8c786a7..91712be 100644
--- a/src/device.c
+++ b/src/device.c
@@ -114,7 +114,7 @@ struct mux_device
 
 static struct collection device_list;
 
-uint64_t mstime64(void)
+static uint64_t mstime64(void)
 {
         struct timeval tv;
         gettimeofday(&tv, NULL);
diff --git a/src/log.c b/src/log.c
index 1973257..2d22e94 100644
--- a/src/log.c
+++ b/src/log.c
@@ -33,7 +33,7 @@ Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301  USA
 
 #include "log.h"
 
-int log_level = LL_WARNING;
+unsigned int log_level = LL_WARNING;
 
 int log_syslog = 0;
 
diff --git a/src/log.h b/src/log.h
index eeefa41..1fb77ee 100644
--- a/src/log.h
+++ b/src/log.h
@@ -33,7 +33,7 @@ enum loglevel {
         LL_FLOOD,
 };
 
-extern int log_level;
+extern unsigned int log_level;
 
 void log_enable_syslog();
 void log_disable_syslog();
diff --git a/src/main.c b/src/main.c
index de5e5cc..32c6a2b 100644
--- a/src/main.c
+++ b/src/main.c
@@ -64,7 +64,7 @@ static int daemon_pipe;
 
 static int report_to_parent = 0;
 
-int create_socket(void) {
+static int create_socket(void) {
         struct sockaddr_un bind_addr;
         int listenfd;
 
@@ -98,7 +98,7 @@ int create_socket(void) {
         return listenfd;
 }
 
-void handle_signal(int sig)
+static void handle_signal(int sig)
 {
         if (sig != SIGUSR1 && sig != SIGUSR2) {
                 usbmuxd_log(LL_NOTICE,"Caught signal %d, exiting", sig);
@@ -124,7 +124,7 @@ void handle_signal(int sig)
         }
 }
 
-void set_signal_handlers(void)
+static void set_signal_handlers(void)
 {
         struct sigaction sa;
         sigset_t set;
@@ -162,7 +162,7 @@ static int ppoll(struct pollfd *fds, nfds_t nfds, const struct timespec *timeout
 }
 #endif
 
-int main_loop(int listenfd)
+static int main_loop(int listenfd)
 {
         int to, cnt, i, dto;
         struct fdlist pollfds;
@@ -519,7 +519,7 @@ int main(int argc, char *argv[])
                 goto terminate;
         }
         sprintf(pids, "%d", getpid());
-        if ((res = write(lfd, pids, strlen(pids))) != strlen(pids)) {
+        if ((size_t)(res = write(lfd, pids, strlen(pids))) != strlen(pids)) {
                 usbmuxd_log(LL_FATAL, "Could not write pidfile!");
                 if(res >= 0)
                         res = -2;
diff --git a/src/usb-linux.c b/src/usb-linux.c
index 334d967..f937de0 100644
--- a/src/usb-linux.c
+++ b/src/usb-linux.c
@@ -121,6 +121,9 @@ static void tx_callback(struct libusb_transfer *xfer)
                                 usbmuxd_log(LL_ERROR, "TX transfer overflow for device %d-%d", dev->bus, dev->address);
                                 break;
                         // and nothing happens (this never gets called) if the device is freed after a disconnect! (bad)
+                        default:
+                                // this should never be reached.
+                                break;
                 }
                 // we can't usb_disconnect here due to a deadlock, so instead mark it as dead and reap it after processing events
                 // we'll do device_remove there too
@@ -190,6 +193,9 @@ static void rx_callback(struct libusb_transfer *xfer)
                                 usbmuxd_log(LL_ERROR, "RX transfer overflow for device %d-%d", dev->bus, dev->address);
                                 break;
                         // and nothing happens (this never gets called) if the device is freed after a disconnect! (bad)
+                        default:
+                                // this should never be reached.
+                                break;
                 }
                 free(xfer->buffer);
                 dev->rx_xfer = NULL;