summaryrefslogtreecommitdiff
path: root/doc/plugins/contrib/xslt/discussion.mdwn
blob: 9cda02f880c06ec7bd08852a2c777b1f0ec6eedd (plain)

security

I'm curious what the security implications of having this plugin on a publically writable wiki are.

First, it looks like the way it looks up the stylesheet file will happily use a regular .mdwn wiki page as the stylsheet. Which means any user can create a stylesheet and have it be used, without needing permission to upload arbitrary files. That probably needs to be fixed; one way would be to mandate that the srcfile has a .xsl extension.

Secondly, if an attacker is able to upload a stylesheet file somehow, could this be used to attack the server where it is built? I know that xslt is really a full programming language, so I assume at least DOS attacks are possible. Can it also read other arbitrary files, run other programs, etc? --[[Joey]]

For the first point, agreed. It should probably check that the data file has a .xml extension also. Have now fixed.

For the second point, I think the main concern would be resource usage. XSLT is a pretty limited language; it can read other XML files, but it can't run other programs so far as I know.

-- [[KathrynAndersen]]