diff options
author | Andreas Rammhold <andreas@rammhold.de> | 2020-06-16T10·34+0200 |
---|---|---|
committer | tazjin <mail@tazj.in> | 2020-07-10T19·41+0000 |
commit | 1937bc86244c0d420f7b40eeb38f14827e92d162 (patch) | |
tree | 4554181e3b27d25e1ef6a83ed2692ffb720b5aeb /third_party/nix/src/libstore/globals.cc | |
parent | d06237707b604da79957282b240baff40bae3ad9 (diff) |
fix(3p/nix/libstore): use exception instead of assertion when storepath is malformed r/1249
Previously the nix-daemon would crash if a user fed it invalid store paths for drv files. The crash was due to the changed assertion triggering. Whenever that assertion would hit the nix-daemon process along with all it's current childs (running builds from all users) would be interrupted. Before this patch: $ nix-store --realise /nix/store/aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa.drv don't know how to build these paths: /nix/store/aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa.drv error: unexpected end-of-file < nix-daemon terminates > With this patch: $ nix-store --realise /nix/store/aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa.drv don't know how to build these paths: /nix/store/aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa.drv error: path '/nix/store/aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa.drv' is not a valid store path < nix-daemon does *NOT* terminate > Change-Id: I01c5048c8a43a8b9154bdeb781d05b7744869ec0 Reviewed-on: https://cl.tvl.fyi/c/depot/+/981 Tested-by: BuildkiteCI Reviewed-by: glittershark <grfn@gws.fyi> Reviewed-by: tazjin <mail@tazj.in>
Diffstat (limited to 'third_party/nix/src/libstore/globals.cc')
0 files changed, 0 insertions, 0 deletions