json.readArray() causes panic when reading empty arrays

[ChiTimesChi]

Component

Forge

Have you ensured that all of these are up to date?

• Foundry
• Foundryup

What version of Foundry are you on?

forge 0.2.0 (624797e 2023-02-20T00:07:44.723592254Z)

What command(s) is the bug in?

forge script

Operating System

Linux

Describe the bug

Empty arrays in JSON files cause a panic, when a corresponding json.readTypeArray() is called. Attaching a minimal example:

{
  "tokens": ["0xC02aaA39b223FE8D0A0e5C4F27eAD9083C756Cc2"],
  "empty": []
}

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

import {console, stdJson, Script} from "forge-std/Script.sol";

contract TestScript is Script {
    using stdJson for string;

    function run() external {
        string memory json = vm.readFile("test.json");
        // This succeeds
        address[] memory tokens = json.readAddressArray(".tokens");
        console.log("Tokens Length: %s", tokens.length);
        // Every one of these causes panic
        json.readAddressArray(".empty");
        json.readBoolArray(".empty");
        json.readBytesArray(".empty");
        json.readBytes32Array(".empty");
        json.readStringArray(".empty");
        json.readIntArray(".empty");
        json.readUintArray(".empty");
    }
}

The application panicked (crashed).
Message:  index out of bounds: the len is 0 but the index is 0
Location: evm/src/executor/inspector/cheatcodes/util.rs:133

This is a bug. Consider reporting it at https://github.com/foundry-rs/foundry

  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ BACKTRACE ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
   1: __libc_start_call_main<unknown>
      at ./csu/../sysdeps/nptl/libc_start_call_main.h:58
   2: __libc_start_main_impl<unknown>
      at ./csu/../csu/libc-start.c:392

Run with COLORBT_SHOW_HIDDEN=1 environment variable to disable frame filtering.
Aborted (core dumped)

Also worth noting that my local .env file contains the line COLORBT_SHOW_HIDDEN=1

[mattsse]

cc @odyslam

[odyslam]

@mattsse can you assign pls

[odyslam]

Will try to tackle this in the week @ChiTimesChi