Commit e1d96937 authored Mar 24, 2022 by Jack Yu

Fixed a race condition when preferred transport updated

This bug is only applicable on the old data stack on T.
Transport manager and access network manager might be running
on a different thread. Fixed a race condition on old data stack
that was introduced in early new data stack development time frame.
The transport evaluation could happen earlier than setting the
preferred transport due to the race condition.

Fixed by setting the preferred transport earlier than notifying
transport manager.

Fix: 226677098
Test: Sanity tests && atest FrameworksTelephonyTests
Change-Id: Id856902c9a73942913e041dc5396b5d17b661089

parent 781b33ed

src/java/com/android/internal/telephony/data/AccessNetworksManager.java

+1 −1

Original line number	Diff line number	Diff line
		@@ -347,8 +347,8 @@ public class AccessNetworksManager extends Handler {
		}

		if (!qualifiedNetworksList.isEmpty()) {
		mQualifiedNetworksChangedRegistrants.notifyResult(qualifiedNetworksList);
		setPreferredTransports(qualifiedNetworksList);
		mQualifiedNetworksChangedRegistrants.notifyResult(qualifiedNetworksList);
		}
		}
		}