| dc0d578a69cc5a57167a508207e2198590142d51 |
|
20-Jul-2016 |
Robin Lee <rgl@google.com> |
UidRanges: use class instead of pair<uid_t, uid_t>
Reuse the UidRange that was introduced in 7.0 (NYC) to ease transition
from CommandListener to binder which supports this as a parcelable type.
There is a small difference in behaviour: UidRange uses signed int32_t
vs. uid_t being unsigned and potentially a different size. This should
not be a problem as all of the java-side code is converting from int.
Updating to use int64_t in future would be a large effort and involve
changing the java-side UidRange class to use longs, and not fixing the
native side would cause unit tests to fail, so it shouldn't be possible
to overlook if that happens.
Committing this early with an appropriately loud warning so that it can
get soak time over the next year.
Test: runtest -x netd_integration_test.cpp
Change-Id: I6c217b347724ba5bfe2df28d6142a4343cb06353
/system/netd/server/binder/android/net/UidRange.cpp
|
| 095ec2f699e9ffa0247f5275c761b5724fbbac42 |
|
03-Jun-2016 |
Robin Lee <rgl@google.com> |
UidRange bounds-checking typo
This is from copy-pasting the line above and changing the assert
message but not the actual check.
(cherry picked from commit 927c40f40148880ce6f6bbe5335c2741b4195ad3)
Change-Id: I5b7c17a5ac3ba07353e57398a8278e7e0e3e6d9f
/system/netd/server/binder/android/net/UidRange.cpp
|
| 9f9aae9102f62f5f96ccec670170ee1fb262ef09 |
|
30-Mar-2016 |
Robin Lee <rgl@google.com> |
Move UidRange aidl to create a native cpp version
Moved from:
//frameworks/base/core/java/android/net/
To:
//system/netd/binder
Since frameworks/base depends on netd but not vice versa, it is cleaner
to keep the internal aidl in the same place as the native implementation
in netd.
Bug: 26694104
Change-Id: If21a72978ad5b93f0eed04c75143b55157c1a014
/system/netd/server/binder/android/net/UidRange.cpp
|