如何解决CWE 117问题

Question

String clean = args[1].replace('\n', '_').replace('\r', '_');

log.info(clean);

我的问题是，在log4j中是否有任何中心位置，在那里一个单独的更改就可以解决这个问题？

Answer 1

负责序列化日志消息的是布局，换行转换代码就属于这里。

我建议创建自己的（平凡的）PatternLayout子类来完成转换。这里的Log4j邮件列表中也讨论了这一点。下面是该线程中建议的解决方案的一个稍微修改版本：

import org.apache.log4j.PatternLayout;
import org.apache.log4j.spi.LoggingEvent;

public class NewLinePatternLayout extends PatternLayout {

    public NewLinePatternLayout() { }

    public NewLinePatternLayout(String pattern) {
        super(pattern);
    }

    public boolean ignoresThrowable() {
        return false;
    }

    public String format(LoggingEvent event) {
        String original = super.format(event);

        // Here your code comes into play
        String clean = original.replace('\n', '_').replace('\r', '_');

        StringBuilder sb = new StringBuilder(clean);

        String[] s = event.getThrowableStrRep();
        if (s != null) {
            for (int i = 0; i < s.length; i++) {
                sb.append(s[i]);
                sb.append('_');
            }
        }
        return sb.toString();
    }
}

相关问题（有可能有用的答案）：

null