From c0cd50e057240c950021bd75462ae5aa7f54b0a6 Mon Sep 17 00:00:00 2001
From: Henry <101552941+henry-0@users.noreply.github.com>
Date: Mon, 1 Aug 2022 19:47:21 +0800
Subject: [PATCH] common/compiler: json unmarshalling error checks (#25449)

complier/solidity:add json.Unmarshal err check
---
 common/compiler/solidity.go | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/common/compiler/solidity.go b/common/compiler/solidity.go
index ad8a44aa04ae6..9de94017c2edf 100644
--- a/common/compiler/solidity.go
+++ b/common/compiler/solidity.go
@@ -66,13 +66,16 @@ func ParseCombinedJSON(combinedJSON []byte, source string, languageVersion strin
 	contracts := make(map[string]*Contract)
 	for name, info := range output.Contracts {
 		// Parse the individual compilation results.
-		var abi interface{}
+		var abi, userdoc, devdoc interface{}
 		if err := json.Unmarshal([]byte(info.Abi), &abi); err != nil {
 			return nil, fmt.Errorf("solc: error reading abi definition (%v)", err)
 		}
-		var userdoc, devdoc interface{}
-		json.Unmarshal([]byte(info.Userdoc), &userdoc)
-		json.Unmarshal([]byte(info.Devdoc), &devdoc)
+		if err := json.Unmarshal([]byte(info.Userdoc), &userdoc); err != nil {
+			return nil, fmt.Errorf("solc: error reading userdoc definition (%v)", err)
+		}
+		if err := json.Unmarshal([]byte(info.Devdoc), &devdoc); err != nil {
+			return nil, fmt.Errorf("solc: error reading devdoc definition (%v)", err)
+		}
 
 		contracts[name] = &Contract{
 			Code:        "0x" + info.Bin,