經得起原始碼資安弱點掃描的程式設計習慣培養(二)_8.Insecure Deserialization ：不安全的反序列化漏洞ASP.NET處理方式

-
近期在專案偕同上被Checkmarx掃碼掃到高風險的程式片段
偵測到Deserialization of Untrusted Data (OWASP  2017第8點Insecure Deserialization)
因此要做程式碼安全品質的修正








情境:
Call web api 回傳回來(讀取外部的json測試檔案)的json  字串要反序列化轉為DataTable
for GridView Bind呈現



CheckMarx掃碼報告給的說明
============================================================
Deserialization of Untrusted Data
風險:
對不受信任的資料進行反序列化，可能允許攻擊者提供惡意的物件給反序列化程式碼使用。如果危險物件被不安全地反序列化，則可能透過反序列化的過程中，呼叫可能可用的類別(Class)或方法(method)來執行程式碼或操作系統命令。
此外，反序列化可能會繞過邏輯驗證。由於反序列化通常使用自己的方法，從序列化資料中構造新物件，因此它可以繞過建構函數或setter中強制執行的檢查，這將允許攻擊者反序列化未經驗證不正確或完全惡意的物件。這可能會導致例外狀況，進而影響程式邏輯。



發生原因:
序列化和反序列化物件，對於遠端Web Service流程是不可或缺的，其中物件通過Web
Service(例如透過網路)在程式碼實作之間傳遞。在反序列化期間，從Web Service上提供的序列化物件建構新物件；但是，如果被反序列化的物件不受信任，則可能包含意外或有潛在危險的物件。


如何避免:
盡可能不要在遠端Web Service之間傳遞序列化物件。相反地，請考慮在Web Service之間傳遞原始值( value primitives)，並使用這些值填入新建構的物件。如果需要，使用白名單方法傳遞物件。
務必確保傳遞的物件是已知的、可信的和可預期的。除非物件已經過驗證且屬於受信任的已知類型，並且不包含不受信任的物件，否則不要從任何來源動態建構物件。

============================================================

攻擊者在欲被反解譯(de-serialize)回物件的byte stream內容
中輸入自訂的字串以達到攻擊目標
Remote code execution
Denial of Service(DoS)
Access-control-related attacks


聽處理過有經驗的同事分享後得知
原因在於直接對string 也就是Stream 直接ReadToEnd()的結果直接做反序列化
會被掃瞄到這類風險被要求改正





Before:

 1
 2
 3
 4
 5
 6
 7
 8
 9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
        string testJsonFilePath = HttpContext.Current.Server.MapPath("~/json/rcmdAssg.json");
        using (StreamReader streamReader = new StreamReader(testJsonFilePath))
        {
            jsonResponse = streamReader.ReadToEnd();
        }
        IC_RcmdAssg_Model iC_RcmdAssg_Model = Newtonsoft.Json.JsonConvert.DeserializeObject<IC_RcmdAssg_Model>(jsonResponse);

        DataTable dtRcmd = new DataTable();        
        dtRcmd.Columns.Add("isChked", Type.GetType("System.Boolean"));
        dtRcmd.Columns.Add("oriAssgName", Type.GetType("System.String"));
        dtRcmd.Columns.Add("oriAssgCountry", Type.GetType("System.String"));
        dtRcmd.Columns.Add("rcmdName", Type.GetType("System.String"));
        dtRcmd.Columns.Add("rcmdCountry", Type.GetType("System.String"));
        DataRow newRow;

        foreach (Assignee_List assgRcmd in iC_RcmdAssg_Model.assignee_list)
        {
            newRow = dtRcmd.NewRow();
            newRow["isChked"] = true;
            newRow["oriAssgName"] = assgRcmd.oriAssgName;
            newRow["oriAssgCountry"] = assgRcmd.oriAssgName;
            newRow["rcmdName"] = assgRcmd.oriAssgName;
            newRow["rcmdCountry"] = assgRcmd.oriAssgName;
            dtRcmd.Rows.Add(newRow);
        }

        gvRcmdAssgData.DataSource = dtRcmd;
        gvRcmdAssgData.DataBind();


