From d958502f2c5c2cfa52b11dcf6478a875c14ccb1f Mon Sep 17 00:00:00 2001 From: AMZN-Phil Date: Mon, 22 Nov 2021 09:26:34 -0800 Subject: [PATCH] Check return value of qputenv Signed-off-by: AMZN-Phil --- Code/Tools/ProjectManager/Platform/Mac/ProjectUtils_mac.cpp | 5 ++++- .../ProjectManager/Platform/Windows/ProjectUtils_windows.cpp | 5 ++++- 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/Code/Tools/ProjectManager/Platform/Mac/ProjectUtils_mac.cpp b/Code/Tools/ProjectManager/Platform/Mac/ProjectUtils_mac.cpp index 0c232d44d1..07b08924f1 100644 --- a/Code/Tools/ProjectManager/Platform/Mac/ProjectUtils_mac.cpp +++ b/Code/Tools/ProjectManager/Platform/Mac/ProjectUtils_mac.cpp @@ -29,7 +29,10 @@ namespace O3DE::ProjectManager if (!pathEnvList.contains("/usr/local/bin")) { pathEnv += ":/usr/local/bin"; - qputenv("PATH", pathEnv.toStdString().c_str()); + if (!qputenv("PATH", pathEnv.toStdString().c_str())) + { + return AZ::Failure(QObject::tr("Failed to set PATH environment variable")); + } } return AZ::Success(); diff --git a/Code/Tools/ProjectManager/Platform/Windows/ProjectUtils_windows.cpp b/Code/Tools/ProjectManager/Platform/Windows/ProjectUtils_windows.cpp index 92863b955c..b8cfe65b1e 100644 --- a/Code/Tools/ProjectManager/Platform/Windows/ProjectUtils_windows.cpp +++ b/Code/Tools/ProjectManager/Platform/Windows/ProjectUtils_windows.cpp @@ -41,7 +41,10 @@ namespace O3DE::ProjectManager if (!pathEnvList.contains(cmakePath.path())) { pathEnv += ";" + cmakePath.path(); - qputenv("Path", pathEnv.toStdString().c_str()); + if (!qputenv("Path", pathEnv.toStdString().c_str())) + { + return AZ::Failure(QObject::tr("Failed to set Path environment variable")); + } } return AZ::Success();