diff options
author | Eike Rathke <erack@redhat.com> | 2018-01-22 19:55:38 +0100 |
---|---|---|
committer | Eike Rathke <erack@redhat.com> | 2018-01-23 11:35:50 +0100 |
commit | 03f429665b4d57f63b8e2862a55e5c4273bc7a2b (patch) | |
tree | 5bc06777123994b00bbeb699c8a67f04680bc342 /sd/res | |
parent | 43836528f445061d622b8edc7773ee40798d3a59 (diff) |
Flatten ScRangeList::Join() recursion
Joining an existing list of identical ranges could end up in a
very deep (but not infinite) recursion which could lead to stack
exhaustion. Recursion is unnecessary if we re-enter the loop with
the picked range. Continuing the loop as it was done after
recursion isn't necessary either, to the contrary, as long as
there is one join try the entire list again.
Commit f6eefd96cb16a9b5607fe59bdbf9b1121c9b56e8 pointed this out
as the UBSan build runs with ulimit -s 8192
Change-Id: Ibbc542fc8ae6d1509744aa731771eb6e32a38841
Reviewed-on: https://gerrit.libreoffice.org/48360
Reviewed-by: Eike Rathke <erack@redhat.com>
Tested-by: Jenkins <ci@libreoffice.org>
Diffstat (limited to 'sd/res')
0 files changed, 0 insertions, 0 deletions