Skip to content

Commit 5801597

Browse files
committed
Ack, wrap advisePersistThreshold() in lock.
Bug: 6449725 Change-Id: I702dc2ff9d4b237c46d89bcae41289bc3e9ef2f8
1 parent 3c5f924 commit 5801597

File tree

1 file changed

+6
-4
lines changed

1 file changed

+6
-4
lines changed

services/java/com/android/server/net/NetworkStatsService.java

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -616,10 +616,12 @@ public void advisePersistThreshold(long thresholdBytes) {
616616
// persist if beyond new thresholds
617617
final long currentTime = mTime.hasCache() ? mTime.currentTimeMillis()
618618
: System.currentTimeMillis();
619-
mDevRecorder.maybePersistLocked(currentTime);
620-
mXtRecorder.maybePersistLocked(currentTime);
621-
mUidRecorder.maybePersistLocked(currentTime);
622-
mUidTagRecorder.maybePersistLocked(currentTime);
619+
synchronized (mStatsLock) {
620+
mDevRecorder.maybePersistLocked(currentTime);
621+
mXtRecorder.maybePersistLocked(currentTime);
622+
mUidRecorder.maybePersistLocked(currentTime);
623+
mUidTagRecorder.maybePersistLocked(currentTime);
624+
}
623625

624626
// re-arm global alert
625627
registerGlobalAlert();

0 commit comments

Comments
 (0)