如何從代碼層防護10大安全威脅中的 Xpath Injection？

時間 2019-12-08

標籤如何代碼防護安全威脅 xpath injection 欄目系統安全简体版

原文原文鏈接

廣泛性和可檢測性：

Xpath 注入是 OWASP TOP10 安全威脅中 A1 Injection 中的一種，注入漏洞發生在應用程序將不可信的數據發送到解釋器時。雖然注入漏洞很容易經過審查代碼發現，可是卻不容易在測試中發現。html

影響嚴重：

注入能致使數據丟失或數據破壞、缺少可審計性或者是拒絕服務。注入漏洞有時候甚至能致使徹底主機接管。前端

從代碼層次如何防護：

首先咱們先來看一下在 Java 中引用 xpath 須要用的 lib 庫：java

javax.xml.xpath
org.jdom.xpath
org.jdom2.xpath等

那麼xpath注入是從哪些途徑進入到代碼邏輯的呢？你們仔細思考一下無外乎三個途徑：cookie、header、request parameters/input。若是咱們能對這三個注入源頭進行嚴格得入參檢查是否就可以防護絕大部分的注入攻擊了呢？git

答案是能夠防護大部分注入攻擊,下面咱們就一塊兒來看一下如何進行有效得進行入參檢查：咱們將入參都轉化爲 Map 對象，Map<K, Collection<V>> asMap();github

而後經過CheckMap（finnal Map<String, String> params）方法，檢查入參是否合法。安全

下面咱們來實現這個CheckMap內部方法：

1. 經過遍歷檢查Map中key得合法性cookie

for (final String key : params.keySet()) {
        if (this.checkString(key)) {
            return true;
        }

2.檢查每個 key 對應得 value 得合法性：網絡

final Collection<String> coll = (Collection<String>)params.get((Object)key);
        for (final String input : coll) {
            if (this.checkString(input)) {
                return true;
            }
        }

作完這些就是細節方面得檢測了，具體得就是 checkString 如何實現。筆者暫時能想到認爲一個入參是 xpath 得檢測條件大概有如下幾點：架構

private boolean checkString(final String input) {
    return null != input && input.length() > 1 && (this.parser.IsOutBoundary(input) || (-1 != input.indexOf(39) && (this.parser.IsOutBoundary(input.replace("'", "''")) || this.parser.IsOutBoundary(input.replace("'", "\\'")))) || (-1 != input.indexOf(34) && (this.parser.IsOutBoundary(input.replace("\"", "\"\"")) || this.parser.IsOutBoundary(input.replace("\"", "\\\"")))) || this.parser.IsQuoteUnbalanced(input));
}

經過查 ASCII 碼錶咱們知道39對應「'」，34對應「"」;因此有了檢測條件less

-1!=input.indexOf(39)&&(this.parser.IsOutBoundary(input.replace("'","''")
-1!=input.indexOf(34)&& this.parser.IsOutBoundary(input.replace("\"", "\"\""))

上述檢測條件中用到兩個關鍵方法IsOutBoundary和IsQuoteUnbalance

public boolean IsOutBoundary(String input) {
    int offset = 0;
    if (null == input || input.length() <= 1) {
        return false;
    }
    input = input.toLowerCase();
    while (true) {
        final int x = this.getRawValue().indexOf(input, offset);
        final int y = x + input.length();
        if (-1 == x) {
            return false;
        }
        final int ceil = this.getCeiling(this.boundaries, x + 1);
        if (-1 != ceil && ceil < y) {
            return true;
        }
        offset = y;
    }
}

public boolean IsQuoteUnbalanced(String input) {
    input = input.toLowerCase();
    return this.getRawValue().contains(input) && this.stack.size() > 0 && input.indexOf(this.stack.peek()) != -1;
}

 public String getRawValue() {
        return this.input;
    }
 
 private int getCeiling(final List<Integer> boundaries, final int value) {
    for (final int x : boundaries) {
        if (x >= value) {
            return x;
        }
    }
    return -1;
}