X-Git-Url: http://sigrok.org/gitweb/?a=blobdiff_plain;f=src%2Fhardware%2Fbeaglelogic%2Fbeaglelogic_tcp.c;h=9cf78bbf854630d7b9f9431fd269a81fddb04ed7;hb=d6d87fa2a28edbfa0f1c66ef4b03c9cd8bda095e;hp=dfd8c624d6f946652e1d2c12c34531963961a8d9;hpb=a31010b3e4222c4d1c404c1f3d0a7c2d09e0cfd5;p=libsigrok.git

diff --git a/src/hardware/beaglelogic/beaglelogic_tcp.c b/src/hardware/beaglelogic/beaglelogic_tcp.c
index dfd8c624..9cf78bbf 100644
--- a/src/hardware/beaglelogic/beaglelogic_tcp.c
+++ b/src/hardware/beaglelogic/beaglelogic_tcp.c
@@ -107,15 +107,17 @@ static int beaglelogic_tcp_send_cmd(struct dev_context *devc,
 
 	if (out < 0) {
 		sr_err("Send error: %s", g_strerror(errno));
+		g_free(buf);
 		return SR_ERR;
 	}
 
 	if (out < (int)strlen(buf)) {
-		sr_dbg("Only sent %d/%lu bytes of command: '%s'.", out,
+		sr_dbg("Only sent %d/%zu bytes of command: '%s'.", out,
 		       strlen(buf), buf);
 	}
 
 	sr_spew("Sent command: '%s'.", buf);
+
 	g_free(buf);
 
 	return SR_OK;
@@ -152,14 +154,14 @@ SR_PRIV int beaglelogic_tcp_drain(struct dev_context *devc)
 
 	do {
 		ret = select(devc->socket + 1, &rset, NULL, NULL, &tv);
-		if (ret > 0) {
+		if (ret > 0)
 			len += beaglelogic_tcp_read_data(devc, buf, 1024);
-		}
 	} while (ret > 0);
 
 	sr_spew("Drained %d bytes of data.", len);
 
 	g_free(buf);
+
 	return SR_OK;
 }
 
@@ -170,10 +172,10 @@ static int beaglelogic_tcp_get_string(struct dev_context *devc, const char *cmd,
 	int len;
 	gint64 timeout;
 
+	*tcp_resp = NULL;
 	if (cmd) {
-		if (beaglelogic_tcp_send_cmd(devc, cmd) != SR_OK) {
+		if (beaglelogic_tcp_send_cmd(devc, cmd) != SR_OK)
 			return SR_ERR;
-		}
 	}
 
 	timeout = g_get_monotonic_time() + devc->read_timeout;
@@ -185,9 +187,8 @@ static int beaglelogic_tcp_get_string(struct dev_context *devc, const char *cmd,
 		return SR_ERR;
 	}
 
-	if (len > 0) {
+	if (len > 0)
 		g_string_set_size(response, len);
-	}
 
 	if (g_get_monotonic_time() > timeout) {
 		sr_err("Timed out waiting for response.");
@@ -207,6 +208,7 @@ static int beaglelogic_tcp_get_string(struct dev_context *devc, const char *cmd,
 		response->str, response->len);
 
 	*tcp_resp = g_string_free(response, FALSE);
+
 	return SR_OK;
 }
 
@@ -242,6 +244,7 @@ SR_PRIV int beaglelogic_tcp_detect(struct dev_context *devc)
 		ret = SR_ERR;
 
 	g_free(resp);
+
 	return ret;
 }
 
@@ -277,15 +280,20 @@ static int beaglelogic_set_buffersize(struct dev_context *devc)
 		ret = SR_ERR;
 
 	g_free(resp);
+
 	return ret;
 }
 
 static int beaglelogic_get_samplerate(struct dev_context *devc)
 {
 	int arg, err;
+
 	err = beaglelogic_tcp_get_int(devc, "samplerate", &arg);
+	if (err)
+		return err;
+
 	devc->cur_samplerate = arg;
-	return err;
+	return SR_OK;
 }
 
 static int beaglelogic_set_samplerate(struct dev_context *devc)
@@ -302,6 +310,7 @@ static int beaglelogic_set_samplerate(struct dev_context *devc)
 		ret = SR_ERR;
 
 	g_free(resp);
+
 	return ret;
 }
 
@@ -324,6 +333,7 @@ static int beaglelogic_set_sampleunit(struct dev_context *devc)
 		ret = SR_ERR;
 
 	g_free(resp);
+
 	return ret;
 }
 
@@ -346,18 +356,21 @@ static int beaglelogic_set_triggerflags(struct dev_context *devc)
 		ret = SR_ERR;
 
 	g_free(resp);
+
 	return ret;
 }
 
 static int beaglelogic_get_lasterror(struct dev_context *devc)
 {
 	devc->last_error = 0;
+
 	return SR_OK;
 }
 
 static int beaglelogic_start(struct dev_context *devc)
 {
 	beaglelogic_tcp_drain(devc);
+
 	return beaglelogic_tcp_send_cmd(devc, "get");
 }
 
@@ -385,12 +398,14 @@ static int beaglelogic_set_bufunitsize(struct dev_context *devc)
 		ret = SR_ERR;
 
 	g_free(resp);
+
 	return ret;
 }
 
 static int dummy(struct dev_context *devc)
 {
 	(void)devc;
+
 	return SR_ERR_NA;
 }