diff options
| author | Elliott Hughes <enh@google.com> | 2014-07-29 18:42:53 +0000 |
|---|---|---|
| committer | Android Git Automerger <android-git-automerger@android.com> | 2014-07-29 18:42:53 +0000 |
| commit | 43eb2ea5f83c79032e6b9b566d0f4e3626683631 (patch) | |
| tree | 0fe65e6eb2f79f7ea3e60acfdcaaba0385dfef14 /adb/services.c | |
| parent | a56ab4b00fe898a1d7768c1d3023fc998c7867b5 (diff) | |
| parent | ec531e5cdde5e0b120bd88aeb2c3d36d18cedfb4 (diff) | |
| download | system_core-43eb2ea5f83c79032e6b9b566d0f4e3626683631.zip system_core-43eb2ea5f83c79032e6b9b566d0f4e3626683631.tar.gz system_core-43eb2ea5f83c79032e6b9b566d0f4e3626683631.tar.bz2 | |
am ec531e5c: am 47a12389: Merge "Fix adb "ptsname is not thread-safe; use ptsname_r instead" build break."
* commit 'ec531e5cdde5e0b120bd88aeb2c3d36d18cedfb4':
Fix adb "ptsname is not thread-safe; use ptsname_r instead" build break.
Diffstat (limited to 'adb/services.c')
| -rw-r--r-- | adb/services.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/adb/services.c b/adb/services.c index 2875ce0..e61371a 100644 --- a/adb/services.c +++ b/adb/services.c @@ -206,7 +206,6 @@ static int create_subproc_pty(const char *cmd, const char *arg0, const char *arg fprintf(stderr, "error: create_subproc_pty not implemented on Win32 (%s %s %s)\n", cmd, arg0, arg1); return -1; #else /* !HAVE_WIN32_PROC */ - char *devname; int ptm; ptm = unix_open("/dev/ptmx", O_RDWR | O_CLOEXEC); // | O_NOCTTY); @@ -215,8 +214,8 @@ static int create_subproc_pty(const char *cmd, const char *arg0, const char *arg return -1; } - if(grantpt(ptm) || unlockpt(ptm) || - ((devname = (char*) ptsname(ptm)) == 0)){ + char devname[64]; + if(grantpt(ptm) || unlockpt(ptm) || ptsname_r(ptm, devname, sizeof(devname)) != 0) { printf("[ trouble with /dev/ptmx - %s ]\n", strerror(errno)); adb_close(ptm); return -1; |