Snap for 10453563 from 7e036a35b97e10017097baccf25364398f5d62c8 to mainline-permission-releaseaml_per_341614000aml_per_341510010aml_per_341410020aml_per_341311000aml_per_341110020aml_per_341110010aml_per_341011100aml_per_341011020aml_per_340916010android14-mainline-permission-release

Change-Id: Ic8bb693740f49b523afb7c78da43fe1587fcca30

3 files changed, 18 insertions, 9 deletions

diff --git a/METADATA b/METADATA
index c138456..78553e1 100644
--- a/METADATA
+++ b/METADATA
@@ -1,6 +1,19 @@
 name: "ppp"
-version: "2.4.7"
-
+description: "ppp (Paul's PPP Package) is an open source package which implements the Point-to-Point Protocol (PPP) on Linux and Solaris systems."
 third_party {
+  url {
+    type: HOMEPAGE
+    value: "https://ppp.samba.org/"
+  }
+  url {
+    type: ARCHIVE
+    value: "https://ftp.samba.org/pub/ppp/ppp-2.4.7.tar.gz"
+  }
+  version: "ppp-2.4.7"
   license_type: RESTRICTED
+  last_upgrade_date {
+    year: 2014
+    month: 12
+    day: 8
+  }
 }
diff --git a/README.version b/README.version
deleted file mode 100644
index 63223dc..0000000
--- a/README.version
+++ /dev/null
@@ -1,3 +0,0 @@
-URL: ftp://ftp.samba.org/pub/ppp/ppp-2.4.7.tar.gz
-Version: 2.4.7
-BugComponent: 36824
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__)