Check the return value before setting the flag for SCC read mutex

Fixes eclipse-openj9#9997 Signed-off-by: Hang Shao <[email protected]>
hangshao0 · Jun 29, 2020 · e20f374 · e20f374
1 parent 67ff58c
commit e20f374
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/runtime/shared_common/CompositeCache.cpp b/runtime/shared_common/CompositeCache.cpp
@@ -2470,12 +2470,13 @@ SH_CompositeCacheImpl::enterReadMutex(J9VMThread* currentThread, const char* cal
 			if (rc != 0) {
 				PORT_ACCESS_FROM_PORT(_portlib);
 				CC_ERR_TRACE1(J9NLS_SHRC_CC_FAILED_EXIT_MUTEX, rc);
+				decReaderCount(currentThread);
 			}
 		}
 	}
-
-	currentThread->privateFlags2 |= J9_PRIVATE_FLAGS2_IN_SHARED_CACHE_READ_MUTEX;
-
+	if (0 == rc) {
+		currentThread->privateFlags2 |= J9_PRIVATE_FLAGS2_IN_SHARED_CACHE_READ_MUTEX;
+	}
 	Trc_SHR_CC_enterReadMutex_Exit(currentThread, caller, rc);
 	return rc;
 }