X-Git-Url: https://git.octo.it/?a=blobdiff_plain;f=src%2Fmain.c;h=31e1f8b73f1de60a216805df140b0af5989eb53d;hb=b23183ce327183d252dd9779d90f09bb32d8d777;hp=d35c30c1e5687da44979157ece95c806aac9b347;hpb=8c609b7cc4cf4adb1d256ee34d07f1ccb385ba03;p=routeros-api.git

diff --git a/src/main.c b/src/main.c
index d35c30c..31e1f8b 100644
--- a/src/main.c
+++ b/src/main.c
@@ -2,18 +2,17 @@
  * librouteros - src/main.c
  * Copyright (C) 2009  Florian octo Forster
  *
- * This program is free software; you can redistribute it and/or modify it
- * under the terms of the GNU General Public License as published by the
- * Free Software Foundation; only version 2 of the License is applicable.
+ * Permission to use, copy, modify, and/or distribute this software for any
+ * purpose with or without fee is hereby granted, provided that the above
+ * copyright notice and this permission notice appear in all copies.
  *
- * This program is distributed in the hope that it will be useful, but
- * WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
- * General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License along
- * with this program; if not, write to the Free Software Foundation, Inc.,
- * 51 Franklin St, Fifth Floor, Boston, MA  02110-1301 USA
+ * THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES WITH
+ * REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF MERCHANTABILITY
+ * AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR ANY SPECIAL, DIRECT,
+ * INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES WHATSOEVER RESULTING FROM
+ * LOSS OF USE, DATA OR PROFITS, WHETHER IN AN ACTION OF CONTRACT, NEGLIGENCE OR
+ * OTHER TORTIOUS ACTION, ARISING OUT OF OR IN CONNECTION WITH THE USE OR
+ * PERFORMANCE OF THIS SOFTWARE.
  *
  * Authors:
  *   Florian octo Forster <octo at verplant.org>
@@ -215,6 +214,7 @@ static void reply_free (ros_reply_t *r) /* {{{ */
 		free (r->values[i]);
 	}
 
+	free (r->status);
 	free (r->keys);
 	free (r->values);
 
@@ -348,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));
 
@@ -412,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)
@@ -558,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;
 
@@ -631,11 +642,11 @@ static int create_socket (const char *node, const char *service) /* {{{ */
 			continue;
 		}
 
+		freeaddrinfo (ai_list);
 		return (fd);
 	}
 
 	freeaddrinfo (ai_list);
-
 	return (-1);
 } /* }}} int create_socket */
 
@@ -845,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);