diff options
| author | Treehugger Robot <treehugger-gerrit@google.com> | 2022-05-19 12:24:48 +0000 |
|---|---|---|
| committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | 2022-05-19 12:24:48 +0000 |
| commit | 0f6a950aa4e5b900c53e679e748a7a2d32e0fdd9 (patch) | |
| tree | 9c7beb6d035ed599e94604752bf84298dd640b37 | |
| parent | 85f860774137fe6fe00e2586f7f44df6d5fc6cb0 (diff) | |
| parent | 966c731295ec0b7722331ea324baed3a06cdbc6f (diff) | |
| download | ppp-0f6a950aa4e5b900c53e679e748a7a2d32e0fdd9.tar.gz | |
Merge "pppd: add error string for setuid" am: f1682f0e43 am: 41bdf52d58 am: 966c731295
Original change: https://android-review.googlesource.com/c/platform/external/ppp/+/2098554
Change-Id: Iccec994fc0e4de530497c05e7b2521d2d3c306a6
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__) |