tree 4d690366c859f04bd4d03c33432a9171612587e4
parent ee0716f386190d2bfb105a0db7400df4915773a1
author David Benjamin <davidben@google.com> 1574889748 -0500
committer CQ bot account: commit-bot@chromium.org <commit-bot@chromium.org> 1575305612 +0000

Give ERR_error_string_n a return value for convenience.

ERR_error_string_n needs to be called in a separate statement, compared
to ERR_error_string(err, NULL), which returns a buffer and is very
convenient to use in an expression. This is unfortunate because it is
not thread-safe.

Give ERR_error_string_n a return value to align. Fixing callers still
requires allocating a buffer somewhere, but the rest of the expression
can remain relatively unperturbed.

Change-Id: I273c9df97f0bb113cdc57cf3896c42195910c67a
Reviewed-on: https://boringssl-review.googlesource.com/c/boringssl/+/38964
Reviewed-by: Steven Valdez <svaldez@google.com>
Commit-Queue: David Benjamin <davidben@google.com>