diff options
author | Daiki Ueno <ueno@gnu.org> | 2020-08-16 18:23:24 +0200 |
---|---|---|
committer | Daiki Ueno <ueno@gnu.org> | 2020-08-16 18:23:24 +0200 |
commit | 458f84498d9e36fb746cc7a6306d859c30617577 (patch) | |
tree | 5c8dc678d294663445f8f9a4b83cd1d86b73a5d2 | |
parent | 64f347bccd94c66a5948d68c2707f8424e9d2720 (diff) | |
download | gnutls-tmp-crt-export2.tar.gz |
gnutls_x509_crt_export2: return 0 instead of the lengthtmp-crt-export2
This aligns the behavior to the documentation.
Signed-off-by: Daiki Ueno <ueno@gnu.org>
-rw-r--r-- | lib/x509/x509.c | 13 | ||||
-rw-r--r-- | tests/mini-x509-2.c | 2 |
2 files changed, 11 insertions, 4 deletions
diff --git a/lib/x509/x509.c b/lib/x509/x509.c index 2b68fe440e..c713f857a0 100644 --- a/lib/x509/x509.c +++ b/lib/x509/x509.c @@ -2997,10 +2997,15 @@ gnutls_x509_crt_export2(gnutls_x509_crt_t cert, if (!cert->modified && cert->der.size) { if (format == GNUTLS_X509_FMT_DER) return _gnutls_set_datum(out, cert->der.data, cert->der.size); - else - return _gnutls_fbase64_encode(PEM_X509_CERT2, cert->der.data, - cert->der.size, out); - + else { + int ret = _gnutls_fbase64_encode(PEM_X509_CERT2, + cert->der.data, + cert->der.size, + out); + if (ret < 0) + return ret; + return 0; + } } return _gnutls_x509_export_int2(cert->cert, format, PEM_X509_CERT2, diff --git a/tests/mini-x509-2.c b/tests/mini-x509-2.c index e336af8367..c1bc544e7d 100644 --- a/tests/mini-x509-2.c +++ b/tests/mini-x509-2.c @@ -296,6 +296,7 @@ void start(const char *prio) fail("gnutls_x509_crt_export2: %s\n", gnutls_strerror(ret)); exit(1); } + assert(ret == 0); gnutls_x509_crt_deinit(crt); if (scert.size != mcert->size || memcmp(scert.data, mcert->data, mcert->size) != 0) { @@ -331,6 +332,7 @@ void start(const char *prio) fail("gnutls_x509_crt_export2: %s\n", gnutls_strerror(ret)); exit(1); } + assert(ret == 0); gnutls_x509_crt_deinit(crt); if (ccert.size != mcert->size || memcmp(ccert.data, mcert->data, mcert->size) != 0) { |