diff options
| author | Rich Salz <rsalz@openssl.org> | 2016-08-30 13:31:18 -0400 |
|---|---|---|
| committer | Rich Salz <rsalz@openssl.org> | 2016-08-31 09:34:16 -0400 |
| commit | bde588df8a4d9f714cefe9cfd1a3931558de4c3c (patch) | |
| tree | 47c66a6223d6529132018e8a514a894dfedfa9fd /crypto/ui | |
| parent | 767ccc3b77cde82c46ab4af541663f6c80e538d3 (diff) | |
Code cleanup UI
Remove NULL check on parameter, and use NULL not ! on buffer.
Reviewed-by: Richard Levitte <levitte@openssl.org>
Diffstat (limited to 'crypto/ui')
| -rw-r--r-- | crypto/ui/ui_lib.c | 22 |
1 files changed, 2 insertions, 20 deletions
diff --git a/crypto/ui/ui_lib.c b/crypto/ui/ui_lib.c index 8992ae77c2..004aeb02b6 100644 --- a/crypto/ui/ui_lib.c +++ b/crypto/ui/ui_lib.c @@ -670,29 +670,21 @@ char *(*UI_method_get_prompt_constructor(UI_METHOD *method)) (UI *, enum UI_string_types UI_get_string_type(UI_STRING *uis) { - if (!uis) - return UIT_NONE; return uis->type; } int UI_get_input_flags(UI_STRING *uis) { - if (!uis) - return 0; return uis->input_flags; } const char *UI_get0_output_string(UI_STRING *uis) { - if (!uis) - return NULL; return uis->out_string; } const char *UI_get0_action_string(UI_STRING *uis) { - if (!uis) - return NULL; switch (uis->type) { case UIT_PROMPT: case UIT_BOOLEAN: @@ -704,8 +696,6 @@ const char *UI_get0_action_string(UI_STRING *uis) const char *UI_get0_result_string(UI_STRING *uis) { - if (!uis) - return NULL; switch (uis->type) { case UIT_PROMPT: case UIT_VERIFY: @@ -717,8 +707,6 @@ const char *UI_get0_result_string(UI_STRING *uis) const char *UI_get0_test_string(UI_STRING *uis) { - if (!uis) - return NULL; switch (uis->type) { case UIT_VERIFY: return uis->_.string_data.test_buf; @@ -729,8 +717,6 @@ const char *UI_get0_test_string(UI_STRING *uis) int UI_get_result_minsize(UI_STRING *uis) { - if (!uis) - return -1; switch (uis->type) { case UIT_PROMPT: case UIT_VERIFY: @@ -742,8 +728,6 @@ int UI_get_result_minsize(UI_STRING *uis) int UI_get_result_maxsize(UI_STRING *uis) { - if (!uis) - return -1; switch (uis->type) { case UIT_PROMPT: case UIT_VERIFY: @@ -759,8 +743,6 @@ int UI_set_result(UI *ui, UI_STRING *uis, const char *result) ui->flags &= ~UI_FLAG_REDOABLE; - if (!uis) - return -1; switch (uis->type) { case UIT_PROMPT: case UIT_VERIFY: @@ -789,7 +771,7 @@ int UI_set_result(UI *ui, UI_STRING *uis, const char *result) } } - if (!uis->result_buf) { + if (uis->result_buf == NULL) { UIerr(UI_F_UI_SET_RESULT, UI_R_NO_RESULT_BUFFER); return -1; } @@ -801,7 +783,7 @@ int UI_set_result(UI *ui, UI_STRING *uis, const char *result) { const char *p; - if (!uis->result_buf) { + if (uis->result_buf == NULL) { UIerr(UI_F_UI_SET_RESULT, UI_R_NO_RESULT_BUFFER); return -1; } |