Skip to content

Commit 6fba657

Browse files
committed
Version 2018-11-29: Security fix for DokuWiki Greebo
If you are using the DokuWiki Greebo release and rely on ACL checks in the include plugin, apply this change as soon as possible. Note that this is only an issue with namespace includes, so if you do not use namespace includes and edits are only allowed for users that have access to your whole wiki, this does not concern you (but updating is still recommended). Note that this is a problem caused by a bug in DokuWiki release Greebo. A future hotfix release of DokuWiki might fix this, too, see dokuwiki/dokuwiki#2609 for further information.
1 parent cf08914 commit 6fba657

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

helper.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -704,7 +704,7 @@ function _get_included_pages($mode, $page, $sect, $parent_id, $flags) {
704704
$ns = utf8_encodeFN(str_replace(':', '/', $page));
705705
// depth is absolute depth, not relative depth, but 0 has a special meaning.
706706
$depth = $flags['depth'] ? $flags['depth'] + substr_count($page, ':') + ($page ? 1 : 0) : 0;
707-
search($pagearrays, $conf['datadir'], 'search_allpages', array('depth' => $depth), $ns);
707+
search($pagearrays, $conf['datadir'], 'search_allpages', array('depth' => $depth, 'skipacl' => false), $ns);
708708
if (is_array($pagearrays)) {
709709
foreach ($pagearrays as $pagearray) {
710710
if (!isHiddenPage($pagearray['id'])) // skip hidden pages

plugin.info.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
base include
22
author Michael Hamann, Gina Häussge, Christopher Smith, Michael Klier, Esther Brunner
33
email michael@content-space.de
4-
date 2018-04-24
4+
date 2018-11-29
55
name include plugin
66
desc Functions to include another page in a wiki page
77
url http://dokuwiki.org/plugin:include

0 commit comments

Comments
 (0)