diff options
| author | Treehugger Robot <treehugger-gerrit@google.com> | 2022-05-19 13:17:13 +0000 |
|---|---|---|
| committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | 2022-05-19 13:17:13 +0000 |
| commit | 5a65d6a78329a9eecb9c1a0a7fc9ea1c4b29aaee (patch) | |
| tree | 9c7beb6d035ed599e94604752bf84298dd640b37 | |
| parent | d03b03a0232827db032ebc3f8558824f02385e9e (diff) | |
| parent | a33087b36f99196d4d561b7a25d32caa0afca6e2 (diff) | |
| download | ppp-5a65d6a78329a9eecb9c1a0a7fc9ea1c4b29aaee.tar.gz | |
Merge "pppd: add error string for setuid" am: f1682f0e43 am: 41bdf52d58 am: 966c731295 am: 0f6a950aa4 am: a33087b36f
Original change: https://android-review.googlesource.com/c/platform/external/ppp/+/2098554
Change-Id: I89a402b582cc66d9dd32fe017e6f4c1a4e5ddf6e
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
| -rw-r--r-- | pppd/main.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/pppd/main.c b/pppd/main.c index 09f5a53..df4e36c 100644 --- a/pppd/main.c +++ b/pppd/main.c @@ -1735,10 +1735,9 @@ device_script(program, in, out, dont_wait) /* here we are executing in the child */ setgid(getgid()); - setuid(uid); - if (getuid() != uid) { - fprintf(stderr, "pppd: setuid failed\n"); - exit(1); + if (setuid(uid) < 0) { + fprintf(stderr, "pppd: setuid failed: %s\n", strerror(errno)); + exit(1); } update_system_environment(); #if defined(__ANDROID__) |