Don't crash on a missing Subject in index.txt
authorMatt Caswell <matt@openssl.org>
Fri, 19 Jan 2018 14:48:45 +0000 (14:48 +0000)
committerMatt Caswell <matt@openssl.org>
Mon, 22 Jan 2018 10:07:41 +0000 (10:07 +0000)
An index.txt entry which has an empty Subject name field will cause ca
to crash. Therefore check it when we load it to make sure its not empty.

Fixes #5109

Reviewed-by: Rich Salz <rsalz@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/5114)

apps/ca.c

index 551d0aa2f8c3b2d2bf3ab01caaecdff880634bd3..9b1d9bdaac8f9003695e788ad1c5632b6493dace 100644 (file)
--- a/apps/ca.c
+++ b/apps/ca.c
@@ -663,6 +663,10 @@ end_of_options:
                 goto end;
             }
         }
+        if (pp[DB_name][0] == '\0') {
+            BIO_printf(bio_err, "entry %d: bad Subject\n", i + 1);
+            goto end;
+        }
     }
     if (verbose) {
         TXT_DB_write(bio_out, db->db);