diff options
| author | Yorick van Pelt <yorick@yorickvanpelt.nl> | 2023-08-11 11:58:33 +0200 |
|---|---|---|
| committer | Yorick van Pelt <yorick@yorickvanpelt.nl> | 2023-08-11 11:58:33 +0200 |
| commit | b9b51f9579c15c9789814cd4b6969b0f85e13980 (patch) | |
| tree | 8d201ba2061e244b139057abd0ca5273f443095c /src/libstore/build/local-derivation-goal.hh | |
| parent | a1fdc68c655e8d7ece51b6c84796d35203386c87 (diff) | |
Prevent overriding virtual methods that are called in a destructor
Virtual methods are no longer valid once the derived destructor has
run. This means the compiler is free to optimize them to be
non-virtual.
Found using clang-tidy
Diffstat (limited to 'src/libstore/build/local-derivation-goal.hh')
| -rw-r--r-- | src/libstore/build/local-derivation-goal.hh | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/libstore/build/local-derivation-goal.hh b/src/libstore/build/local-derivation-goal.hh index 9acd7593d..8827bfca3 100644 --- a/src/libstore/build/local-derivation-goal.hh +++ b/src/libstore/build/local-derivation-goal.hh @@ -272,8 +272,10 @@ struct LocalDerivationGoal : public DerivationGoal /** * Forcibly kill the child process, if any. + * + * Called by destructor, can't be overridden */ - void killChild() override; + void killChild() override final; /** * Kill any processes running under the build user UID or in the |