After:

 1
 2
 3
 4
 5
 6
 7
 8
 9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
	    string testJsonFilePath = HttpContext.Current.Server.MapPath("~/json/rcmdAssg.json");
            using (StreamReader streamReader = new StreamReader(testJsonFilePath))
            {
                //jsonResponse = streamReader.ReadToEnd();
                JsonSerializer jsonSerializer = new JsonSerializer();
                IC_RcmdAssg_Model iC_RcmdAssg_Model = (IC_RcmdAssg_Model)jsonSerializer.Deserialize(streamReader, typeof(IC_RcmdAssg_Model));
                DataTable dtRcmd = new DataTable();
                dtRcmd.Columns.Add("isChked", Type.GetType("System.Boolean"));
                dtRcmd.Columns.Add("oriAssgName", Type.GetType("System.String"));
                dtRcmd.Columns.Add("oriAssgCountry", Type.GetType("System.String"));
                dtRcmd.Columns.Add("rcmdName", Type.GetType("System.String"));
                dtRcmd.Columns.Add("rcmdCountry", Type.GetType("System.String"));
                DataRow newRow;

                foreach (Assignee_List assgRcmd in iC_RcmdAssg_Model.assignee_list)
                {
                    newRow = dtRcmd.NewRow();
                    newRow["isChked"] = false;
                    newRow["oriAssgName"] = assgRcmd.oriAssgName;
                    newRow["oriAssgCountry"] = assgRcmd.oriAssgCountry;
                    newRow["rcmdName"] = assgRcmd.rcmdName;
                    newRow["rcmdCountry"] = assgRcmd.rcmdCountry;
                    dtRcmd.Rows.Add(newRow);
                }

                gvRcmdAssgData.DataSource = dtRcmd;
                gvRcmdAssgData.DataBind();
            }


通常這種反序列化情境
1.盡量避免用generic types 像是object 或是 dynamic這種，應提早就限制住反序列的物件型態。
2.使用Json.NET時候 請勿使用「無」以外的 TypeNameHandling


Ref:
Attacking .NET deserialization - Alvaro Muñoz
https://www.youtube.com/watch?v=eDfGpu3iE4Q&ab_channel=Scrtinsomnihack
STOP Insecure Deserialization with C#
https://medium.com/@tiagodaraujo/stop-insecure-deserialization-with-c-6a488c95cf2f
TypeNameHandling Enumeration
https://www.newtonsoft.com/json/help/html/T_Newtonsoft_Json_TypeNameHandling.htm
CA2326:請勿使用「無」以外的 TypeNameHandling 值
https://docs.microsoft.com/zh-tw/dotnet/fundamentals/code-analysis/quality-rules/ca2326

CA2329: Do not deserialize with JsonSerializer using an insecure configuration
https://docs.microsoft.com/en-us/dotnet/fundamentals/code-analysis/quality-rules/ca2329

留言

張貼留言

這個網誌中的熱門文章

何謂淨重(Net Weight)、皮重(Tare Weight)與毛重(Gross Weight)

