feedc0de/wolfssl
1
0
Fork 0
forked from wolfSSL/wolfssl
Code Pull Requests Activity

crl: refactoring DoMonitor to reduce stack usage:

Browse Source 
--- variable buff moved to the heap (8192 bytes saved)
This commit is contained in:
Moisés Guimarães
2014-07-21 13:18:12 -03:00
parent 108b21e36f
commit 3ae9105b05
1 changed files with 19 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

23
src/crl.c
View File

@ -546,6 +546,11 @@ static void* DoMonitor(void* arg)
int notifyFd; int notifyFd;
int wd = -1; int wd = -1;
CYASSL_CRL* crl = (CYASSL_CRL*)arg; CYASSL_CRL* crl = (CYASSL_CRL*)arg;
#ifdef CYASSL_SMALL_STACK
char* buff;
#else
char buff[8192];
#endif
CYASSL_ENTER("DoMonitor"); CYASSL_ENTER("DoMonitor");
@ -584,10 +589,16 @@ static void* DoMonitor(void* arg)
} }
} }
#ifdef CYASSL_SMALL_STACK
buff = (char*)XMALLOC(8192, NULL, DYNAMIC_TYPE_TMP_BUFFER);
if (buff == NULL)
return NULL;
#endif
for (;;) { for (;;) {
fd_set readfds; fd_set readfds;
char buff[8192]; int result;
int result, length; int length;
FD_ZERO(&readfds); FD_ZERO(&readfds);
FD_SET(notifyFd, &readfds); FD_SET(notifyFd, &readfds);
@ -607,7 +618,7 @@ static void* DoMonitor(void* arg)
break; break;
} }
length = read(notifyFd, buff, sizeof(buff)); length = read(notifyFd, buff, 8192);
if (length < 0) { if (length < 0) {
CYASSL_MSG("notify read problem, continue"); CYASSL_MSG("notify read problem, continue");
continue; continue;
@ -618,6 +629,10 @@ static void* DoMonitor(void* arg)
} }
} }
#ifdef CYASSL_SMALL_STACK
XFREE(buff, NULL, DYNAMIC_TYPE_TMP_BUFFER);
#endif
if (wd > 0) if (wd > 0)
inotify_rm_watch(notifyFd, wd); inotify_rm_watch(notifyFd, wd);
close(crl->mfd); close(crl->mfd);