Skip to content

Commit fb0caad

Browse files
jsharkeyAndroid (Google) Code Review
authored andcommitted
Merge "Ack, wrap advisePersistThreshold() in lock." into jb-dev
2 parents ee00b31 + 5801597 commit fb0caad

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)