Prevent transformer list from being modified

Author: ThisTestUser

src/java/com/javadeobfuscator/deobfuscator/ui/FxWindow.java

@@ -1,7 +1,8 @@
 package com.javadeobfuscator.deobfuscator.ui;
 
 import java.io.PrintStream;
-
+import java.util.Collection;
+import java.util.List;
 import javax.swing.JOptionPane;
 
 import org.controlsfx.control.ListSelectionView;
@@ -14,6 +15,7 @@
 
 import javafx.application.Application;
 import javafx.application.Platform;
+import javafx.collections.ObservableListBase;
 import javafx.event.ActionEvent;
 import javafx.event.EventHandler;
 import javafx.scene.Scene;
@@ -52,7 +54,7 @@ protected void updateItem(Class<?> item, boolean empty) {
 				}
 			}
 		});
-		selectedTransformers.getSourceItems().addAll(trans.getTransformers());
+		selectedTransformers.setSourceItems(new ImmutableTransformerList(trans.getTransformers()));
 		TitledPane wrapper2 = new TitledPane("Transformers", selectedTransformers);
 		// log
 		ListView<String> logging = new ListView<>();
@@ -176,4 +178,114 @@ public static void fatalFX(String title, String text) {
 			.hideAfter(time).showError();
 		//@formatter:on
 	}
+	
+	private class ImmutableTransformerList extends ObservableListBase<Class<?>> 
+	{
+		private List<Class<?>> value;
+		
+		public ImmutableTransformerList(List<Class<?>> value)
+		{
+			super.addAll(value);
+			this.value = value;
+		}
+		
+		@Override
+		public boolean add(Class<?> arg0)
+		{
+			return false;
+		}
+
+		@Override
+		public void add(int arg0, Class<?> arg1)
+		{
+		}
+
+		@Override
+		public boolean addAll(Collection<? extends Class<?>> arg0)
+		{
+			return false;
+		}
+
+		@Override
+		public boolean addAll(int arg0, Collection<? extends Class<?>> arg1)
+		{
+			return false;
+		}
+
+		@Override
+		public void clear()
+		{
+		}
+
+		@Override
+		public boolean remove(Object arg0)
+		{
+			return false;
+		}
+
+		@Override
+		public Class<?> remove(int arg0)
+		{
+			return null;
+		}
+
+		@Override
+		public boolean removeAll(Collection<?> c)
+		{
+			return false;
+		}
+
+		@Override
+		public boolean retainAll(Collection<?> c)
+		{
+			return false;
+		}
+
+		@Override
+		public Class<?> set(int index, Class<?> element)
+		{
+			return null;
+		}
+		
+		@Override
+		public boolean addAll(Class<?>... elements)
+		{
+			return false;
+		}
+
+		@Override
+		public boolean setAll(Class<?>... elements)
+		{
+			return false;
+		}
+
+		@Override
+		public boolean setAll(Collection<? extends Class<?>> col)
+		{
+			return false;
+		}
+
+		@Override
+		public boolean removeAll(Class<?>... elements)
+		{
+			return false;
+		}
+
+		@Override
+		public void remove(int from, int to)
+		{
+		}
+
+		@Override
+		public Class<?> get(int arg0)
+		{
+			return value.get(arg0);
+		}
+
+		@Override
+		public int size()
+		{
+			return value.size();
+		}
+	}
 }

src/java/com/javadeobfuscator/deobfuscator/ui/component/StringListEditor.java

@@ -1,6 +1,5 @@
 package com.javadeobfuscator.deobfuscator.ui.component;
 
-import java.io.File;
 import java.util.List;
 
 import org.controlsfx.control.PropertySheet.Item;

src/java/com/javadeobfuscator/deobfuscator/ui/wrap/Transformers.java

@@ -1,15 +1,12 @@
 package com.javadeobfuscator.deobfuscator.ui.wrap;
 
-import java.lang.reflect.Field;
 import java.lang.reflect.Method;
 import java.util.ArrayList;
 import java.util.Collections;
 import java.util.List;
-import java.util.Set;
 
 import com.javadeobfuscator.deobfuscator.ui.FxWindow;
 import com.javadeobfuscator.deobfuscator.ui.util.ByteLoader;
-import com.javadeobfuscator.deobfuscator.ui.util.Reflect;
 
 public class Transformers {
 	// load with:

src/java/com/javadeobfuscator/deobfuscator/ui/wrap/WrapperFactory.java

@@ -151,7 +151,14 @@ private static Map<String, byte[]> readClasses(File jar) throws IOException, Inv
 					// skip non-classes (Deobfuscator doesn't have any resources aside for
 					// META, which we will bundle to appease SLF4J's ServiceLoader screwery.)
 					byte[] value = from(is);
-					String className = new MiniClassReader(value).getClassName();
+					String className;
+					try
+					{
+						className = new MiniClassReader(value).getClassName();
+					}catch(Exception e)
+					{
+						continue;
+					}
 					// We know this class is in the deobfuscator jar, so if the jar does 
 					// not contain it, it is not the correct file.
 					if (className.contains("com/javadeobfuscator/deobfuscator/Deobfuscator")) {