Solidity中bytes转uint256调用报param.substring错修复

问题根因 #

报错和逻辑异常来自两个层面的问题：

• 调用层传参格式错误：web3.utils.hexToBytes('0xa')返回的是JavaScript原生Uint8Array类型字节数组，Truffle合约实例处理bytes类型入参时，要求传入带0x前缀的十六进制字符串，ABI编码逻辑内部会调用字符串的substring方法做格式校验，传入数组对象时不存在该方法，因此在本地编码阶段就抛出错误，请求根本没有发送到链上。
• 合约层转换逻辑缺陷：现有内联汇编代码直接读取bytes数据起始位置的32字节内容，仅当输入bytes长度恰好为32字节时结果正确。如果输入是短字节（例如测试用的0xa长度仅1字节），mload会把相邻内存位置的脏数据一并读取，即使传参正确也无法返回预期的数值10。

修复方案 #

1. 调用命令修正 #

不需要手动把十六进制转成字节数组，直接传入带0x前缀的十六进制字符串即可，Web3会自动完成符合ABI规范的编码：

(await contract.getValue('0xa')).toString()

2. 合约转换逻辑修正 #

需要增加长度校验，同时通过位运算清除高位脏数据，兼容任意长度不超过32字节的bytes输入，修正后的合约代码如下：

// SPDX-License-Identifier: MIT
pragma solidity 0.8.13;

contract Sample {
    function getValue(bytes memory a) external pure returns(uint256 value) {
        // uint256最多存储32字节数据，超长输入直接回滚
        require(a.length <= 32, "bytes length exceed 32");
        assembly {
            // 读取bytes数据段起始位置的32字节内容
            value := mload(add(a, 0x20))
            // 计算需要右移的位数：(32 - 实际字节长度) * 8，右移清除高位脏数据
            value := shr(mul(sub(32, mload(a)), 8), value)
        }
    }
}

逻辑说明：

• mload(a)读取的是bytes类型的长度值，例如输入0xa时长度为1
• shr是EVM的逻辑右移指令，1字节有效数据需要右移248位，把后续31字节的内存脏数据全部移出，最终得到正确值10。

内容的提问来源于stack exchange，提问作者sel