commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <>
Subject [jira] [Commented] (COLLECTIONS-673) ListUtils.partition potential integer overflow
Date Sat, 09 Jun 2018 17:34:00 GMT


ASF GitHub Bot commented on COLLECTIONS-673:

GitHub user sfuhrm opened a pull request:

    COLLECTIONS-673: Fix inspired by the Guava partition() implementation

    A fix for the COLLECTIONS-673 bug and a unit test proving the fix for the shown defect.

You can merge this pull request into a Git repository by running:

    $ git pull COLLECTIONS-673

Alternatively you can review and apply these changes as the patch at:

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #37
commit faf27f611f4429c77a800124b5fb6f641f871c0f
Author: Stephan Fuhrmann <s@...>
Date:   2018-06-09T17:30:13Z

    COLLECTIONS-673: Fix inspired by the Guava partition() implementation


> ListUtils.partition potential integer overflow
> ----------------------------------------------
>                 Key: COLLECTIONS-673
>                 URL:
>             Project: Commons Collections
>          Issue Type: Bug
>          Components: List
>    Affects Versions: 4.1
>            Reporter: John Mark
>            Priority: Major
> When calling {{ListUtils.partition()}} with a large size and large list, it is possible
that an integer overflow will occur in the {{size()}} method that causes incorrect behavior.
This will occur when using a size that, when added to list.size() will be larger than {{Integer.MAX_VALUE}}
> Current version of Guava's {{Lists.partition()}} handle this correctly, so perhaps the
code for {{ListUtils.partition()}} needs to be updated based on the latest Guava code.
> A simple illustration of this:
> {code}
> List<String> aList = Arrays.asList("1", "2", "3", "4", "5");
> List<List<String>> partitioned = ListUtils.partition(aList, Integer.MAX_VALUE);
> System.out.println("Number of partitions: " + partitioned.size());
> for(List<String> l : partitioned)  {
>      System.out.println(l);
> }
> {code}
> The above code works correctly when using Guava's {{Lists.partition()}} instead.

This message was sent by Atlassian JIRA

View raw message