diff options
| author | Nikos Mavrogiannopoulos <nmav@gnutls.org> | 2017-09-23 11:09:00 +0200 |
|---|---|---|
| committer | Nikos Mavrogiannopoulos <nmav@gnutls.org> | 2017-09-24 10:29:52 +0000 |
| commit | f6ee63173f892cb324bffb4da173334a1c2fe8d0 (patch) | |
| tree | 06451728b45453ab6a2abb922679a26956efe7c5 | |
| parent | 38e625b48faad3bf803d187bd86a96b9bda46e6a (diff) | |
| download | gnutls-f6ee63173f892cb324bffb4da173334a1c2fe8d0.tar.gz | |
certtool: use assert to protect var access
The code correctly uses the variables, but the assert ensures
that static analyzers follow the intended paths too.
Signed-off-by: Nikos Mavrogiannopoulos <nmav@gnutls.org>
| -rw-r--r-- | src/certtool.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/certtool.c b/src/certtool.c index e82203fa6e..55f5184e48 100644 --- a/src/certtool.c +++ b/src/certtool.c @@ -2951,7 +2951,7 @@ void generate_pkcs12(common_info_st * cinfo) app_exit(1); } - assert(crts[i] != NULL); + assert(crts != NULL && crts[i] != NULL); result = gnutls_pkcs12_bag_set_crt(bag, crts[i]); if (result < 0) { fprintf(stderr, "set_crt[%d]: %s\n", i, @@ -3086,6 +3086,8 @@ void generate_pkcs12(common_info_st * cinfo) app_exit(1); } + assert(keys != NULL && keys[i] != NULL); + size = lbuffer_size; result = gnutls_x509_privkey_export_pkcs8(keys[i], |