removes unreachable code with a single return point

This commit is contained in:
Moisés Guimarães
2017-11-07 10:39:55 -03:00
parent 90c5a64547
commit 7674a24972

View File

@@ -2881,38 +2881,44 @@ static void TLSX_PointFormat_FreeAll(PointFormat* list, void* heap)
static int TLSX_SupportedCurve_Append(SupportedCurve* list, word16 name, static int TLSX_SupportedCurve_Append(SupportedCurve* list, word16 name,
void* heap) void* heap)
{ {
if (list == NULL) int ret = BAD_FUNC_ARG;
return BAD_FUNC_ARG;
while (1) { while (list) {
if (list->name == name) if (list->name == name) {
return 0; /* curve alreay in use */ ret = 0; /* curve alreay in use */
break;
}
if (list->next == NULL) if (list->next == NULL) {
return TLSX_SupportedCurve_New(&list->next, name, heap); ret = TLSX_SupportedCurve_New(&list->next, name, heap);
break;
}
list = list->next; list = list->next;
} }
return 0; return ret;
} }
static int TLSX_PointFormat_Append(PointFormat* list, byte format, void* heap) static int TLSX_PointFormat_Append(PointFormat* list, byte format, void* heap)
{ {
if (list == NULL) int ret = BAD_FUNC_ARG;
return BAD_FUNC_ARG;
while (1) { while (1) {
if (list->format == format) if (list->format == format) {
return 0; /* format already in use */ ret = 0; /* format already in use */
break;
}
if (list->next == NULL) if (list->next == NULL) {
return TLSX_PointFormat_New(&list->next, format, heap); ret = TLSX_PointFormat_New(&list->next, format, heap);
break;
}
list = list->next; list = list->next;
} }
return 0; return ret;
} }
#ifndef NO_WOLFSSL_CLIENT #ifndef NO_WOLFSSL_CLIENT