am cd868370: am b2c3473d: am 70df6f85: Merge "Just use snprintf for android_get_control_socket."

* commit 'cd8683709c2d1bcac7e3b9de827a2dfc8f24bf09':
  Just use snprintf for android_get_control_socket.
This commit is contained in:
Elliott Hughes 2015-04-03 14:53:01 +00:00 committed by Android Git Automerger
commit 16bcaac5fd

View file

@ -18,6 +18,7 @@
#define __CUTILS_SOCKETS_H
#include <errno.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <stdbool.h>
@ -46,30 +47,19 @@ extern "C" {
*/
static inline int android_get_control_socket(const char *name)
{
char key[64] = ANDROID_SOCKET_ENV_PREFIX;
const char *val;
int fd;
char key[64];
snprintf(key, sizeof(key), ANDROID_SOCKET_ENV_PREFIX "%s", name);
/* build our environment variable, counting cycles like a wolf ... */
#if HAVE_STRLCPY
strlcpy(key + sizeof(ANDROID_SOCKET_ENV_PREFIX) - 1,
name,
sizeof(key) - sizeof(ANDROID_SOCKET_ENV_PREFIX));
#else /* for the host, which may lack the almightly strncpy ... */
strncpy(key + sizeof(ANDROID_SOCKET_ENV_PREFIX) - 1,
name,
sizeof(key) - sizeof(ANDROID_SOCKET_ENV_PREFIX));
key[sizeof(key)-1] = '\0';
#endif
val = getenv(key);
if (!val)
const char* val = getenv(key);
if (!val) {
return -1;
}
errno = 0;
fd = strtol(val, NULL, 10);
if (errno)
int fd = strtol(val, NULL, 10);
if (errno) {
return -1;
}
return fd;
}