Only a single CRL could be loaded at once. This has been fixed.

Closes #1442. Thanks to charlemagnelasse.
eclipse · Oct 2, 2019 · cfacd96 · cfacd96
1 parent c471dfb
commit cfacd96
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/ChangeLog.txt b/ChangeLog.txt
@@ -5,6 +5,8 @@ Broker:
  Closes #1437.
 - Fix subscription topics being limited to 200 characters instead of 200
  hierarchy levels. Closes #1441.
+- Only a single CRL could be loaded at once. This has been fixed.
+ Closes #1442.
 
 Client library:
 - Fix publish properties not being passed to on_message_v5 callback for QoS 2

diff --git a/src/net.c b/src/net.c
@@ -426,7 +426,7 @@ int net__load_crl_file(struct mosquitto__listener *listener)
  }
  lookup = X509_STORE_add_lookup(store, X509_LOOKUP_file());
  rc = X509_load_crl_file(lookup, listener->crlfile, X509_FILETYPE_PEM);
- if(rc != 1){
+ if(rc < 1){
  log__printf(NULL, MOSQ_LOG_ERR, "Error: Unable to load certificate revocation file \"%s\". Check crlfile.", listener->crlfile);
  net__print_error(MOSQ_LOG_ERR, "Error: %s");
  return 1;