Confirmed bug in apps/req.c.

When executed w/-batch argument, cannot retry input from user which fails min/max length check. Must instead fail.

Proposed patch

--- a/apps/req.c
+++ b/apps/req.c
@@ -1477,7 +1477,13 @@
 #ifdef CHARSET_EBCDIC
     ebcdic2ascii(buf, buf, i);
 #endif
-    if(!req_check_len(i, n_min, n_max)) goto start;
+    if (!req_check_len(i, n_min, n_max))
+        {
+        if (!batch)
+            goto start;
+        else
+            return 0;
+        }
     if (!X509_NAME_add_entry_by_NID(n,nid, chtype,
                 (unsigned char *) buf, -1,-1,mval)) goto err;
     ret=1;
@@ -1536,7 +1542,13 @@
 #ifdef CHARSET_EBCDIC
     ebcdic2ascii(buf, buf, i);
 #endif
-    if(!req_check_len(i, n_min, n_max)) goto start;
+    if (!req_check_len(i, n_min, n_max))
+        {
+        if (!batch)
+            goto start;
+        else
+            return 0;
+        }

     if(!X509_REQ_add1_attr_by_NID(req, nid, chtype,
                     (unsigned char *)buf, -1)) {




--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to