Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix PermissionDeniedException in NativeBroker.defragXMLresource #5311

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Commits on May 23, 2024

  1. [bugfix] do not set permissions on temporary document

    When XML resources are modified using an effective group because the setGID bit is set on the resource NativeBroker.defragXMLResource might throw a PermissionDeniedException.
    
    With this patch applied, this can no longer happen as the call to DocumentImpl#copyOf is removed in defragXMLresource.
    The temporary document is just a container to realign nodes on one BTree page and is destroyed within the method.
    line-o committed May 23, 2024
    Configuration menu
    Copy the full SHA
    fe48be4 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    e6edd24 View commit details
    Browse the repository at this point in the history

Commits on May 24, 2024

  1. [bugfix] revert to previous constructor call for tempDoc in defragXML…

    …Resource
    
    Testing has shown that the previous call to construct tempDoc was actually the correct one,
    as it is using the default file permissions of the pool.
    That is key for situations in which the subject initiating the defragmentation is not the
    _owner_ of the document and only belongs to the group with write permission.
    line-o committed May 24, 2024
    Configuration menu
    Copy the full SHA
    cf0ec0f View commit details
    Browse the repository at this point in the history