X-Git-Url: https://gerrit.automotivelinux.org/gerrit/gitweb?a=blobdiff_plain;f=src%2Fwgtpkg-certs.c;h=0db72eb0744c5349c95539e5261addc7780e5aa8;hb=3a6e947bef1b2942e24d2fdee1a76dbf3305b508;hp=da5029cba29f9ec756c37b98ff1a2c5d69bccab1;hpb=41e3ea4dbae634ca13941d134e990d953b741c4b;p=src%2Fapp-framework-main.git

diff --git a/src/wgtpkg-certs.c b/src/wgtpkg-certs.c
index da5029c..0db72eb 100644
--- a/src/wgtpkg-certs.c
+++ b/src/wgtpkg-certs.c
@@ -1,5 +1,7 @@
 /*
- Copyright 2015 IoT.bzh
+ Copyright 2015, 2016, 2017 IoT.bzh
+
+ author: José Bollo <jose.bollo@iot.bzh>
 
  Licensed under the Apache License, Version 2.0 (the "License");
  you may not use this file except in compliance with the License.
@@ -18,10 +20,11 @@
 #include <openssl/x509.h>
 
 #include "verbose.h"
-#include "wgtpkg.h"
+#include "wgtpkg-certs.h"
+#include "wgtpkg-base64.h"
 
 struct x509l {
-	int count;
+	unsigned count;
 	X509 **certs;
 };
 
@@ -29,7 +32,8 @@ static struct x509l certificates = { .count = 0, .certs = NULL };
 
 static int add_certificate_x509(X509 *x)
 {
-	X509 **p = realloc(certificates.certs, (certificates.count + 1) * sizeof(X509*));
+	X509 **p = realloc(certificates.certs,
+			(certificates.count + 1) * sizeof(X509*));
 	if (!p) {
 		ERROR("reallocation failed for certificate");
 		return -1;
@@ -39,7 +43,7 @@ static int add_certificate_x509(X509 *x)
 	return 0;
 }
 
-static int add_certificate_bin(const char *bin, int len)
+static int add_certificate_bin(const char *bin, size_t len)
 {
 	int rc;
 	const char *b, *e;
@@ -63,12 +67,15 @@ static int add_certificate_bin(const char *bin, int len)
 int add_certificate_b64(const char *b64)
 {
 	char *d;
-	int l = base64dec(b64, &d);
-	if (l > 0) {
-		l = add_certificate_bin(d, l);
+	ssize_t l = base64dec(b64, &d);
+	int rc;
+	if (l < 0)
+		rc = -1;
+	else {
+		rc = add_certificate_bin(d, (size_t)l);
 		free(d);
 	}
-	return l;
+	return rc;
 }
 
 void clear_certificates()