diff options
| author | Richard Levitte <levitte@openssl.org> | 2016-12-07 16:36:44 +0100 |
|---|---|---|
| committer | Richard Levitte <levitte@openssl.org> | 2016-12-08 00:06:43 +0100 |
| commit | 0a687ab0a92d2d68289364a6e232028c229f44bb (patch) | |
| tree | 840d4e1949a942b758fc9fd69ad7fef146ff0fb0 /crypto/ui/ui_err.c | |
| parent | 7cb58c0ffa7203e8ad4d39c912a1ab5e21a12b85 (diff) | |
UI_process() didn't generate errors
Since there are many parts of UI_process() that can go wrong, it isn't
very helpful to only return -1 with no further explanation. With this
change, the error message will at least show which part went wrong.
Reviewed-by: Rich Salz <rsalz@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/2037)
Diffstat (limited to 'crypto/ui/ui_err.c')
| -rw-r--r-- | crypto/ui/ui_err.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/crypto/ui/ui_err.c b/crypto/ui/ui_err.c index ef03815ea2..b89f9aebb5 100644 --- a/crypto/ui/ui_err.c +++ b/crypto/ui/ui_err.c @@ -30,6 +30,7 @@ static ERR_STRING_DATA UI_str_functs[] = { {ERR_FUNC(UI_F_UI_DUP_VERIFY_STRING), "UI_dup_verify_string"}, {ERR_FUNC(UI_F_UI_GET0_RESULT), "UI_get0_result"}, {ERR_FUNC(UI_F_UI_NEW_METHOD), "UI_new_method"}, + {ERR_FUNC(UI_F_UI_PROCESS), "UI_process"}, {ERR_FUNC(UI_F_UI_SET_RESULT), "UI_set_result"}, {0, NULL} }; @@ -40,6 +41,7 @@ static ERR_STRING_DATA UI_str_reasons[] = { {ERR_REASON(UI_R_INDEX_TOO_LARGE), "index too large"}, {ERR_REASON(UI_R_INDEX_TOO_SMALL), "index too small"}, {ERR_REASON(UI_R_NO_RESULT_BUFFER), "no result buffer"}, + {ERR_REASON(UI_R_PROCESSING_ERROR), "processing error"}, {ERR_REASON(UI_R_RESULT_TOO_LARGE), "result too large"}, {ERR_REASON(UI_R_RESULT_TOO_SMALL), "result too small"}, {ERR_REASON(UI_R_UNKNOWN_CONTROL_COMMAND), "unknown control command"}, |