Block modifying types of channels

The type of channels should not be changed once it's set.
Blocks this column from being modified.

Test: cts-tradefed run cts -m CtsTvTestCases
Change-Id: I089ca51a4861fdb0b72b1a0efdb18f6f469c87cc

1 files changed, 15 insertions, 1 deletions

diff --git a/src/com/android/providers/tv/TvProvider.java b/src/com/android/providers/tv/TvProvider.java
index b6b660d..842244c 100644
--- a/src/com/android/providers/tv/TvProvider.java
+++ b/src/com/android/providers/tv/TvProvider.java
@@ -1421,6 +1421,11 @@ public class TvProvider extends ContentProvider {
         boolean containUnmodifiableColumn = false;
         if (params.getTables().equals(CHANNELS_TABLE)) {
             filterContentValues(values, sChannelProjectionMap);
+            containUnmodifiableColumn = disallowModifyChannelType(values, params);
+            if (containUnmodifiableColumn && Channels.CONTENT_URI.equals(uri)) {
+                Log.i(TAG, "Updating failed. Attempt to change unmodifiable column for channels.");
+                return 0;
+            }
             blockIllegalAccessToChannelsSystemColumns(values);
         } else if (params.getTables().equals(PROGRAMS_TABLE)) {
             filterContentValues(values, sProgramProjectionMap);
@@ -1806,9 +1811,18 @@ public class TvProvider extends ContentProvider {
         }
     }
 
+    private boolean disallowModifyChannelType(ContentValues values, SqlParams params) {
+        if (values.containsKey(Channels.COLUMN_TYPE)) {
+            params.appendWhere(Channels.COLUMN_TYPE + "=?",
+                    values.getAsString(Channels.COLUMN_TYPE));
+            return true;
+        }
+        return false;
+    }
+
     private boolean disallowModifyChannelId(ContentValues values, SqlParams params) {
         if (values.containsKey(PreviewPrograms.COLUMN_CHANNEL_ID)) {
-            params.appendWhere(Programs.COLUMN_CHANNEL_ID + "=?",
+            params.appendWhere(PreviewPrograms.COLUMN_CHANNEL_ID + "=?",
                     values.getAsString(PreviewPrograms.COLUMN_CHANNEL_ID));
             return true;
         }