diff options
author | Lorenzo Colitti <lorenzo@google.com> | 2020-02-18 00:53:59 +0900 |
---|---|---|
committer | Lorenzo Colitti <lorenzo@google.com> | 2020-02-18 10:50:58 +0900 |
commit | 2b413f31d4ed156b24987e65f2d8c3c74c980770 (patch) | |
tree | 5620d39a77e44ccf40bf1bf6169fcf7497ce7820 /src | |
parent | b4af853264c8d0083491e35b2e157355bf8919bd (diff) |
Never register a null lease callback.
Bug: 149458372
Test: new unit test
Test: tethering no longer crashes
Change-Id: Ic5f709c1ce50d3bb7af26a698dd32adb87012316
Diffstat (limited to 'src')
-rw-r--r-- | src/android/net/dhcp/DhcpServer.java | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/android/net/dhcp/DhcpServer.java b/src/android/net/dhcp/DhcpServer.java index bcca47a..9e54a69 100644 --- a/src/android/net/dhcp/DhcpServer.java +++ b/src/android/net/dhcp/DhcpServer.java @@ -359,7 +359,9 @@ public class DhcpServer extends IDhcpServer.Stub { final Pair<INetworkStackStatusCallback, IDhcpLeaseCallbacks> obj = (Pair<INetworkStackStatusCallback, IDhcpLeaseCallbacks>) msg.obj; cb = obj.first; - mLeaseRepo.addLeaseCallbacks(obj.second); + if (obj.second != null) { + mLeaseRepo.addLeaseCallbacks(obj.second); + } mPacketListener = mDeps.makePacketListener(); mPacketListener.start(); break; |