tree 301876b776b0d64493d76d12d98b3ee173ec1909
parent 0c9ac2e7bf6f5d611ee5ff6f47eb98bcc2855a5a
author Steven Valdez <svaldez@google.com> 1525367712 -0400
committer CQ bot account: commit-bot@chromium.org <commit-bot@chromium.org> 1525369243 +0000

Fix bugs in X509_NAME_add_entry.

|set| should be evaluated to determine whether to insert/append before
it is reused as a temporary variable.

When incrementing the |set| of X509_NAME_ENTRY, the inserted entry
should not be incremented.

Thanks to Ingo Schwarze for extensive debugging and the initial
fix.

(Imported from upstream bbf27cd58337116c57a1c942153330ff83d5540a)

Change-Id: Ib45d92fc6d52d7490b01d3c475eafc42dd6ef721
Reviewed-on: https://boringssl-review.googlesource.com/28005
Commit-Queue: Steven Valdez <svaldez@google.com>
CQ-Verified: CQ bot account: commit-bot@chromium.org <commit-bot@chromium.org>
Reviewed-by: David Benjamin <davidben@google.com>