Merge pull request #16 from JasperJH/master

Fixes searchFieldByClass looping forever and incorrect cache keys

Author: paulzhng

src/main/java/net/tcpshield/tcpshield/ReflectionUtils.java

@@ -73,16 +73,16 @@ public static Field searchFieldByClass(Class<?> clazz, Class<?> searchFor) {
         if (cachedField != null) return cachedField;
 
         Class<?> currentClass = clazz;
-        while (currentClass.getSuperclass() != null) {
-            for (Field field : clazz.getDeclaredFields()) {
+        do { 
+            for (Field field : currentClass.getDeclaredFields()) {
                 if (field.getType() != searchFor) continue;
 
                 CACHED_FIELDS_BY_CLASS.put(clazz, searchFor, field);
                 return field;
             }
 
-            currentClass = clazz.getSuperclass();
-        }
+            currentClass = currentClass.getSuperclass();
+        } while (currentClass != null);
 
         throw new IllegalArgumentException("no " + searchFor.getName() + " field for clazz = " + clazz.getName() + " found");
     }