Skip to content

Commit edea6c5

Browse files
committed
remove unused log
1 parent 658beab commit edea6c5

File tree

3 files changed

+11
-10
lines changed

3 files changed

+11
-10
lines changed

rasp/jvm/JVMProbe/src/main/java/com/security/smith/SmithProbeProxy.java

+2-2
Original file line numberDiff line numberDiff line change
@@ -167,8 +167,8 @@ public boolean checkReflectEvil(String classname, String fieldname, boolean isMe
167167

168168
JsRuleResult result = SmithProbeObj.getJsRuleEngine().detect(2, argsX);
169169
if (result != null) {
170-
SmithLogger.logger.info("classname = " + classname + ", fieldname = " + fieldname + ", result = " + result.rulename);
171-
SmithLogger.logger.info("classname = " + classname + ", fieldname = " + fieldname + ", result = " + result.ruleid);
170+
// SmithLogger.logger.info("classname = " + classname + ", fieldname = " + fieldname + ", result = " + result.rulename);
171+
// SmithLogger.logger.info("classname = " + classname + ", fieldname = " + fieldname + ", result = " + result.ruleid);
172172
return true;
173173
}
174174
} catch (Throwable e) {

rasp/jvm/JVMProbe/src/main/java/com/security/smith/log/SmithLogger.java

+1
Original file line numberDiff line numberDiff line change
@@ -48,6 +48,7 @@ public static void loggerProberUnInit() {
4848

4949
if (logger != null) {
5050
logger.removeHandler(fileHandler);
51+
logger = null;
5152
}
5253
fileHandler.close();
5354
fileHandler = null;

rasp/jvm/JVMProbe/src/main/java/com/security/smith/rulemgr/StackRuleManager.java

+8-8
Original file line numberDiff line numberDiff line change
@@ -202,7 +202,7 @@ public boolean isMatched(Integer ruleId, boolean isWhite) {
202202
needClearStack = true;
203203
formatStack();
204204
}
205-
SmithLogger.logger.info("isWhite: " + isWhite + " ruleId: " + ruleId);
205+
// SmithLogger.logger.info("isWhite: " + isWhite + " ruleId: " + ruleId);
206206

207207
try {
208208
StackRule stackRule = stackRuleMaps.get(ruleId);
@@ -223,7 +223,7 @@ public boolean isMatched(Integer ruleId, boolean isWhite) {
223223
for (Map.Entry<Long, String> entry : curStack.entrySet()) {
224224
String value = entry.getValue();
225225
if (value.contains(stack.substring(0, stack.length() - 1))) {
226-
SmithLogger.logger.info("find the match, string: " + value + " hashcode: " + entry.getKey());
226+
// SmithLogger.logger.info("find the match, string: " + value + " hashcode: " + entry.getKey());
227227
return true;
228228
}
229229
}
@@ -234,7 +234,7 @@ public boolean isMatched(Integer ruleId, boolean isWhite) {
234234
} else {
235235
try {
236236
if (curStack.containsKey(item.getHashcode())) {
237-
SmithLogger.logger.info("find the match, string: " + curStack.get(item.getHashcode()) + " hashcode: " + item.getHashcode());
237+
// SmithLogger.logger.info("find the match, string: " + curStack.get(item.getHashcode()) + " hashcode: " + item.getHashcode());
238238
return true;
239239
}
240240
} catch (Exception e) {
@@ -253,7 +253,7 @@ public boolean isMatched(Integer ruleId, boolean isWhite) {
253253
}
254254
}
255255

256-
SmithLogger.logger.info("not find the match");
256+
// SmithLogger.logger.info("not find the match");
257257
} catch (Exception e) {
258258
SmithLogger.exception(e);
259259
}
@@ -266,8 +266,8 @@ public void formatStack() {
266266
}
267267
try {
268268
Map<Long, String> cur1Stack = currentStack.get();
269-
SmithLogger.logger.info("curStack size: " + cur1Stack.size());
270-
SmithLogger.logger.info("now to get current stack");
269+
// SmithLogger.logger.info("curStack size: " + cur1Stack.size());
270+
//SmithLogger.logger.info("now to get current stack");
271271
StackTraceElement[] stack_str = Thread.currentThread().getStackTrace();
272272
String[] stackTraceStrings = new String[stack_str.length];
273273
Map<Long, String> curStack = new HashMap<>();
@@ -293,9 +293,9 @@ public void clearStack() {
293293
}
294294
try {
295295
Map<Long, String> curStack = currentStack.get();
296-
SmithLogger.logger.info("curStack size: " + curStack.size());
296+
//SmithLogger.logger.info("curStack size: " + curStack.size());
297297
curStack.clear();
298-
SmithLogger.logger.info("clear currentStack");
298+
//SmithLogger.logger.info("clear currentStack");
299299
} catch (Exception e) {
300300
SmithLogger.exception(e);
301301
}

0 commit comments

Comments
 (0)