This is an automated email from the ASF dual-hosted git repository. pauls pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/sling-org-apache-sling-resourceresolver.git
commit cbf85ddfc9e417958152b42b4f0ac376866c7c10 Author: Karl Pauls <[email protected]> AuthorDate: Wed Oct 10 23:57:15 2018 +0200 SLING-8006: Double check that the list is not used when empty to make sure. --- .../impl/providers/ResourceProviderTracker.java | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/src/main/java/org/apache/sling/resourceresolver/impl/providers/ResourceProviderTracker.java b/src/main/java/org/apache/sling/resourceresolver/impl/providers/ResourceProviderTracker.java index 97bb488..98d60ff 100644 --- a/src/main/java/org/apache/sling/resourceresolver/impl/providers/ResourceProviderTracker.java +++ b/src/main/java/org/apache/sling/resourceresolver/impl/providers/ResourceProviderTracker.java @@ -152,10 +152,12 @@ public class ResourceProviderTracker implements ResourceProviderStorageProvider synchronized ( this.handlers ) { this.reporterGenerator = generator; for (List<ResourceProviderHandler> list : handlers.values()) { - final ResourceProviderHandler h = list.get(0); - if (h != null) { - updateProviderContext(h); - h.update(); + if ( !list.isEmpty() ) { + final ResourceProviderHandler h = list.get(0); + if (h != null) { + updateProviderContext(h); + h.update(); + } } } }
