oysteikt/heimdal
0
0
Fork 0
Code Issues 5 Pull Requests Releases Activity

IPC: Solaris doors backend: avoid using TLS where possible

Browse Source 
Optimistically delay using thread-local storage for the door reply
buffer until we actually need to allocate from the heap.
This commit is contained in:
Luke Howard
2018-05-14 12:14:59 +10:00
parent c5db6e691a
commit 927185e732
1 changed files with 8 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
lib/ipc/server.c
View File

@@ -1191,12 +1191,11 @@ door_complete(heim_sipc_call ctx, int returnvalue, heim_idata *reply)
struct door_reply reply;
char buffer[offsetof(struct door_reply, data) + BUFSIZ];
} replyBuf;
int ret;
heim_base_once_f(&once, &door_key, door_key_create);
/* free previously saved reply, if any */
free(HEIMDAL_getspecific(door_key));
/* door_return() doesn't return; don't leak cred */
heim_ipc_free_cred(cs->cred);
error_reply:
rlen = offsetof(struct door_reply, data);
@@ -1205,11 +1204,16 @@ error_reply:
/* long replies (> BUFSIZ) are allocated from the heap */
if (rlen > BUFSIZ) {
r = malloc(rlen);
int ret;
/* door_return() doesn't return, so stash reply buffer in TLS */
r = realloc(HEIMDAL_getspecific(door_key), rlen);
if (r == NULL) {
returnvalue = EAGAIN; /* don't leak ENOMEM to caller */
goto error_reply;
}
HEIMDAL_setspecific(door_key, r, ret);
} else {
r = &replyBuf.reply;
}
@@ -1222,10 +1226,6 @@ error_reply:
r->length = 0;
}
/* door_return() doesn't return; don't leak cred */
heim_ipc_free_cred(cs->cred);
HEIMDAL_setspecific(door_key, r == &replyBuf.reply ? NULL : r, ret);
door_return((char *)r, rlen, NULL, 0);
}