fix an issue on 64bit

This reverts commit 185e7608ae.

.gitignore

@@ -1,7 +1,6 @@
 classes
 bin
 gen
-obj
 target
 local.properties
 .classpath
@@ -24,11 +23,10 @@ local.sbt
 out/
 
 #NDK
+src/main/obj
 src/main/libs
 src/main/assets/armeabi
 src/main/assets/armeabi-v7a
 src/main/assets/x86
 src/main/assets/api-16/
 jni/Application.mk
-obj
-libs

.travis.yml

@@ -18,5 +18,6 @@ install:
 - echo "y" | android update sdk --filter extra-android-m2repository --no-ui --no-https -a
 script:
 - cp local.properties.travis local.properties
+- ./build.sh
 - sbt android:package-release
 - exit 0

README.md

@@ -28,6 +28,9 @@ Help to translate shadowsocks: http://crowdin.net/project/shadowsocks/invite
 
 ```bash
     git submodule update --init
+
+    # Build native binaries
+    ./build.sh
     
     # Build the App
     sbt clean android:package-release
@@ -47,6 +50,9 @@ Help to translate shadowsocks: http://crowdin.net/project/shadowsocks/invite
 ```bash
     git submodule update --init
 
+    # Build native binaries
+    ./build.sh
+
     # Build the apk
     sbt clean android:package-release
 ```

build.sbt

@@ -12,6 +12,8 @@ compileOrder in Compile := CompileOrder.JavaThenScala
 
 ndkJavah in Android := List()
 
+ndkBuild in Android := List()
+
 resolvers += "JRAF" at "http://JRAF.org/static/maven/2"
 
 resolvers += "madeye private releases" at "http://madeye-maven-repository.googlecode.com/git/"

build.sh

@@ -0,0 +1,24 @@
+#!/bin/bash
+
+function try () {
+"$@" || exit -1
+}
+
+try pushd src/main
+
+# Build
+try $ANDROID_NDK_HOME/ndk-build clean
+try $ANDROID_NDK_HOME/ndk-build
+
+# copy executables
+rm -rf assets/armeabi-v7a
+rm -rf assets/x86
+mkdir -p assets/armeabi-v7a
+mkdir -p assets/x86
+for app in pdnsd redsocks ss-local ss-tunnel tun2socks
+do
+    try mv libs/armeabi-v7a/$app assets/armeabi-v7a/
+    try mv libs/x86/$app assets/x86/
+done
+
+try popd

project/plugins.sbt

@@ -2,4 +2,6 @@ resolvers += Resolver.url("scalasbt releases", new URL("http://scalasbt.artifact
 
 addSbtPlugin("com.hanhuy.sbt" % "android-sdk-plugin" % "1.3.10")
 
-addSbtPlugin("com.github.mpeltonen" % "sbt-idea" % "1.6.0")
+resolvers += Resolver.sbtPluginRepo("snapshots")
+
+addSbtPlugin("com.hanhuy.sbt" % "sbt-idea" % "1.7.0-SNAPSHOT")

src/main/AndroidManifest.xml

@@ -1,7 +1,7 @@
 <manifest xmlns:android="http://schemas.android.com/apk/res/android"
     package="com.github.shadowsocks"
-    android:versionCode="83"
-    android:versionName="2.4.3">
+    android:versionCode="86"
+    android:versionName="2.4.6">
 
   <uses-permission android:name="android.permission.INTERNET"/>
   <uses-permission android:name="android.permission.ACCESS_NETWORK_STATE"/>

src/main/java/com/github/shadowsocks/Core.java

@@ -1,55 +0,0 @@
-/* Shadowsocks - A shadowsocks client for Android
- * Copyright (C) 2012 <max.c.lv@gmail.com>
- *
- * This program is free software: you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program.  If not, see <http://www.gnu.org/licenses/>.
- *
- *
- *                            ___====-_  _-====___
- *                      _--^^^#####//      \\#####^^^--_
- *                   _-^##########// (    ) \\##########^-_
- *                  -############//  |\^^/|  \\############-
- *                _/############//   (@::@)   \\############\_
- *               /#############((     \\//     ))#############\
- *              -###############\\    (oo)    //###############-
- *             -#################\\  / VV \  //#################-
- *            -###################\\/      \//###################-
- *           _#/|##########/\######(   /\   )######/\##########|\#_
- *           |/ |#/\#/\#/\/  \#/\##\  |  |  /##/\#/  \/\#/\#/\#| \|
- *           `  |/  V  V  `   V  \#\| |  | |/#/  V   '  V  V  \|  '
- *              `   `  `      `   / | |  | | \   '      '  '   '
- *                               (  | |  | |  )
- *                              __\ | |  | | /__
- *                             (vvv(VVV)(VVV)vvv)
- *
- *                              HERE BE DRAGONS
- *
- */
-
-package com.github.shadowsocks;
-
-public class Core {
-  static {
-    java.lang.System.loadLibrary("ss-local-jni");
-    java.lang.System.loadLibrary("ss-tunnel-jni");
-    java.lang.System.loadLibrary("tun2socks-jni");
-    java.lang.System.loadLibrary("pdnsd-jni");
-    java.lang.System.loadLibrary("redsocks-jni");
-  }
-
-  public static native void sslocal(String[] cmd);
-  public static native void sstunnel(String[] cmd);
-  public static native void tun2socks(String[] cmd);
-  public static native void pdnsd(String[] cmd);
-  public static native void redsocks(String[] cmd);
-}

src/main/jni/Android.mk

@@ -120,13 +120,13 @@ REDSOCKS_SOURCES := base.c dnstc.c http-connect.c \
 
 LOCAL_STATIC_LIBRARIES := libevent
 
-LOCAL_MODULE := redsocks-jni
-LOCAL_SRC_FILES := $(addprefix redsocks/, $(REDSOCKS_SOURCES)) main-jni.cpp
-LOCAL_CFLAGS := -O2 -std=gnu99 -DREDSOCKS_JNI -I$(LOCAL_PATH)/redsocks \
+LOCAL_MODULE := redsocks
+LOCAL_SRC_FILES := $(addprefix redsocks/, $(REDSOCKS_SOURCES)) 
+LOCAL_CFLAGS := -O2 -std=gnu99 -I$(LOCAL_PATH)/redsocks \
 	-I$(LOCAL_PATH)/libevent/include \
 	-I$(LOCAL_PATH)/libevent
 
-include $(BUILD_SHARED_LIBRARY)
+include $(BUILD_EXECUTABLE)
 
 ########################################################
 ## pdnsd library
@@ -134,13 +134,13 @@ include $(BUILD_SHARED_LIBRARY)
 
 include $(CLEAR_VARS)
 
-PDNSD_SOURCES  := $(wildcard $(LOCAL_PATH)/pdnsd/src/*.c) main-jni.cpp
+PDNSD_SOURCES  := $(wildcard $(LOCAL_PATH)/pdnsd/src/*.c)
 
-LOCAL_MODULE    := pdnsd-jni
+LOCAL_MODULE    := pdnsd
 LOCAL_SRC_FILES := $(PDNSD_SOURCES:$(LOCAL_PATH)/%=%)
-LOCAL_CFLAGS    := -Wall -O2 -DPDNSD_JNI -I$(LOCAL_PATH)/pdnsd
+LOCAL_CFLAGS    := -Wall -O2 -I$(LOCAL_PATH)/pdnsd
 
-include $(BUILD_SHARED_LIBRARY)
+include $(BUILD_EXECUTABLE)
 
 ########################################################
 ## shadowsocks local library
@@ -150,9 +150,9 @@ include $(CLEAR_VARS)
 
 SHADOWSOCKS_SOURCES := local.c cache.c udprelay.c encrypt.c utils.c json.c jconf.c acl.c 
 
-LOCAL_MODULE    := ss-local-jni
-LOCAL_SRC_FILES := $(addprefix shadowsocks/src/, $(SHADOWSOCKS_SOURCES)) main-jni.cpp
-LOCAL_CFLAGS    := -Wall -O2 -fno-strict-aliasing -DUDPRELAY_LOCAL -DSSLOCAL_JNI \
+LOCAL_MODULE    := ss-local
+LOCAL_SRC_FILES := $(addprefix shadowsocks/src/, $(SHADOWSOCKS_SOURCES))
+LOCAL_CFLAGS    := -Wall -O2 -fno-strict-aliasing -DUDPRELAY_LOCAL \
 					-DUSE_CRYPTO_OPENSSL -DANDROID -DHAVE_CONFIG_H \
 					-I$(LOCAL_PATH)/libev/ \
 					-I$(LOCAL_PATH)/openssl/include  \
@@ -163,22 +163,18 @@ LOCAL_STATIC_LIBRARIES := libev libcrypto libipset libcork
 
 LOCAL_LDLIBS := -llog
 
-include $(BUILD_SHARED_LIBRARY)
+include $(BUILD_EXECUTABLE)
 
 ########################################################
 ## shadowsocks tunnel
 ########################################################
 
-########################################################
-## shadowsocks tunnel library
-########################################################
-
 include $(CLEAR_VARS)
 
 SHADOWSOCKS_SOURCES := tunnel.c cache.c udprelay.c encrypt.c utils.c json.c jconf.c
 
-LOCAL_MODULE    := ss-tunnel-jni
-LOCAL_SRC_FILES := $(addprefix shadowsocks/src/, $(SHADOWSOCKS_SOURCES)) main-jni.cpp
+LOCAL_MODULE    := ss-tunnel
+LOCAL_SRC_FILES := $(addprefix shadowsocks/src/, $(SHADOWSOCKS_SOURCES))
 LOCAL_CFLAGS    := -Wall -O2 -fno-strict-aliasing -DUDPRELAY_LOCAL -DUDPRELAY_TUNNEL \
 					-DUSE_CRYPTO_OPENSSL -DANDROID -DHAVE_CONFIG_H -DSSTUNNEL_JNI \
 					-I$(LOCAL_PATH)/libev/ \
@@ -188,7 +184,7 @@ LOCAL_STATIC_LIBRARIES := libev libcrypto
 
 LOCAL_LDLIBS := -llog
 
-include $(BUILD_SHARED_LIBRARY)
+include $(BUILD_EXECUTABLE)
 
 ########################################################
 ## system
@@ -208,7 +204,7 @@ LOCAL_STATIC_LIBRARIES := cpufeatures
 include $(BUILD_SHARED_LIBRARY)
 
 ########################################################
-## tun2socks-library
+## tun2socks
 ########################################################
 
 include $(CLEAR_VARS)
@@ -218,7 +214,7 @@ LOCAL_CFLAGS += -DBADVPN_THREADWORK_USE_PTHREAD -DBADVPN_LINUX -DBADVPN_BREACTOR
 LOCAL_CFLAGS += -DBADVPN_USE_SELFPIPE -DBADVPN_USE_EPOLL
 LOCAL_CFLAGS += -DBADVPN_LITTLE_ENDIAN -DBADVPN_THREAD_SAFE
 LOCAL_CFLAGS += -DNDEBUG -DANDROID
-LOCAL_CFLAGS += -DTUN2SOCKS_JNI
+# LOCAL_CFLAGS += -DTUN2SOCKS_JNI
 
 LOCAL_C_INCLUDES:= \
         $(LOCAL_PATH)/badvpn/lwip/src/include/ipv4 \
@@ -288,13 +284,13 @@ TUN2SOCKS_SOURCES := \
         tun2socks/SocksUdpGwClient.c \
         udpgw_client/UdpGwClient.c
 
-LOCAL_MODULE := tun2socks-jni
+LOCAL_MODULE := tun2socks
 
 LOCAL_LDLIBS := -ldl -llog
 
-LOCAL_SRC_FILES := $(addprefix badvpn/, $(TUN2SOCKS_SOURCES)) main-jni.cpp
+LOCAL_SRC_FILES := $(addprefix badvpn/, $(TUN2SOCKS_SOURCES))
 
-include $(BUILD_SHARED_LIBRARY)
+include $(BUILD_EXECUTABLE)
 
 # OpenSSL
 openssl_subdirs := $(addprefix $(LOCAL_PATH)/openssl/,$(addsuffix /Android.mk, \

src/main/jni/Application.mk

@@ -1,4 +1,4 @@
-APP_ABI      			:= armeabi-v7a arm64-v8a x86 x86_64
+APP_ABI      			:= armeabi-v7a x86
 APP_PLATFORM 			:= android-16
 APP_STL      			:= stlport_static
 NDK_TOOLCHAIN_VERSION 	:= 4.9

src/main/jni/main-jni.cpp

@@ -1,86 +0,0 @@
-#include <jni.h>
-#include <string.h>
-#include <unistd.h>
-
-typedef unsigned short char16_t;
-
-class String8 {
-public:
-    String8() {
-        mString = 0;
-    }
-
-    ~String8() {
-        if (mString) {
-            free(mString);
-        }
-    }
-
-    void set(const char16_t* o, size_t numChars) {
-        if (mString) {
-            free(mString);
-        }
-        mString = (char*) malloc(numChars + 1);
-        if (!mString) {
-            return;
-        }
-        for (size_t i = 0; i < numChars; i++) {
-            mString[i] = (char) o[i];
-        }
-        mString[numChars] = '\0';
-    }
-
-    const char* string() {
-        return mString;
-    }
-private:
-    char* mString;
-};
-
-extern "C" {
-
-int main(int argc, char* args[]);
-
-#if defined(PDNSD_JNI)
-jint Java_com_github_shadowsocks_Core_pdnsd(JNIEnv *env, jobject thiz, jobjectArray argv) {
-#elif defined(SSLOCAL_JNI)
-jint Java_com_github_shadowsocks_Core_sslocal(JNIEnv *env, jobject thiz, jobjectArray argv) {
-#elif defined(SSTUNNEL_JNI)
-jint Java_com_github_shadowsocks_Core_sstunnel(JNIEnv *env, jobject thiz, jobjectArray argv) {
-#elif defined(REDSOCKS_JNI)
-jint Java_com_github_shadowsocks_Core_redsocks(JNIEnv *env, jobject thiz, jobjectArray argv) {
-#elif defined(TUN2SOCKS_JNI)
-jint Java_com_github_shadowsocks_Core_tun2socks(JNIEnv *env, jobject thiz, jobjectArray argv) {
-#else
-#error "invalid header"
-#endif
-
-    int argc = argv ? env->GetArrayLength(argv) : 0;
-    char **args = NULL;
-    String8 tmp_8;
-
-    if (argc > 0) {
-        args = (char **)malloc((argc+1)*sizeof(char *));
-        for (int i = 0; i < argc; ++i) {
-            jstring arg = reinterpret_cast<jstring>(env->GetObjectArrayElement(argv, i));
-            const jchar *str = env->GetStringCritical(arg, 0);
-            tmp_8.set(str, env->GetStringLength(arg));
-            env->ReleaseStringCritical(arg, str);
-            args[i] = strdup(tmp_8.string());
-        }
-        args[argc] = NULL;
-
-        pid_t pid;
-        pid = fork();
-        if (pid == 0) {
-            int ret = main(argc, args);
-            return ret;
-        }
-
-        if (args != NULL) free(args);
-    }
-
-    return 0;
-}
-
-}

src/main/jni/openssl/android-config.mk

@@ -15,7 +15,7 @@ LOCAL_CFLAGS += -DOPENSSL_NO_EC_NISTP_64_GCC_128 -DOPENSSL_NO_SCTP \
 # LOCAL_CFLAGS += -DOPENSSL_NO_DEPRECATED
 
 # Extra
-LOCAL_CFLAGS += -DOPENSSL_NO_HW -DOPENSSL_NO_ASM -DOPENSSL_NO_ENGINE -DZLIB
+LOCAL_CFLAGS += -DOPENSSL_NO_HW -DOPENSSL_NO_ENGINE -DZLIB
 
 # Debug
 # LOCAL_CFLAGS += -DCIPHER_DEBUG

src/main/jni/openssl/crypto/evp/e_rc4_hmac_md5.c

@@ -100,8 +100,8 @@ static int rc4_hmac_md5_init_key(EVP_CIPHER_CTX *ctx,
 	return 1;
 	}
 
+// Disable asm for Android x86_64
 #if	!defined(OPENSSL_NO_ASM) &&	( \
-	defined(__x86_64)	|| defined(__x86_64__)	|| \
 	defined(_M_AMD64)	|| defined(_M_X64)	|| \
 	defined(__INTEL__)		) && \
 	!(defined(__APPLE__) && defined(__MACH__))

src/main/jni/system.cpp

@@ -32,9 +32,6 @@ jstring Java_com_github_shadowsocks_system_getabi(JNIEnv *env, jobject thiz) {
 
 void Java_com_github_shadowsocks_system_exec(JNIEnv *env, jobject thiz, jstring cmd) {
     const char *str  = env->GetStringUTFChars(cmd, 0);
-    setenv("LD_LIBRARY_PATH", "/vendor/lib:/system/lib", 1);
-    setegid(getgid());
-    seteuid(getuid());
     system(str);
     env->ReleaseStringUTFChars(cmd, str);
 }
@@ -48,6 +45,8 @@ static JNINativeMethod method_table[] = {
         (void*) Java_com_github_shadowsocks_system_getabi }
 };
 
+
+
 /*
  * Register several native methods for one class.
  */

src/main/scala/com/github/shadowsocks/Shadowsocks.scala

@@ -123,7 +123,7 @@ object Shadowsocks {
   val FEATRUE_PREFS = Array(Key.route, Key.isGlobalProxy, Key.proxyedApps,
     Key.isUdpDns, Key.isAutoConnect)
 
-  val EXECUTABLES = Array(Executable.PDNSD, Executable.REDSOCKS, Executable.IPTABLES)
+  val EXECUTABLES = Array(Executable.PDNSD, Executable.REDSOCKS, Executable.SS_TUNNEL, Executable.SS_LOCAL, Executable.TUN2SOCKS)
 
   // Helper functions
   def updateListPreference(pref: Preference, value: String) {
@@ -834,6 +834,14 @@ class Shadowsocks
 
     crash_recovery()
 
+    copyAssets(System.getABI)
+
+    val ab = new ArrayBuffer[String]
+    for (executable <- Shadowsocks.EXECUTABLES) {
+      ab.append("chmod 755 " + Path.BASE + executable)
+    }
+    Console.runCommand(ab.toArray)
+
   }
 
   private def recovery() {

src/main/scala/com/github/shadowsocks/ShadowsocksNatService.scala

@@ -102,14 +102,17 @@ class ShadowsocksNatService extends Service with BaseService {
       })
     }
 
+    val conf = ConfigUtils
+      .SHADOWSOCKS.format(config.proxy, config.remotePort, config.localPort,
+        config.sitekey, config.encMethod, 10)
+    ConfigUtils.printToFile(new File(Path.BASE + "ss-local.json"))(p => {
+      p.println(conf)
+    })
+
     val cmd = new ArrayBuffer[String]
-    cmd += ("ss-local"
+    cmd += (Path.BASE + "ss-local"
           , "-b" , "127.0.0.1"
-          , "-s" , config.proxy
-          , "-p" , config.remotePort.toString
-          , "-l" , config.localPort.toString
-          , "-k" , config.sitekey
-          , "-m" , config.encMethod
+          , "-c" , Path.BASE + "ss-local.json"
           , "-f" , Path.BASE + "ss-local.pid")
 
     if (config.route != Route.ALL) {
@@ -118,49 +121,59 @@ class ShadowsocksNatService extends Service with BaseService {
     }
 
     if (BuildConfig.DEBUG) Log.d(TAG, cmd.mkString(" "))
-
-    Core.sslocal(cmd.toArray)
+    Console.runCommand(cmd.mkString(" "))
   }
 
-  def startDnsDaemon() {
+  def startTunnel() {
     if (config.isUdpDns) {
+      val conf = ConfigUtils
+        .SHADOWSOCKS.format(config.proxy, config.remotePort, 8153,
+          config.sitekey, config.encMethod, 10)
+      ConfigUtils.printToFile(new File(Path.BASE + "ss-tunnel.json"))(p => {
+        p.println(conf)
+      })
       val cmd = new ArrayBuffer[String]
-      cmd += ("ss-tunnel" , "-u"
+      cmd += (Path.BASE + "ss-tunnel" , "-u"
             , "-b" , "127.0.0.1"
             , "-l" , "8153" 
             , "-L" , "8.8.8.8:53"
-            , "-s" , config.proxy
-            , "-p" , config.remotePort.toString
-            , "-k" , config.sitekey
-            , "-m" , config.encMethod
+            , "-c" , Path.BASE + "ss-tunnel.json"
             , "-f" , Path.BASE + "ss-tunnel.pid")
-      if (BuildConfig.DEBUG) Log.d(TAG, cmd.mkString(" "))
-      Core.sstunnel(cmd.toArray)
-    } else {
-      val cmdBuf = new ArrayBuffer[String]
-      cmdBuf += ("ss-tunnel"
-        , "-b", "127.0.0.1"
-        , "-l", "8163"
-        , "-L", "8.8.8.8:53"
-        , "-s", config.proxy
-        , "-p", config.remotePort.toString
-        , "-k", config.sitekey
-        , "-m", config.encMethod
-        , "-f", Path.BASE + "ss-tunnel.pid")
-      if (BuildConfig.DEBUG) Log.d(TAG, cmdBuf.mkString(" "))
-      Core.sstunnel(cmdBuf.toArray)
 
+      if (BuildConfig.DEBUG) Log.d(TAG, cmd.mkString(" "))
+      Console.runCommand(cmd.mkString(" "))
+    } else {
       val conf = ConfigUtils
-        .PDNSD_BYPASS.format("127.0.0.1", getString(R.string.exclude), 8163)
-      ConfigUtils.printToFile(new File(Path.BASE + "pdnsd.conf"))(p => {
-         p.println(conf)
+        .SHADOWSOCKS.format(config.proxy, config.remotePort, 8163,
+          config.sitekey, config.encMethod, 10)
+      ConfigUtils.printToFile(new File(Path.BASE + "ss-tunnel.json"))(p => {
+        p.println(conf)
       })
-      val cmd = Path.BASE + "pdnsd -c " + Path.BASE + "pdnsd.conf"
-      if (BuildConfig.DEBUG) Log.d(TAG, cmd)
-      Core.pdnsd(cmd.split(" "))
+      val cmdBuf = new ArrayBuffer[String]
+      cmdBuf += (Path.BASE + "ss-tunnel"
+        , "-b" , "127.0.0.1"
+        , "-l" , "8163"
+        , "-L" , "8.8.8.8:53"
+        , "-c" , Path.BASE + "ss-tunnel.json"
+        , "-f" , Path.BASE + "ss-tunnel.pid")
+
+      if (BuildConfig.DEBUG) Log.d(TAG, cmdBuf.mkString(" "))
+      Console.runCommand(cmdBuf.mkString(" "))
     }
   }
 
+  def startDnsDaemon() {
+    val conf = ConfigUtils
+      .PDNSD_BYPASS.format("127.0.0.1", getString(R.string.exclude), 8163)
+    ConfigUtils.printToFile(new File(Path.BASE + "pdnsd.conf"))(p => {
+       p.println(conf)
+    })
+    val cmd = Path.BASE + "pdnsd -c " + Path.BASE + "pdnsd.conf"
+
+    if (BuildConfig.DEBUG) Log.d(TAG, cmd)
+    Console.runCommand(cmd)
+  }
+
   def getVersionName: String = {
     var version: String = null
     try {
@@ -175,20 +188,21 @@ class ShadowsocksNatService extends Service with BaseService {
 
   def startRedsocksDaemon() {
     val conf = ConfigUtils.REDSOCKS.format(config.localPort)
-    val cmd = "redsocks -p %sredsocks.pid -c %sredsocks.conf"
+    val cmd = Path.BASE + "redsocks -p %sredsocks.pid -c %sredsocks.conf"
       .format(Path.BASE, Path.BASE)
     ConfigUtils.printToFile(new File(Path.BASE + "redsocks.conf"))(p => {
       p.println(conf)
     })
-    if (BuildConfig.DEBUG) Log.d(TAG, cmd)
 
-    Core.redsocks(cmd.split(" "))
+    if (BuildConfig.DEBUG) Log.d(TAG, cmd)
+    Console.runCommand(cmd);
   }
 
   /** Called when the activity is first created. */
   def handleConnection: Boolean = {
 
-    startDnsDaemon()
+    startTunnel()
+    if (!config.isUdpDns) startDnsDaemon()
     startRedsocksDaemon()
     startShadowsocksDaemon()
     setupIptables()
@@ -266,22 +280,16 @@ class ShadowsocksNatService extends Service with BaseService {
   }
 
   def killProcesses() {
+    for (task <- Array("ss-local", "ss-tunnel", "pdnsd", "redsocks")) {
+      try {
+        val pid = scala.io.Source.fromFile(Path.BASE + task + ".pid").mkString.trim.toInt
+        Process.killProcess(pid)
+        Log.d(TAG, "kill pid: " + pid)
+      } catch {
+        case e: Throwable => Log.e(TAG, "unable to kill " + task, e)
+      }
+    }
     Console.runRootCommand(Utils.getIptables + " -t nat -F OUTPUT")
-
-    val ab = new ArrayBuffer[String]
-
-    ab.append("kill -9 `cat " + Path.BASE + "ss-tunnel.pid`")
-    ab.append("kill -9 `cat " + Path.BASE +"redsocks.pid`")
-    ab.append("killall -9 redsocks")
-    ab.append("kill -15 `cat " + Path.BASE + "pdnsd.pid`")
-    ab.append("killall -15 pdnsd")
-
-    Console.runRootCommand(ab.toArray)
-
-    ab.clear()
-    ab.append("kill -9 `cat " + Path.BASE + "ss-local.pid`")
-
-    Console.runCommand(ab.toArray)
   }
 
   def flushDNS() {

src/main/scala/com/github/shadowsocks/ShadowsocksVpnService.scala

@@ -45,7 +45,7 @@ import java.net.InetAddress
 import android.app._
 import android.content._
 import android.content.pm.{PackageInfo, PackageManager}
-import android.net.VpnService
+import android.net.{LocalServerSocket, VpnService, LocalSocket, LocalSocketAddress}
 import android.os._
 import android.support.v4.app.NotificationCompat
 import android.util.Log
@@ -107,14 +107,17 @@ class ShadowsocksVpnService extends VpnService with BaseService {
       })
     }
 
+    val conf = ConfigUtils
+      .SHADOWSOCKS.format(config.proxy, config.remotePort, config.localPort,
+        config.sitekey, config.encMethod, 10)
+    ConfigUtils.printToFile(new File(Path.BASE + "ss-local.json"))(p => {
+      p.println(conf)
+    })
+
     val cmd = new ArrayBuffer[String]
-    cmd +=("ss-local", "-u"
+    cmd +=(Path.BASE + "ss-local", "-u"
       , "-b", "127.0.0.1"
-      , "-s", config.proxy
-      , "-p", config.remotePort.toString
-      , "-l", config.localPort.toString
-      , "-k", config.sitekey
-      , "-m", config.encMethod
+      , "-c", Path.BASE + "ss-local.json"
       , "-f", Path.BASE + "ss-local.pid")
 
     if (Utils.isLollipopOrAbove && config.route != Route.ALL) {
@@ -123,22 +126,26 @@ class ShadowsocksVpnService extends VpnService with BaseService {
     }
 
     if (BuildConfig.DEBUG) Log.d(TAG, cmd.mkString(" "))
-    Core.sslocal(cmd.toArray)
+    Console.runCommand(cmd.mkString(" "))
   }
 
   def startDnsTunnel() = {
+    val conf = ConfigUtils
+      .SHADOWSOCKS.format(config.proxy, config.remotePort, 8163,
+        config.sitekey, config.encMethod, 10)
+    ConfigUtils.printToFile(new File(Path.BASE + "ss-tunnel.json"))(p => {
+      p.println(conf)
+    })
     val cmd = new ArrayBuffer[String]
-    cmd +=("ss-tunnel"
+    cmd +=(Path.BASE + "ss-tunnel"
       , "-b", "127.0.0.1"
       , "-l", "8163"
       , "-L", "8.8.8.8:53"
-      , "-s", config.proxy
-      , "-p", config.remotePort.toString
-      , "-k", config.sitekey
-      , "-m", config.encMethod
+      , "-c", Path.BASE + "ss-tunnel.json"
       , "-f", Path.BASE + "ss-tunnel.pid")
+
     if (BuildConfig.DEBUG) Log.d(TAG, cmd.mkString(" "))
-    Core.sstunnel(cmd.toArray)
+    Console.runCommand(cmd.mkString(" "))
   }
 
   def startDnsDaemon() {
@@ -153,8 +160,9 @@ class ShadowsocksVpnService extends VpnService with BaseService {
       p.println(conf)
     })
     val cmd = Path.BASE + "pdnsd -c " + Path.BASE + "pdnsd.conf"
+
     if (BuildConfig.DEBUG) Log.d(TAG, cmd)
-    Core.pdnsd(cmd.split(" "))
+    Console.runCommand(cmd)
   }
 
   def getVersionName: String = {
@@ -208,27 +216,6 @@ class ShadowsocksVpnService extends VpnService with BaseService {
     } else {
       if (!Utils.isLollipopOrAbove) {
         config.route match {
-          case Route.ALL =>
-            for (i <- 1 to 223) {
-              if (i != 26 && i != 127) {
-                val addr = i.toString + ".0.0.0"
-                val cidr = addr + "/8"
-                val net = new SubnetUtils(cidr)
-
-                if (!isByass(net)) {
-                  builder.addRoute(addr, 8)
-                } else {
-                  for (j <- 0 to 255) {
-                    val subAddr = i.toString + "." + j.toString + ".0.0"
-                    val subCidr = subAddr + "/16"
-                    val subNet = new SubnetUtils(subCidr)
-                    if (!isByass(subNet)) {
-                      builder.addRoute(subAddr, 16)
-                    }
-                  }
-                }
-              }
-            }
           case Route.BYPASS_LAN =>
             for (i <- 1 to 223) {
               if (i != 26 && i != 127) {
@@ -265,6 +252,27 @@ class ShadowsocksVpnService extends VpnService with BaseService {
                 builder.addRoute(addr(0), addr(1).toInt)
               }
             })
+          case _ =>
+            for (i <- 1 to 223) {
+              if (i != 26 && i != 127) {
+                val addr = i.toString + ".0.0.0"
+                val cidr = addr + "/8"
+                val net = new SubnetUtils(cidr)
+
+                if (!isByass(net)) {
+                  builder.addRoute(addr, 8)
+                } else {
+                  for (j <- 0 to 255) {
+                    val subAddr = i.toString + "." + j.toString + ".0.0"
+                    val subCidr = subAddr + "/16"
+                    val subNet = new SubnetUtils(subCidr)
+                    if (!isByass(subNet)) {
+                      builder.addRoute(subAddr, 16)
+                    }
+                  }
+                }
+              }
+            }
         }
       } else {
         if (config.route == Route.ALL) {
@@ -312,19 +320,23 @@ class ShadowsocksVpnService extends VpnService with BaseService {
     else
       cmd += " --dnsgw %s:8153".format(PRIVATE_VLAN.format("1"))
 
+    if (Utils.isLollipopOrAbove) {
+      cmd += " --fake-proc";
+    }
+
     if (BuildConfig.DEBUG) Log.d(TAG, cmd)
 
-    Core.tun2socks(cmd.split(" "))
+    System.exec(cmd);
   }
 
   /** Called when the activity is first created. */
   def handleConnection: Boolean = {
-    startVpn()
     startShadowsocksDaemon()
     if (!config.isUdpDns) {
       startDnsDaemon()
       startDnsTunnel()
     }
+    startVpn()
     true
   }
 
@@ -352,7 +364,7 @@ class ShadowsocksVpnService extends VpnService with BaseService {
   }
 
   def killProcesses() {
-    for (task <- Array("ss-local", "ss-tunnel", "tun2socks", "pdnsd")) {
+    for (task <- Array("ss-local", "ss-tunnel", "pdnsd", "tun2socks")) {
       try {
         val pid = scala.io.Source.fromFile(Path.BASE + task + ".pid").mkString.trim.toInt
         Process.killProcess(pid)

src/main/scala/com/github/shadowsocks/utils/ConfigUtils.scala

@@ -46,7 +46,7 @@ import scalaj.http.{HttpOptions, Http}
 import com.github.shadowsocks.aidl.Config
 
 object ConfigUtils {
-  val SHADOWSOCKS = "{\"server\": [%s], \"server_port\": %d, \"local_port\": %d, \"password\": %s, \"timeout\": %d}"
+  val SHADOWSOCKS = "{\"server\": \"%s\", \"server_port\": %d, \"local_port\": %d, \"password\": \"%s\", \"method\":\"%s\", \"timeout\": %d}"
   val REDSOCKS = "base {" +
     " log_debug = off;" +
     " log_info = off;" +