Last modified: 2014-02-08 16:53:31 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T20647, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 18647 - When rendering math, TeX output is sent to stderr which ends up in Apache's error log
When rendering math, TeX output is sent to stderr which ends up in Apache's e...
Status: RESOLVED INVALID
Product: MediaWiki extensions
Classification: Unclassified
Math (Other open bugs)
unspecified
All All
: Low minor (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-05-01 21:44 UTC by Ken Healy
Modified: 2014-02-08 16:53 UTC (History)
4 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Ken Healy 2009-05-01 21:44:39 UTC
When TeX is executed by MathRenderer->render() to render math, the informational messages that TeX prints to stderr end up in Apache's error log because stderr is not redirected.

In my opinion, this is not the correct behavior, as
 1) These are informational messages, not errors
 2) These messages are written directly to Apache's error log without any time, date or severity information.

The relevant code is Math.php line 72:

$contents = `$cmd`;

Suggested fix: Alter this line to

$contents = `$cmd 2>/dev/null`;

Could this fix have any unforeseen consequences?
Comment 1 Roan Kattouw 2009-05-01 21:57:46 UTC
(In reply to comment #0)
> Suggested fix: Alter this line to
> 
> $contents = `$cmd 2>/dev/null`;
> 
> Could this fix have any unforeseen consequences?
> 

My gut says 2> redirection doesn't work on Windows shells, although I might be wrong; /dev/null needs to be different on Windows at least.
Comment 2 physikerwelt 2014-02-08 16:53:31 UTC
I don't know if that's still relevant. But wfShellExec is used now. So it would be a core issue. Feel free to reopen it as core bug.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links