diff options
| author | mmcc@openbsd.org <mmcc@openbsd.org> | 2015-10-20 03:36:35 +0000 |
|---|---|---|
| committer | Damien Miller <djm@mindrot.org> | 2015-10-25 11:42:03 +1100 |
| commit | f98a09cacff7baad8748c9aa217afd155a4d493f (patch) | |
| tree | 3ed17a7a6de78979338316b4be8712803469db53 /clientloop.c | |
| parent | ac908c1eeacccfa85659594d92428659320fd57e (diff) | |
upstream commit
Replace a function-local allocation with stack memory.
ok djm@
Upstream-ID: c09fbbab637053a2ab9f33ca142b4e20a4c5a17e
Diffstat (limited to 'clientloop.c')
| -rw-r--r-- | clientloop.c | 9 |
1 files changed, 2 insertions, 7 deletions
diff --git a/clientloop.c b/clientloop.c index 87ceb3da..1e05cba2 100644 --- a/clientloop.c +++ b/clientloop.c @@ -1,4 +1,4 @@ -/* $OpenBSD: clientloop.c,v 1.275 2015/07/10 06:21:53 markus Exp $ */ +/* $OpenBSD: clientloop.c,v 1.276 2015/10/20 03:36:35 mmcc Exp $ */ /* * Author: Tatu Ylonen <ylo@cs.hut.fi> * Copyright (c) 1995 Tatu Ylonen <ylo@cs.hut.fi>, Espoo, Finland @@ -311,11 +311,10 @@ client_x11_get_proto(const char *display, const char *xauth_path, static char proto[512], data[512]; FILE *f; int got_data = 0, generated = 0, do_unlink = 0, i; - char *xauthdir, *xauthfile; + char xauthdir[PATH_MAX] = "", xauthfile[PATH_MAX] = ""; struct stat st; u_int now, x11_timeout_real; - xauthdir = xauthfile = NULL; *_proto = proto; *_data = data; proto[0] = data[0] = '\0'; @@ -343,8 +342,6 @@ client_x11_get_proto(const char *display, const char *xauth_path, display = xdisplay; } if (trusted == 0) { - xauthdir = xmalloc(PATH_MAX); - xauthfile = xmalloc(PATH_MAX); mktemp_proto(xauthdir, PATH_MAX); /* * The authentication cookie should briefly outlive @@ -407,8 +404,6 @@ client_x11_get_proto(const char *display, const char *xauth_path, unlink(xauthfile); rmdir(xauthdir); } - free(xauthdir); - free(xauthfile); /* * If we didn't get authentication data, just make up some |