Fix some alt-network issues.
Two issues. 1) remove default routes for non-default networks. 2) don't report mobile is the active default network just because it is active. bug:4157610 Change-Id: I9e7c94718a5b1f08840b219b304ba3904259a65f
This commit is contained in:
@@ -591,18 +591,8 @@ public class ConnectivityService extends IConnectivityManager.Stub {
|
|||||||
*/
|
*/
|
||||||
public NetworkInfo getActiveNetworkInfo() {
|
public NetworkInfo getActiveNetworkInfo() {
|
||||||
enforceAccessPermission();
|
enforceAccessPermission();
|
||||||
for (int type=0; type <= ConnectivityManager.MAX_NETWORK_TYPE; type++) {
|
if (mActiveDefaultNetwork != -1) {
|
||||||
if (mNetAttributes[type] == null || !mNetAttributes[type].isDefault()) {
|
return mNetTrackers[mActiveDefaultNetwork].getNetworkInfo();
|
||||||
continue;
|
|
||||||
}
|
|
||||||
NetworkStateTracker t = mNetTrackers[type];
|
|
||||||
NetworkInfo info = t.getNetworkInfo();
|
|
||||||
if (info.isConnected()) {
|
|
||||||
if (DBG && type != mActiveDefaultNetwork) Slog.e(TAG,
|
|
||||||
"connected default network is not " +
|
|
||||||
"mActiveDefaultNetwork!");
|
|
||||||
return info;
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
return null;
|
return null;
|
||||||
}
|
}
|
||||||
@@ -1338,6 +1328,15 @@ public class ConnectivityService extends IConnectivityManager.Stub {
|
|||||||
if (mNetAttributes[netType].isDefault()) {
|
if (mNetAttributes[netType].isDefault()) {
|
||||||
mNetTrackers[netType].addDefaultRoute();
|
mNetTrackers[netType].addDefaultRoute();
|
||||||
} else {
|
} else {
|
||||||
|
// many radios add a default route even when we don't want one.
|
||||||
|
// remove the default interface unless we need it for our active network
|
||||||
|
if (mActiveDefaultNetwork != -1) {
|
||||||
|
String defaultIface = mNetTrackers[mActiveDefaultNetwork].getInterfaceName();
|
||||||
|
if (defaultIface != null &&
|
||||||
|
!defaultIface.equals(mNetTrackers[netType].getInterfaceName())) {
|
||||||
|
mNetTrackers[netType].removeDefaultRoute();
|
||||||
|
}
|
||||||
|
}
|
||||||
mNetTrackers[netType].addPrivateDnsRoutes();
|
mNetTrackers[netType].addPrivateDnsRoutes();
|
||||||
}
|
}
|
||||||
} else {
|
} else {
|
||||||
|
|||||||
Reference in New Issue
Block a user