commit 664d208a23be1bc6a0b1d6efdf62b711885394bc
author Erik Kline <ek@google.com> Thu Jun 15 18:06:34 2017 +0900
committer Erik Kline <ek@google.com> Fri Jun 16 21:11:39 2017 +0900
tree faf40b184568b5accf41142d1d87a707c88be45c
parent 8a945baf4acd0eb35c1b894c889fa47c5a73463e

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

Merged-In: I5fa930be0c9eda491cf17bd4de9b55ab33672d25
Merged-In: Id1037d22826f4d426bccfa17dce0962c54518d64
Change-Id: I91314440c3a04e5f2502579b5f06dac9f25cf0cd
(cherry picked from commit 1185459c4169891bd7550655218e738c8bf1abe2)