X-Git-Url: https://git.octo.it/?a=blobdiff_plain;f=src%2Fmain.c;h=010572d5b5153490e81aa2a2975ffe90a6243d81;hb=7046644fa5f9aac043ac53e3ab9f7f7c56c45b05;hp=169e39c86fe39b114d3253ae8261931b7dca826a;hpb=c4ad812587c29fea8fbbb6179b762d6b31f15e7a;p=routeros-api.git

diff --git a/src/main.c b/src/main.c
index 169e39c..010572d 100644
--- a/src/main.c
+++ b/src/main.c
@@ -52,6 +52,10 @@
 # define ros_debug(...) /**/
 #endif
 
+#if !__GNUC__
+# define __attribute__(x) /**/
+#endif
+
 /* FIXME */
 char *strdup (const char *);
 
@@ -112,7 +116,7 @@ static int read_exact (int fd, void *buffer, size_t buffer_size) /* {{{ */
 				return (status);
 		}
 
-		assert (status <= want_bytes);
+		assert (((size_t) status) <= want_bytes);
 		have_bytes += status;
 		buffer_ptr += status;
 	}
@@ -344,6 +348,12 @@ static int send_command (ros_connection_t *c, /* {{{ */
 	size_t i;
 	int status;
 
+	assert (c != NULL);
+	assert (command != NULL);
+
+	if ((args == NULL) && (args_num > 0))
+		return (EINVAL);
+
 	/* FIXME: For debugging only */
 	memset (buffer, 0, sizeof (buffer));
 
@@ -389,7 +399,7 @@ static int send_command (ros_connection_t *c, /* {{{ */
 			else
 				return (errno);
 		}
-		assert (bytes_written <= buffer_size);
+		assert (((size_t) bytes_written) <= buffer_size);
 
 		buffer_ptr += bytes_written;
 		buffer_size -= bytes_written;
@@ -408,6 +418,8 @@ static int read_word (ros_connection_t *c, /* {{{ */
 	if ((buffer == NULL) || (*buffer_size < 1))
 		return (EINVAL);
 
+	assert (c != NULL);
+
 	/* read one byte from the socket */
 	status = read_exact (c->fd, word_length, 1);
 	if (status != 0)
@@ -554,6 +566,9 @@ static ros_reply_t *receive_reply (ros_connection_t *c) /* {{{ */
 	ros_reply_t *head;
 	ros_reply_t *tail;
 
+	if (c == NULL)
+		return (NULL);
+
 	head = NULL;
 	tail = NULL;
 
@@ -635,8 +650,9 @@ static int create_socket (const char *node, const char *service) /* {{{ */
 	return (-1);
 } /* }}} int create_socket */
 
-static int login2_handler (ros_connection_t *c, const ros_reply_t *r, /* {{{ */
-		void *user_data)
+static int login2_handler (__attribute__((unused)) ros_connection_t *c, /* {{{ */
+		const ros_reply_t *r,
+		__attribute__((unused)) void *user_data)
 {
 	if (r == NULL)
 		return (EINVAL);
@@ -840,6 +856,9 @@ int ros_query (ros_connection_t *c, /* {{{ */
 	int status;
 	ros_reply_t *r;
 
+	if ((c == NULL) || (command == NULL) || (handler == NULL))
+		return (EINVAL);
+
 	status = send_command (c, command, args_num, args);
 	if (status != 0)
 		return (status);