From 5d819563d3969ab2f39464fd1cd9e86e22ef9c3e Mon Sep 17 00:00:00 2001 From: hydrogen666 Date: Fri, 16 Aug 2019 15:13:42 +0800 Subject: [PATCH] Throw IllegalAccessException out when Configure set filed failed. (#818) --- .../com/taobao/arthas/core/config/Configure.java | 12 ++++-------- 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/core/src/main/java/com/taobao/arthas/core/config/Configure.java b/core/src/main/java/com/taobao/arthas/core/config/Configure.java index ad351fb0d..f0b41d037 100644 --- a/core/src/main/java/com/taobao/arthas/core/config/Configure.java +++ b/core/src/main/java/com/taobao/arthas/core/config/Configure.java @@ -122,18 +122,14 @@ public class Configure { * @param toString 序列化字符串 * @return 反序列化的对象 */ - public static Configure toConfigure(String toString) { + public static Configure toConfigure(String toString) throws IllegalAccessException { final Configure configure = new Configure(); final Map map = codec.toMap(toString); for (Map.Entry entry : map.entrySet()) { - try { - final Field field = ArthasReflectUtils.getField(Configure.class, entry.getKey()); - if (null != field && !isStatic(field.getModifiers())) { - ArthasReflectUtils.set(field, ArthasReflectUtils.valueOf(field.getType(), entry.getValue()), configure); - } - } catch (Throwable t) { - // + final Field field = ArthasReflectUtils.getField(Configure.class, entry.getKey()); + if (null != field && !isStatic(field.getModifiers())) { + ArthasReflectUtils.set(field, ArthasReflectUtils.valueOf(field.getType(), entry.getValue()), configure); } } return configure;