Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Appropriate short-circuit for shell commands. Fixes #4361.
The original logic was a bad port of the following C code: ```c static const char posix_sh_cmds[][9] = { ... } if (first.len > 0 && first.len <= sizeof(posix_sh_cmds[0]) && ... ``` In C, the array allocation sets all elements to be sizeof = 9, even though most terminate via null at a shorter C string length. The check here is to filter out any commands longer than those known to be shell commands; my fix is to just have that max length be a constant and reference it where sizeof is used in MRI.