1185459c4169891bd7550655218e738c8bf1abe2 - platform/frameworks/base

commit	1185459c4169891bd7550655218e738c8bf1abe2	[log] [tgz]
author	Erik Kline <ek@google.com>	Thu Jun 15 18:06:34 2017 +0900
committer	Erik Kline <ek@google.com>	Fri Jun 16 17:06:05 2017 +0900
tree	f0fa46fd1a673c4e03c26dad9d1547f2b7eb8e4a
parent	69a4e37d19542a9a88c939b7a61f0c3de39b5c1b [diff]

Fixes for tetheroffload crashes

Now that we can talk to the HALs (with some out of tree CLs and
"setenforce 0"), several crashes were encountered.

Fixes here include:
    - avoid hidl_handle move semantics
    - check HIDL method status return value (isOk())
    - convert Java short port numbers to ints
    - don't pass nulls to HIDL where Strings are required
      (limitations in parceling)

Test: as follows
    - built
    - flashed
    - booted
    - runtest frameworks-net passes
    - "setenforce 0" and start tethering
Bug: 29337859
Bug: 32163131

Change-Id: I91314440c3a04e5f2502579b5f06dac9f25cf0cd

services/core/java/com/android/server/connectivity/tethering/OffloadController.java[diff]
services/core/java/com/android/server/connectivity/tethering/OffloadHardwareInterface.java[diff]
services/core/jni/com_android_server_connectivity_tethering_OffloadHardwareInterface.cpp[diff]
tests/net/java/com/android/server/connectivity/tethering/OffloadControllerTest.java[diff]

4 files changed

tree: f0fa46fd1a673c4e03c26dad9d1547f2b7eb8e4a