Revert "audit: TOB-ACCB-009"

This reverts commit fcb700d.
Because Windows system may have no right file mode.

cmd/swaptools/sendbtc.go

@@ -4,14 +4,14 @@ package main
 import (
 	"encoding/hex"
 	"fmt"
+	"io/ioutil"
 	"strings"
 
 	"github.com/anyswap/CrossChain-Bridge/cmd/utils"
 	"github.com/anyswap/CrossChain-Bridge/common"
 	"github.com/anyswap/CrossChain-Bridge/log"
 	"github.com/anyswap/CrossChain-Bridge/tokens"
 	"github.com/anyswap/CrossChain-Bridge/tokens/btc"
-	"github.com/anyswap/CrossChain-Bridge/tools"
 	"github.com/btcsuite/btcd/btcec"
 	"github.com/btcsuite/btcutil"
 	"github.com/btcsuite/btcutil/base58"
@@ -143,13 +143,13 @@ func (bts *btcTxSender) initBridge() {
 func (bts *btcTxSender) loadWIFForAddress() string {
 	var wifStr string
 	if bts.wifFile != "" {
-		wifdata, err := tools.SafeReadFile(bts.wifFile)
+		wifdata, err := ioutil.ReadFile(bts.wifFile)
 		if err != nil {
 			log.Fatal("Read WIF file failed", "err", err)
 		}
 		wifStr = strings.TrimSpace(string(wifdata))
 	} else {
-		pridata, err := tools.SafeReadFile(bts.priFile)
+		pridata, err := ioutil.ReadFile(bts.priFile)
 		if err != nil {
 			log.Fatal("Read private key file failed", "err", err)
 		}

cmd/swaptools/sendltc.go

@@ -4,14 +4,14 @@ package main
 import (
 	"encoding/hex"
 	"fmt"
+	"io/ioutil"
 	"strings"
 
 	"github.com/anyswap/CrossChain-Bridge/cmd/utils"
 	"github.com/anyswap/CrossChain-Bridge/common"
 	"github.com/anyswap/CrossChain-Bridge/log"
 	"github.com/anyswap/CrossChain-Bridge/tokens"
 	"github.com/anyswap/CrossChain-Bridge/tokens/ltc"
-	"github.com/anyswap/CrossChain-Bridge/tools"
 	"github.com/ltcsuite/ltcd/btcec"
 	"github.com/ltcsuite/ltcutil"
 	"github.com/ltcsuite/ltcutil/base58"
@@ -143,13 +143,13 @@ func (bts *ltcTxSender) initBridge() {
 func (bts *ltcTxSender) loadWIFForAddress() string {
 	var wifStr string
 	if bts.wifFile != "" {
-		wifdata, err := tools.SafeReadFile(bts.wifFile)
+		wifdata, err := ioutil.ReadFile(bts.wifFile)
 		if err != nil {
 			log.Fatal("Read WIF file failed", "err", err)
 		}
 		wifStr = strings.TrimSpace(string(wifdata))
 	} else {
-		pridata, err := tools.SafeReadFile(bts.priFile)
+		pridata, err := ioutil.ReadFile(bts.priFile)
 		if err != nil {
 			log.Fatal("Read private key file failed", "err", err)
 		}

tools/crypto/crypto.go

@@ -167,13 +167,6 @@ func HexToECDSA(hexkey string) (*ecdsa.PrivateKey, error) {
 // LoadECDSA loads a secp256k1 private key from the given file.
 func LoadECDSA(file string) (*ecdsa.PrivateKey, error) {
 	buf := make([]byte, 64)
-	fi, err := os.Stat(file)
-	if err != nil {
-		return nil, err
-	}
-	if fi.Mode() != 0400 {
-		return nil, errors.New("unsafe file permissions, want 0400")
-	}
 	fd, err := os.Open(file)
 	if err != nil {
 		return nil, err

tools/crypto/crypto_test.go

@@ -150,7 +150,7 @@ func TestLoadECDSAFile(t *testing.T) {
 		}
 	}
 
-	_ = ioutil.WriteFile(fileName0, []byte(testPrivHex), 0400)
+	_ = ioutil.WriteFile(fileName0, []byte(testPrivHex), 0600)
 	defer os.Remove(fileName0)
 
 	key0, err := LoadECDSA(fileName0)

tools/loadkeystore.go

@@ -1,36 +1,20 @@
 package tools
 
 import (
-	"errors"
 	"fmt"
 	"io/ioutil"
-	"os"
 	"strings"
 
 	"github.com/anyswap/CrossChain-Bridge/tools/keystore"
 )
 
-var errUnsafeFilePermissions = errors.New("unsafe file permissions, want 0400")
-
-// SafeReadFile check permissions is '0400' and read file
-func SafeReadFile(file string) ([]byte, error) {
-	fi, err := os.Stat(file)
-	if err != nil {
-		return nil, err
-	}
-	if fi.Mode() != 0400 {
-		return nil, errUnsafeFilePermissions
-	}
-	return ioutil.ReadFile(file)
-}
-
 // LoadKeyStore load keystore from keyfile and passfile
 func LoadKeyStore(keyfile, passfile string) (*keystore.Key, error) {
-	keyjson, err := SafeReadFile(keyfile)
+	keyjson, err := ioutil.ReadFile(keyfile)
 	if err != nil {
 		return nil, fmt.Errorf("read keystore fail %w", err)
 	}
-	passdata, err := SafeReadFile(passfile)
+	passdata, err := ioutil.ReadFile(passfile)
 	if err != nil {
 		return nil, fmt.Errorf("read password fail %w", err)
 	}