Fix wrong parser variable in directoryName trailing data check

ParseGeneralName uses two parsers: `parser` for the outer
GeneralName TLV, and `name_parser` for the inner directoryName
content. The trailing-data check on line 155 calls
parser.HasMore() instead of name_parser.HasMore().

Since parser is already fully consumed by ReadTagAndValue() at
line 116, parser.HasMore() always returns false. Any bytes after
the SEQUENCE inside a directoryName are silently accepted.

Bug: None
Change-Id: I5cd8d4de2a43c88e4959ec43ef85624de297401c
Reviewed-on: https://boringssl-review.googlesource.com/c/boringssl/+/92927
Reviewed-by: David Benjamin <davidben@google.com>
Commit-Queue: David Benjamin <davidben@google.com>
Reviewed-by: Lily Chen <chlily@google.com>
Reviewed-by: Adam Langley <agl@google.com>

pki/general_names.cc

diff --git a/pki/general_names.cc b/pki/general_names.cc
index d4ab234..c239946 100644
--- a/pki/general_names.cc
+++ b/pki/general_names.cc
@@ -152,7 +152,7 @@
     der::Parser name_parser(value);
     der::Input name_value;
     if (!name_parser.ReadTag(CBS_ASN1_SEQUENCE, &name_value) ||
-        parser.HasMore()) {
+        name_parser.HasMore()) {
       return false;
     }
     subtrees->directory_names.push_back(name_value);