X-Git-Url: https://sigrok.org/gitweb/?a=blobdiff_plain;f=output%2Fbinary.c;h=bbf49c705e6bf4f7b10b18bd8fbdef3e7a066706;hb=3f8d59721634e1d69251078c254caef81effc62c;hp=039d10bcea3d09428fafc29cc47ea2c39466ffa6;hpb=0da5b6a9f5a119ab485ed3dba08aeb4a3ad2c8f4;p=libsigrok.git

diff --git a/output/binary.c b/output/binary.c
index 039d10bc..bbf49c70 100644
--- a/output/binary.c
+++ b/output/binary.c
@@ -23,7 +23,6 @@
 #include <glib.h>
 #include "sigrok.h"
 #include "sigrok-internal.h"
-#include "config.h"
 
 static int data(struct sr_output *o, const char *data_in, uint64_t length_in,
 		char **data_out, uint64_t *length_out)
@@ -34,22 +33,22 @@ static int data(struct sr_output *o, const char *data_in, uint64_t length_in,
 	(void)o;
 
 	if (!data_in) {
-		sr_warn("binary output: %s: data_in was NULL", __func__);
-		return SR_ERR;
+		sr_err("binary out: %s: data_in was NULL", __func__);
+		return SR_ERR_ARG;
 	}
 
 	if (!length_out) {
-		sr_warn("binary output: %s: length_out was NULL", __func__);
-		return SR_ERR;
+		sr_err("binary out: %s: length_out was NULL", __func__);
+		return SR_ERR_ARG;
 	}
 
 	if (length_in == 0) {
-		sr_warn("binary output: %s: length_in was 0", __func__);
-		return SR_ERR;
+		sr_err("binary out: %s: length_in was 0", __func__);
+		return SR_ERR_ARG;
 	}
 
-	if (!(outbuf = calloc(1, length_in))) {
-		sr_warn("binary output: %s: outbuf calloc failed", __func__);
+	if (!(outbuf = g_try_malloc0(length_in))) {
+		sr_err("binary out: %s: outbuf malloc failed", __func__);
 		return SR_ERR_MALLOC;
 	}
 
@@ -60,7 +59,7 @@ static int data(struct sr_output *o, const char *data_in, uint64_t length_in,
 	return SR_OK;
 }
 
-struct sr_output_format output_binary = {
+SR_PRIV struct sr_output_format output_binary = {
 	.id = "binary",
 	.description = "Raw binary",
 	.df_type = SR_DF_LOGIC,