commit d611ed211899d28525758fd70eb8ab96e0902132
author Sagar Miglani <saga@adobe.com> Mon Jan 31 17:59:15 2022 +0530
committer Sagar Miglani <saga@adobe.com> Mon Jan 31 17:59:15 2022 +0530
tree 42bb98045fe14153cd4800c8de4f3f921e7ca247
parent f9c1f1e1e7601ffb4640fb394e649a052c58a7fd

SLING-11103 - Sync updates with super

src/main/java/org/apache/sling/rewriter/impl/TransformerFactoryServiceTracker.java

diff --git a/src/main/java/org/apache/sling/rewriter/impl/TransformerFactoryServiceTracker.java b/src/main/java/org/apache/sling/rewriter/impl/TransformerFactoryServiceTracker.java
index 6b9d390..d80a676 100644
--- a/src/main/java/org/apache/sling/rewriter/impl/TransformerFactoryServiceTracker.java
+++ b/src/main/java/org/apache/sling/rewriter/impl/TransformerFactoryServiceTracker.java
@@ -63,12 +63,13 @@
     public Object addingService(ServiceReference reference) {
         final boolean isGlobal = isGlobal(reference);
         LOGGER.debug("Adding service {}, isGlobal={}", reference.getClass(), isGlobal);
-        if ( isGlobal ) {
-            synchronized (this) {
+        Object obj = null;
+        synchronized (this) {
+            if ( isGlobal ) {
                 this.cacheIsValid = false;
             }
+            obj = super.addingService(reference);
         }
-        Object obj = super.addingService(reference);
         if ( obj == null && isGlobal ) {
             obj = this.context.getService(reference);
         }
@@ -82,12 +83,12 @@
     public void removedService(ServiceReference reference, Object service) {
         final boolean isGlobal = isGlobal(reference);
         LOGGER.debug("Removing service {}, isGlobal={}", reference.getClass(), isGlobal);
-        if ( isGlobal ) {
-            synchronized (this) {
+        synchronized (this) {
+            if ( isGlobal ) {
                 this.cacheIsValid = false;
             }
+            super.removedService(reference, service);
         }
-        super.removedService(reference, service);
     }
 
     /**