Merge "prevent nullptr dereference when calling unwanted"

This commit is contained in:
Treehugger Robot
2021-08-20 08:57:35 +00:00
committed by Gerrit Code Review

View File

@@ -506,12 +506,15 @@ public class EthernetNetworkFactory extends NetworkFactory {
mNetworkFactory.getProvider(), new EthernetNetworkAgent.Callbacks() { mNetworkFactory.getProvider(), new EthernetNetworkAgent.Callbacks() {
@Override @Override
public void onNetworkUnwanted() { public void onNetworkUnwanted() {
// if mNetworkAgent is null, we have already called stop.
if (mNetworkAgent == null) return;
if (this == mNetworkAgent.getCallbacks()) { if (this == mNetworkAgent.getCallbacks()) {
stop(); stop();
} else if (mNetworkAgent != null) { } else {
Log.d(TAG, "Ignoring unwanted as we have a more modern " + Log.d(TAG, "Ignoring unwanted as we have a more modern " +
"instance"); "instance");
} // Otherwise, we've already called stop. }
} }
}); });
mNetworkAgent.register(); mNetworkAgent.register();