Remove double cast + move min() calls

This commit is contained in:
Levi Rak
2017-06-21 11:09:40 -06:00
parent 17936d65e0
commit 247388903b

View File

@@ -4152,32 +4152,29 @@ int PemToDer(const unsigned char* buff, long longSz, int type,
{ {
/* remove encrypted header if there */ /* remove encrypted header if there */
char encHeader[] = "Proc-Type"; char encHeader[] = "Proc-Type";
word32 headerEndSz = min((word32)((size_t)(bufferEnd - headerEnd)), word32 headerEndSz = (word32)(bufferEnd - headerEnd);
PEM_LINE_LEN); char* line = XSTRNSTR(headerEnd, encHeader, min(headerEndSz,
char* line = XSTRNSTR(headerEnd, encHeader, headerEndSz); PEM_LINE_LEN));
if (line != NULL) { if (line != NULL) {
word32 lineSz = min((word32)((size_t)(bufferEnd - line)),
PEM_LINE_LEN);
char* newline; char* newline;
char* finish; char* finish;
word32 finishSz; word32 finishSz;
char* start = XSTRNSTR(line, "DES", lineSz);
word32 startSz; word32 startSz;
word32 lineSz = (word32)(bufferEnd - line);
char* start = XSTRNSTR(line, "DES", min(lineSz, PEM_LINE_LEN));
if (start == NULL) if (start == NULL)
start = XSTRNSTR(line, "AES", lineSz); start = XSTRNSTR(line, "AES", min(lineSz, PEM_LINE_LEN));
if (start == NULL) return SSL_BAD_FILE; if (start == NULL) return SSL_BAD_FILE;
if (info == NULL) return SSL_BAD_FILE; if (info == NULL) return SSL_BAD_FILE;
startSz = min((word32)((size_t)(bufferEnd - line)), startSz = (word32)(bufferEnd - start);
PEM_LINE_LEN); finish = XSTRNSTR(start, ",", min((word32)startSz, PEM_LINE_LEN));
finish = XSTRNSTR(start, ",", startSz);
if ((start != NULL) && (finish != NULL) && (start < finish)) { if ((start != NULL) && (finish != NULL) && (start < finish)) {
finishSz = min((word32)((size_t)(bufferEnd - finish)), finishSz = (word32)(bufferEnd - finish);
PEM_LINE_LEN); newline = XSTRNSTR(finish, "\r", min(finishSz, PEM_LINE_LEN));
newline = XSTRNSTR(finish, "\r", finishSz);
if (XMEMCPY(info->name, start, finish - start) == NULL) if (XMEMCPY(info->name, start, finish - start) == NULL)
return SSL_FATAL_ERROR; return SSL_FATAL_ERROR;
@@ -4186,7 +4183,8 @@ int PemToDer(const unsigned char* buff, long longSz, int type,
return SSL_FATAL_ERROR; return SSL_FATAL_ERROR;
if (newline == NULL) if (newline == NULL)
newline = XSTRNSTR(finish, "\n", finishSz); newline = XSTRNSTR(finish, "\n", min(finishSz,
PEM_LINE_LEN));
if ((newline != NULL) && (newline > finish)) { if ((newline != NULL) && (newline > finish)) {
info->ivSz = (word32)(newline - (finish + 1)); info->ivSz = (word32)(newline - (finish + 1));
info->set = 1; info->set = 1;