-
圖片
當談到貨物的重量時，往往會分成所謂的淨重(Net Weight)與毛重(Gross Weight)。 會有以下公式關係存在 淨重=毛重-皮重(包裝物的重量) 毛重=淨重+皮重 淨重( Net Weight 縮寫為N.W.) 指物品本身的重量，不包括容器或包裝材料的重量。 凈重是國際貿易中最常見的計重辦法。 例如，如果運輸原材料，原材料的重量就是所運輸貨物的淨重。包裝原材料的容器或包裝不計入淨重。同樣地，當我們考慮一罐豆子時，罐子內所含物品的重量就算作淨重。因此，淨重將包括豆子、罐頭里的任何液體，以及保存豆子所含的物品。 大多數 以重量為計價的貨物買賣 ，採用此計價方法。 若合同中並未明確規範以何種方式計量和計價時，按慣例以凈重計價。 毛重( Gross Weight 縮寫G.W.) 貨物連同它的包裝的重量，是你托運的所有物品的總重量。 皮重(Tare Weight) 指商品外包裝材料的重量(即運輸包裝的重量)，不包括內包裝材料和襯墊物的重量。 比方一個空的貨櫃、容器、或任何包裝材料的重量。 Net Net Weight純淨重 (指從Net Weight中扣除內包裝後的重量) 進出口商品為了因應運輸及消費需求, 一般都有內包裝與外包裝。 一般在買賣合同與信用狀中,通常只要求於裝箱單上註明Gross Weight和Net Weight即可, 但有些L/C 要求 Net Net Weight,大多是海關徵收從量稅時的依據 運輸重量對於航運業或者航空運輸等各自都通用這些術語 為何特別在意重量? 主要就在於會影響像是傳輸安全乘載限制。 就運輸層面(不同方式的運輸) 航空運輸：對機長而言，毛重包括皮重、產品淨重以及飛機、燃料、乘客和機組人員的重量。 公路/鐵路運輸:對於火車車長而言，毛重包括產品淨重、皮重和運輸車輛的重量。 水路航運運輸：對船長而言，毛重就是淨重和皮重之和。 就公司ERP成本角度 成本一般都會採用 "淨重" 主要是因為若用毛重，可能會有輔材不可控因素存在 比方一個 5cm 固定塑膠件，今天用一個 10cm 紙箱測重，明天用一個 100cm 紙箱測重，那可能就容易導致ERP資料比較會失真。 就物流層面來探討 通常還會有一名詞即所謂「實際重量」(Actual Weight) 根據稱重(過磅)所得到的重量，實際重量分為實際毛重(Gross Weig...
閱讀完整內容

Architecture(架構) 和 Framework(框架) 有何不同?_軟體設計前的事前規劃的藍圖概念

-
圖片
一個應用程式(軟體) 不管是  互動拍照 也好、 投影也好、車牌偵測... 舞台演唱會後方電視牆的互動應用程序、 粒子特效、3D或2D遊戲  等等 開發者通常都需要在一開始做規劃 ㄟ  假設我今天要寫 跟影像有關 可能需要偵測、可能需要提取前景、可能需要偵測圓形 之類的 我有  emgucv 、AForge.Net、 opencv 等等可以用 但是 假設我今天要做的應用程序是需要有視窗介面的~~ emgucv 跟 C# windows form 媒合性 就很高 而且直接在 visual studio 做開發 opencv 我可能還需要依賴 Qt 之類等等  framework 假設我今天是要寫一個 演唱會電視牆互動粒子特效 可能就會挑   openframeworks  、 processing 、kinect 來寫互動 Achitecture  --->全稱 :  Software Architecture   軟體架構 Achitecture   由好多 framework 組成 軟體架構（Software Architecture）是一種軟體在開發前的設計藍圖， 用來告訴軟體的結構，功能，介面，用法，與其他系統的構連以及資料交換等等規範， 但它並沒有叫你要用什麼方式實作， 因此軟體架構通常會產生文件，圖樣，原型以及規格等， 就是沒有可用的程式碼，因為那不是軟體架構應該有的東西， 就像蓋房子時是給你藍圖，而不是一幢蓋好的房子。 軟體框架（Framework):是一個已經成形的方法，而且有程式碼實體 （例如鋼構工法也是要有鋼材才能做），並且會告訴你要如何使用它 （即 Framework Documentation，MSDN Library 即為一最佳例子）
閱讀完整內容

經得起原始碼資安弱點掃描的程式設計習慣培養(五)_Missing HSTS Header

-
圖片
  接續前幾篇系列 經得起原始碼資安弱點掃描的程式設計習慣培養(一)_OWASP Top 10(十大網站安全風險)_學習寫更安全程式碼的網站推薦 經得起原始碼資安弱點掃描的程式設計習慣培養(二)_8.Insecure Deserialization ：不安全的反序列化漏洞ASP.NET處理方式 經得起原始碼資安弱點掃描的程式設計習慣培養(三)_7.Cross Site Scripting(XSS)_Stored XSS_Reflected XSS All Clients 經得起原始碼資安弱點掃描的程式設計習慣培養(四)_1.Injection注入攻擊_SQL Injection)_order by 語句正確參數化套入 最近又遇到被若安掃描到高風險要修正的程式碼啦.... 一看這還真是搞不懂該怎麼修呢 HSTS(HTTP Strict Transport Security)  主要是一份國際標準規格 (RFC 6797) 的網際網路瀏覽安全機制， 用於宣告瀏覽器與伺服器之間通訊方式必須強制採用 TLS/SSL 加密通道。 只要從伺服器端送出一個 Strict-Transport-Security 標頭 (Header) 給瀏覽器，即可告知瀏覽器於未來的某段時間內一律使用 SSL 來和該網站連接 (設定可涵蓋所有子域名網站)， 一旦發生憑證失效情況，使用者將無法再瀏覽該網站，如此一來便可大幅減少中間人攻擊的問題發生。 再更白話一點 HSTS Header 就是負責將 http 強制轉為 https CheckMarx掃碼報告給的說明 ============================================================ 風險: 如果web config沒有設置HSTS表頭、"max-age"(有效期)不足一年， 或沒有設定"includeSubDomains"(包含所有子網域)，可能會使用戶容易遭受中間人攻擊。 發生原因: 許多使用者只要在瀏覽器的網址列輸入網域名稱(Domain Name)即可瀏覽網站， 並沒有輸入協定(Protocol)的名稱。 瀏覽器會自動假設使用者使用的是HTTP協定而不是有加密機制的HTTPS協定。 在第一次發出前往特定網站的請求時，攻擊者可以執行中間人攻擊並將用戶轉址到...
閱讀完整內容