robojerk/codeql-action

Merge pull request #817 from edoardopirovano/respect-ld-preload

Browse source 
Respect value of `LD_PRELOAD` given by the CLI
This commit is contained in:
Edoardo Pirovano 2021-11-16 00:22:50 +00:00 • committed by GitHub
commit 2ecc17d74f
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
3 changed files with 28 additions and 28 deletions
Download patch file Download diff file Expand all files Collapse all files

2
lib/tracer-config.js generated
View file

@ -182,7 +182,6 @@ async function getCombinedTracerConfig(config, codeql) {
tracedLanguageConfigs[language] = await getTracerConfigForLanguage(codeql, config, language);
}
mainTracerConfig = concatTracerConfigs(tracedLanguageConfigs, config);
}
// Add a couple more variables
mainTracerConfig.env["ODASA_TRACER_CONFIGURATION"] = mainTracerConfig.spec;
const codeQLDir = path.dirname(codeql.getPath());
@ -192,6 +191,7 @@ async function getCombinedTracerConfig(config, codeql) {
else if (process.platform !== "win32") {
mainTracerConfig.env["LD_PRELOAD"] = path.join(codeQLDir, "tools", "linux64", "${LIB}trace.so");
}
}
// On macos it's necessary to prefix the build command with the runner executable
// on order to trace when System Integrity Protection is enabled.
// The executable also exists and works for other platforms so we output this env

2
lib/tracer-config.js.map
View file

File diff suppressed because one or more lines are too long

2
src/tracer-config.ts
View file

@ -212,7 +212,6 @@ export async function getCombinedTracerConfig(
);
}
mainTracerConfig = concatTracerConfigs(tracedLanguageConfigs, config);
}
// Add a couple more variables
mainTracerConfig.env["ODASA_TRACER_CONFIGURATION"] = mainTracerConfig.spec;
@ -232,6 +231,7 @@ export async function getCombinedTracerConfig(
"${LIB}trace.so"
);
}
}
// On macos it's necessary to prefix the build command with the runner executable
// on order to trace when System Integrity Protection is enabled.