commit a68120512c3930e30912379385a11502931dfcaf
author Wink Saville <wink@google.com> Wed Mar 27 11:12:23 2013 -0700
committer Android Git Automerger <android-git-automerger@android.com> Wed Mar 27 11:12:23 2013 -0700
tree faed561c43a17a3a5fc0f2ff603cdf6de74a542f
parent 03147422e8ba952de0b9f4de334f68f3ba0f2c35
parent 1695fa599191f6f8edc7ed4d3824f482b827d3cb

am 1695fa59: Merge "modify fail fast mode for loading apns"

* commit '1695fa599191f6f8edc7ed4d3824f482b827d3cb':
  modify fail fast mode for loading apns

src/com/android/providers/telephony/TelephonyProvider.java

diff --git a/src/com/android/providers/telephony/TelephonyProvider.java b/src/com/android/providers/telephony/TelephonyProvider.java
index 62435ee..b7cc7a9 100644
--- a/src/com/android/providers/telephony/TelephonyProvider.java
+++ b/src/com/android/providers/telephony/TelephonyProvider.java
@@ -342,21 +342,21 @@
             if (parser != null) {
                 try {
                     db.beginTransaction();
-                    while (true) {
-                        XmlUtils.nextElement(parser);
+                    XmlUtils.nextElement(parser);
+                    while (parser.getEventType() != XmlPullParser.END_DOCUMENT) {
                         ContentValues row = getRow(parser);
-                        if (row != null) {
-                            insertAddingDefaults(db, CARRIERS_TABLE, row);
-                        } else {
-                            break;  // do we really want to skip the rest of the file?
+                        if (row == null) {
+                            throw new XmlPullParserException("Expected 'apn' tag", parser, null);
                         }
+                        insertAddingDefaults(db, CARRIERS_TABLE, row);
+                        XmlUtils.nextElement(parser);
                     }
                     db.setTransactionSuccessful();
-                } catch (XmlPullParserException e)  {
-                    Log.e(TAG, "Got execption while loading apns.", e);
+                } catch (XmlPullParserException e) {
+                    Log.e(TAG, "Got XmlPullParserException while loading apns.", e);
                 } catch (IOException e) {
-                    Log.e(TAG, "Got IOExecption while loading apns.", e);
-                } catch (SQLException e){
+                    Log.e(TAG, "Got IOException while loading apns.", e);
+                } catch (SQLException e) {
                     Log.e(TAG, "Got SQLException while loading apns.", e);
                 } finally {
                     db.endTransaction();