Merge topic 'fix_potential_use_of_freed_stack_memory'

1304fb4027 PkgC: Make sure we don't use deleted stack objects

Acked-by: Kitware Robot <kwrobot@kitware.com>
Tested-by: buildbot <buildbot@kitware.com>
Acked-by: Vito Gamberini <vito.gamberini@kitware.com>
Merge-request: !11965
This commit is contained in:
Brad King
2026-04-29 14:30:26 +00:00
committed by Kitware Robot
2 changed files with 7 additions and 7 deletions

View File

@@ -227,7 +227,7 @@ void cmPkgConfigResolver::ReplaceSep(std::string& list)
}
cm::optional<cmPkgConfigResult> cmPkgConfigResolver::ResolveStrict(
std::vector<cmPkgConfigEntry> const& entries, cmPkgConfigEnv env)
std::vector<cmPkgConfigEntry> const& entries, cmPkgConfigEnv& env)
{
cm::optional<cmPkgConfigResult> result;
cmPkgConfigResult config;
@@ -289,11 +289,11 @@ cm::optional<cmPkgConfigResult> cmPkgConfigResolver::ResolveStrict(
}
cm::optional<cmPkgConfigResult> cmPkgConfigResolver::ResolvePermissive(
std::vector<cmPkgConfigEntry> const& entries, cmPkgConfigEnv env)
std::vector<cmPkgConfigEntry> const& entries, cmPkgConfigEnv& env)
{
cm::optional<cmPkgConfigResult> result;
cmPkgConfigResult config = ResolveBestEffort(entries, std::move(env));
cmPkgConfigResult config = ResolveBestEffort(entries, env);
auto const& keys = config.Keywords;
if (keys.find("Name") == keys.end() ||
@@ -307,7 +307,7 @@ cm::optional<cmPkgConfigResult> cmPkgConfigResolver::ResolvePermissive(
}
cmPkgConfigResult cmPkgConfigResolver::ResolveBestEffort(
std::vector<cmPkgConfigEntry> const& entries, cmPkgConfigEnv env)
std::vector<cmPkgConfigEntry> const& entries, cmPkgConfigEnv& env)
{
cmPkgConfigResult result;

View File

@@ -99,13 +99,13 @@ class cmPkgConfigResolver
public:
static cm::optional<cmPkgConfigResult> ResolveStrict(
std::vector<cmPkgConfigEntry> const& entries, cmPkgConfigEnv env);
std::vector<cmPkgConfigEntry> const& entries, cmPkgConfigEnv& env);
static cm::optional<cmPkgConfigResult> ResolvePermissive(
std::vector<cmPkgConfigEntry> const& entries, cmPkgConfigEnv env);
std::vector<cmPkgConfigEntry> const& entries, cmPkgConfigEnv& env);
static cmPkgConfigResult ResolveBestEffort(
std::vector<cmPkgConfigEntry> const& entries, cmPkgConfigEnv env);
std::vector<cmPkgConfigEntry> const& entries, cmPkgConfigEnv& env);
static cmPkgConfigVersionReq ParseVersion(std::string const& version);