CVE-2023-45136 (Other, published 2023-10-25T20:15Z)
Description: value: XWiki Platform is a generic wiki platform offering runtime services for applications built on top of it. When document names are validated according to a name strategy (disabled by default), XWiki starting in version 12.0-rc-1 and prior to versions 12.10.12 and 15.5-rc-1 is vulnerable to a reflected cross-site scripting attack in the page creation form. This allows an attacker to execute arbitrary actions with the rights of the user opening the malicious link. Depending on the rights of the user, this may allow remote code execution and full read and write access to the whole XWiki installation. This has been patched in XWiki 14.10.12 and 15.5-rc-1 by adding appropriate escaping. The vulnerable template file `createinline.vm` is part of XWiki's WAR and can be patched by manually applying the changes from the fix.
Severity: nan (CVSS v3 base score: 9.6)
Weakness: CWE-79 \u2014 Improper Neutralization of Input During Web Page Generation ('Cross-site Scripting')
The product does not neutralize or incorrectly neutralizes user-controllable input before it is placed in output that is used as a web page that is served to other users.
Repository: https://github.com/xwiki/xwiki-platform
Files changed:
xwiki-platform-core/xwiki-platform-web/xwiki-platform-web-templates/src/main/resources/templates/createinline.vm
xwiki-platform-core/xwiki-platform-web/xwiki-platform-web-templates/src/test/java/org/xwiki/web/CreateInlinePageTest.java
Commit message: XWIKI-20962: Escape template provider errors during document creation

* Add escaping and a test


Vulnerable code:
      $services.localization.render('core.create.template.allowedspace.inline', [$templateProvider, $allowedSpaces.get(0)])
      $services.localization.render('core.create.template.allowedspaces.inline', [$templateProvider, $allowedSpaces.toString()])

Fixed code:
      $escapetool.xml($services.localization.render('core.create.template.allowedspace.inline', [$templateProvider,
        $allowedSpaces.get(0)]))
      $escapetool.xml($services.localization.render('core.create.template.allowedspaces.inline', [$templateProvider,
        $allowedSpaces.toString()]))
import java.util.stream.Stream;
import org.junit.jupiter.params.ParameterizedTest;
import org.junit.jupiter.params.provider.MethodSource;

    /**
     * Test that when there is an exception about the template provider not allowing the chosen space, the allowed
     * spaces are correctly escaped.
     */
    @ParameterizedTest
    @MethodSource("allowedSpacesProvider")
    void templateProviderRestrictionErrorEscaping(List<String> allowedSpaces) throws Exception
    {
        String provider = "\"provider</div>";
        this.request.put("templateprovider", provider);
        String template = "template</div>";

        // Set "createException" to an XWikiException to simulate a template provider restriction error.
        Object[] args = { template, DOCUMENT_REFERENCE, DOCUMENT_REFERENCE };
        XWikiException exception = new XWikiException(XWikiException.MODULE_XWIKI_STORE,
            XWikiException.ERROR_XWIKI_APP_TEMPLATE_NOT_AVAILABLE,
            "Template {0} cannot be used in space {1} when creating page {2}", null, args);
        this.velocityManager.getVelocityContext().put(CREATE_EXCEPTION_VELOCITY_KEY, exception);
        // Set the allowed spaces to a list containing some HTML.
        this.velocityManager.getVelocityContext().put("createAllowedSpaces", allowedSpaces);

        // Render the template.
        Document document = Jsoup.parse(this.templateManager.render(CREATE_INLINE_VM));
        Element errormessage = document.getElementsByClass(ERROR_MESSAGE_CLASS).first();
        assertNotNull(errormessage);

        String expectedMessage;
        if (allowedSpaces.size() == 1) {
            expectedMessage = String.format("core.create.template.allowedspace.inline [%s, %s]",
                provider, allowedSpaces.get(0));
        } else {
            expectedMessage = String.format("core.create.template.allowedspaces.inline [%s, %s]",
                provider, allowedSpaces);
        }
        assertEquals(expectedMessage, errormessage.text());
    }

    static Stream<List<String>> allowedSpacesProvider()
    {
        return Stream.of(
            List.of("allowedSpace</div>"),
            List.of("allowedSpace1</div>", "allowedSpace2</div>")
        );
    }