You are viewing a plain text version of this content. The canonical link for it is here.
Posted to oak-issues@jackrabbit.apache.org by "angela (JIRA)" <ji...@apache.org> on 2015/01/30 12:35:35 UTC

[jira] [Comment Edited] (OAK-2465) Long overflow in PermissionEntryProviderImpl

    [ https://issues.apache.org/jira/browse/OAK-2465?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14298516#comment-14298516 ] 

angela edited comment on OAK-2465 at 1/30/15 11:35 AM:
-------------------------------------------------------

updated patch incorporating the fix provided by antonio but still building the 'existingNames' set properly in case of the number returned by the store is Long.MAX_VALUE (without that the permission evaluation will most probably be wrong).

[~asanso], i would be glad if you could review my patch as well and check if it still fixes the problem you spotted. in the mean time i will run the whole build.


was (Author: anchela):
updated patch incorporating the fix provided by antonio but still building the 'existingNames' set properly in case of the number returned by the store is Long.MAX_VALUE.

[~asanso], i would be glad if you could review my patch as well and check if it still fixes the problem you spotted. in the mean time i will run the whole build.

> Long overflow in PermissionEntryProviderImpl
> --------------------------------------------
>
>                 Key: OAK-2465
>                 URL: https://issues.apache.org/jira/browse/OAK-2465
>             Project: Jackrabbit Oak
>          Issue Type: Bug
>          Components: core, mongomk
>            Reporter: Antonio Sanso
>            Assignee: angela
>             Fix For: 1.6
>
>         Attachments: OAK-2465-patch.txt, OAK-2465_2.patch
>
>
> PermissionEntryProviderImpl#init can end up in a Long overflow if the underlying implementation does not know the exact value of the number children, and the child node count is higher than maxSize.
> I will attach a patch with a test case



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)