diff options
author | Linus Heckemann <git@sphalerite.org> | 2023-06-01T19·22+0200 |
---|---|---|
committer | Linus Heckemann <git@sphalerite.org> | 2023-06-12T08·38+0000 |
commit | 5733274876e6f6c0be8b43b4a46bdfeea73fa761 (patch) | |
tree | 0bf8e31ba76cb27a2f6f0ddaa7c7f01456b024fa /tvix/eval/src/tests/tvix_tests/eval-okay-builtins-substring-negative-length.nix | |
parent | 2b4ad47c16dc0fb114be0fd03526ffb871aa1875 (diff) |
fix(tvix/eval): allow negative substring lengths r/6267
Nix uses string::substr without checking the sign of the length[1]. The NixOS testing infrastructure relies on this[2], and on the implicit conversion of that to the maximum possible value for a size_t. [1]: https://github.com/NixOS/nix/blob/ecae62020b64914d9859a71ce197d03688c6133c/src/libexpr/primops.cc#L3597 [2]: https://github.com/NixOS/nixpkgs/blob/c7c298471676ac1c7789ab3c424fbcebecaa6791/nixos/lib/testing/driver.nix#L29 Change-Id: I6d0caf6830b6bda3fdf44c40c81de6a1befeca7b Reviewed-on: https://cl.tvl.fyi/c/depot/+/8746 Reviewed-by: flokli <flokli@flokli.de> Tested-by: BuildkiteCI
Diffstat (limited to '')
0 files changed, 0 insertions, 0 deletions