summaryrefslogtreecommitdiff
path: root/os2
diff options
context:
space:
mode:
authorSteffen Mueller <smueller@cpan.org>2012-10-09 11:19:37 +0200
committerSteffen Mueller <smueller@cpan.org>2012-11-05 08:01:43 +0100
commit284167a54e2da949b77d1e736a8b0a0d21210803 (patch)
tree6abdbd2e28d67b392a2137f37554b758ecac0900 /os2
parente88567f2acf38fe5ed90a88569b808e82cd3eca1 (diff)
downloadperl-284167a54e2da949b77d1e736a8b0a0d21210803.tar.gz
Add C define to remove taint support from perl
By defining NO_TAINT_SUPPORT, all the various checks that perl does for tainting become no-ops. It's not an entirely complete change: it doesn't attempt to remove the taint-related interpreter variables, but instead virtually eliminates access to it. Why, you ask? Because it appears to speed up perl's run-time significantly by avoiding various "are we running under taint" checks and the like. This change is not in a state to go into blead yet. The actual way I implemented it might raise some (valid) objections. Basically, I replaced all uses of the global taint variables (but not PL_taint_warn!) with an extra layer of get/set macros (TAINT_get/TAINTING_get). Furthermore, the change is not complete: - PL_taint_warn would likely deserve the same treatment. - Obviously, tests fail. We have tests for -t/-T - Right now, I added a Perl warn() on startup when -t/-T are detected but the perl was not compiled support it. It might be argued that it should be silently ignored! Needs some thinking. - Code quality concerns - needs review. - Configure support required. - Needs thinking: How does this tie in with CPAN XS modules that use PL_taint and friends? It's easy to backport the new macros via PPPort, but that doesn't magically change all code out there. Might be harmless, though, because whenever you're running under NO_TAINT_SUPPORT, any check of PL_taint/etc is going to come up false. Thus, the only CPAN code that SHOULD be adversely affected is code that changes taint state.
Diffstat (limited to 'os2')
-rw-r--r--os2/os2.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/os2/os2.c b/os2/os2.c
index 7dffd42d36..87f88e8c02 100644
--- a/os2/os2.c
+++ b/os2/os2.c
@@ -1568,7 +1568,7 @@ my_syspopen4(pTHX_ char *cmd, char *mode, I32 cnt, SV** args)
/* `this' is what we use in the parent, `that' in the child. */
this = (*mode == 'w');
that = !this;
- if (PL_tainting) {
+ if (TAINTING_get) {
taint_env();
taint_proper("Insecure %s%s", "EXEC");
}