Fix SparseArray accesses (360e454b) · Commits · e / os / android_frameworks_base

core/java/android/content/SyncStorageEngine.java

+130 −130

Original line number	Original line	Diff line number	Diff line
	@@ -369,7 +369,7 @@ public class SyncStorageEngine extends Handler {
	int i = mAuthorities.size();		int i = mAuthorities.size();
	while (i > 0) {		while (i > 0) {
	i--;		i--;
	AuthorityInfo authority = mAuthorities.get(i);		AuthorityInfo authority = mAuthorities.valueAt(i);
	if (authority.authority.equals(providerName)		if (authority.authority.equals(providerName)
	&& authority.enabled) {		&& authority.enabled) {
	return true;		return true;
	@@ -408,7 +408,7 @@ public class SyncStorageEngine extends Handler {
	int i = mAuthorities.size();		int i = mAuthorities.size();
	while (i > 0) {		while (i > 0) {
	i--;		i--;
	AuthorityInfo authority = mAuthorities.get(i);		AuthorityInfo authority = mAuthorities.valueAt(i);
	if (authority.authority.equals(providerName)) {		if (authority.authority.equals(providerName)) {
	return authority.syncable;		return authority.syncable;
	}		}
	@@ -581,7 +581,7 @@ public class SyncStorageEngine extends Handler {
	mPendingOperations.clear();		mPendingOperations.clear();
	final int N = mSyncStatus.size();		final int N = mSyncStatus.size();
	for (int i=0; i<N; i++) {		for (int i=0; i<N; i++) {
	mSyncStatus.get(i).pending = false;		mSyncStatus.valueAt(i).pending = false;
	}		}
	writePendingOperationsLocked();		writePendingOperationsLocked();
	}		}
	@@ -876,7 +876,7 @@ public class SyncStorageEngine extends Handler {
	SyncStatusInfo best = null;		SyncStatusInfo best = null;
	final int N = mSyncStatus.size();		final int N = mSyncStatus.size();
	for (int i=0; i<N; i++) {		for (int i=0; i<N; i++) {
	SyncStatusInfo cur = mSyncStatus.get(i);		SyncStatusInfo cur = mSyncStatus.valueAt(i);
	AuthorityInfo ainfo = mAuthorities.get(cur.authorityId);		AuthorityInfo ainfo = mAuthorities.get(cur.authorityId);
	if (ainfo != null && ainfo.authority.equals(authority)) {		if (ainfo != null && ainfo.authority.equals(authority)) {
	if (best == null) {		if (best == null) {
	@@ -897,7 +897,7 @@ public class SyncStorageEngine extends Handler {
	synchronized (mAuthorities) {		synchronized (mAuthorities) {
	final int N = mSyncStatus.size();		final int N = mSyncStatus.size();
	for (int i=0; i<N; i++) {		for (int i=0; i<N; i++) {
	SyncStatusInfo cur = mSyncStatus.get(i);		SyncStatusInfo cur = mSyncStatus.valueAt(i);
	AuthorityInfo ainfo = mAuthorities.get(cur.authorityId);		AuthorityInfo ainfo = mAuthorities.get(cur.authorityId);
	if (ainfo == null) {		if (ainfo == null) {
	continue;		continue;
	@@ -1187,7 +1187,7 @@ public class SyncStorageEngine extends Handler {

	final int N = mAuthorities.size();		final int N = mAuthorities.size();
	for (int i=0; i<N; i++) {		for (int i=0; i<N; i++) {
	AuthorityInfo authority = mAuthorities.get(i);		AuthorityInfo authority = mAuthorities.valueAt(i);
	out.startTag(null, "authority");		out.startTag(null, "authority");
	out.attribute(null, "id", Integer.toString(authority.ident));		out.attribute(null, "id", Integer.toString(authority.ident));
	out.attribute(null, "account", authority.account.name);		out.attribute(null, "account", authority.account.name);
	@@ -1290,7 +1290,7 @@ public class SyncStorageEngine extends Handler {
	SyncStatusInfo st = null;		SyncStatusInfo st = null;
	while (i > 0) {		while (i > 0) {
	i--;		i--;
	st = mSyncStatus.get(i);		st = mSyncStatus.valueAt(i);
	if (st.authorityId == authority.ident) {		if (st.authorityId == authority.ident) {
	found = true;		found = true;
	break;		break;
	@@ -1333,7 +1333,7 @@ public class SyncStorageEngine extends Handler {
	int i = mAuthorities.size();		int i = mAuthorities.size();
	while (i > 0) {		while (i > 0) {
	i--;		i--;
	AuthorityInfo authority = mAuthorities.get(i);		AuthorityInfo authority = mAuthorities.valueAt(i);
	if (authority.authority.equals(provider)) {		if (authority.authority.equals(provider)) {
	authority.enabled = value == null \|\| Boolean.parseBoolean(value);		authority.enabled = value == null \|\| Boolean.parseBoolean(value);
	authority.syncable = 1;		authority.syncable = 1;