Revert r44254 "Cleanup to wfGetNull(): * Add override in case /dev/null isn't accessi...
authorBrion Vibber <brion@users.mediawiki.org>
Wed, 10 Dec 2008 22:58:24 +0000 (22:58 +0000)
committerBrion Vibber <brion@users.mediawiki.org>
Wed, 10 Dec 2008 22:58:24 +0000 (22:58 +0000)
commit4e01dba6cc649578b0326c28a45a14960fcb4564
tree086785b59d16977f07561e808f2d5cc25a7ee518
parent3b571e768cf80b39a2464a089d52c2b49187e2c2
Revert r44254 "Cleanup to wfGetNull(): * Add override in case /dev/null isn't accessible (bug 11009)"
Doesn't seem like a proper fix here... At best, this'll be dumping random crap to some random file unless the user has a local copy of the /dev/null device file, which seems.... wrong. :)
For anything generating command lines, it probably won't make any difference (assuming exec() is enabled at all!) since open_basedir won't be searching through the command line (I think).
Assuming the core use case actually happens (open_basedir is set, but proc_open() is available to run tidy), a more correct fix is probably to go ahead and read in stderr and toss the results, or maybe better pass it through to PHP's stderr FD instead of opening /dev/null ourselves. Tidy has a -q option which should suppress random "hi i'm tidy version XYZ" if it's currently present.
RELEASE-NOTES
includes/DefaultSettings.php
includes/GlobalFunctions.php
includes/ProxyTools.php