Test |BN_mod_exp_mont| with zero and even modulus too.

|BN_mod_exp_mont| should be tested the same way as the other variants,
especially since it is exported.

Change-Id: I8c05725289c0ebcce7aba7e666915c4c1a841c2b
Reviewed-on: https://boringssl-review.googlesource.com/8590
Reviewed-by: David Benjamin <davidben@google.com>
diff --git a/crypto/bn/bn_test.cc b/crypto/bn/bn_test.cc
index 446fd27..ccce995 100644
--- a/crypto/bn/bn_test.cc
+++ b/crypto/bn/bn_test.cc
@@ -1542,6 +1542,13 @@
   }
   ERR_clear_error();
 
+  if (BN_mod_exp_mont(a.get(), BN_value_one(), BN_value_one(), zero.get(), ctx,
+                      NULL)) {
+    fprintf(stderr, "BN_mod_exp_mont with zero modulus succeeded!\n");
+    return 0;
+  }
+  ERR_clear_error();
+
   if (BN_mod_exp_mont_consttime(a.get(), BN_value_one(), BN_value_one(),
                                 zero.get(), ctx, nullptr)) {
     fprintf(stderr, "BN_mod_exp_mont_consttime with zero modulus succeeded!\n");
@@ -1567,6 +1574,13 @@
   }
   ERR_clear_error();
 
+  if (BN_mod_exp_mont(a.get(), BN_value_one(), BN_value_one(), b.get(), ctx,
+                      NULL)) {
+    fprintf(stderr, "BN_mod_exp_mont with even modulus succeeded!\n");
+    return 0;
+  }
+  ERR_clear_error();
+
   if (BN_mod_exp_mont_consttime(a.get(), BN_value_one(), BN_value_one(),
                                 b.get(), ctx, nullptr)) {
     fprintf(stderr, "BN_mod_exp_mont_consttime with even modulus succeeded!\n");