commit 050b06271a2d636e8e5db33fd0dde65b65003db3
author Keun-young Park <keunyoung@google.com> Sun Nov 13 05:07:00 2016 +0000
committer android-build-merger <android-build-merger@google.com> Sun Nov 13 05:07:00 2016 +0000
tree a94bb8664c4dff211def211c6278cd5717bd6fd9
parent 642687eff1077137e4d62916e2f250f20ace0eb4
parent 9366ac021746043dadadf14561d64a46f22257ed

Merge "fix swapped min / max in priority parsing" am: 557359863c am: ec2d5d8f04
am: ca2714b02a

Change-Id: I3091ec4046b5d36d8b5ee0359a97a1ed0e0bed9e

init/service.cpp

diff --git a/init/service.cpp b/init/service.cpp
index 9fa11b8..f093dd9 100644
--- a/init/service.cpp
+++ b/init/service.cpp
@@ -357,8 +357,8 @@
 bool Service::ParsePriority(const std::vector<std::string>& args, std::string* err) {
     priority_ = 0;
     if (!ParseInt(args[1], &priority_,
-                  static_cast<int>(ANDROID_PRIORITY_LOWEST),
-                  static_cast<int>(ANDROID_PRIORITY_HIGHEST))) {
+                  static_cast<int>(ANDROID_PRIORITY_HIGHEST), // highest is negative
+                  static_cast<int>(ANDROID_PRIORITY_LOWEST))) {
         *err = StringPrintf("process priority value must be range %d - %d",
                 ANDROID_PRIORITY_HIGHEST, ANDROID_PRIORITY_LOWEST);
         return false;