Attachment 48041 Details for Bug 149863 – Simplified patch for the unsynchronized access fix

Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

[patch] Simplified patch for the unsynchronized access fix

patch.txt (text/plain), 3.24 KB, created by BJ Hargrave

on 2006-08-16 13:42:48 EDT

(hide)

Description:

Filename:

MIME Type:

Creator: BJ Hargrave

Created: 2006-08-16 13:42:48 EDT

Size: 3.24 KB

patch

obsolete

>### Eclipse Workspace Patch 1.0
>#P org.eclipse.osgi
>Index: core/framework/org/eclipse/osgi/framework/internal/core/BundleContextImpl.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.osgi/core/framework/org/eclipse/osgi/framework/internal/core/BundleContextImpl.java,v
>retrieving revision 1.23
>diff -u -r1.23 BundleContextImpl.java
>--- core/framework/org/eclipse/osgi/framework/internal/core/BundleContextImpl.java	21 Jun 2006 03:41:01 -0000	1.23
>+++ core/framework/org/eclipse/osgi/framework/internal/core/BundleContextImpl.java	16 Aug 2006 17:44:01 -0000
>@@ -92,21 +92,29 @@
> 	protected void close() {
> 		valid = false; /* invalidate context */
> 
>-		if (serviceEvent != null) {
>-			framework.serviceEvent.removeListener(this);
>-			serviceEvent = null;
>-		}
>-		if (frameworkEvent != null) {
>-			framework.frameworkEvent.removeListener(this);
>-			frameworkEvent = null;
>-		}
>-		if (bundleEvent != null) {
>-			framework.bundleEvent.removeListener(this);
>-			bundleEvent = null;
>-		}
>-		if (bundleEventSync != null) {
>-			framework.bundleEventSync.removeListener(this);
>-			bundleEventSync = null;
>+		synchronized (framework.serviceEvent) {
>+			if (serviceEvent != null) {
>+				framework.serviceEvent.removeListener(this);
>+				serviceEvent = null;
>+			}
>+		}
>+		synchronized (framework.frameworkEvent) {
>+			if (frameworkEvent != null) {
>+				framework.frameworkEvent.removeListener(this);
>+				frameworkEvent = null;
>+			}
>+		}
>+		synchronized (framework.bundleEvent) {
>+			if (bundleEvent != null) {
>+				framework.bundleEvent.removeListener(this);
>+				bundleEvent = null;
>+			}
>+		}
>+		synchronized (framework.bundleEventSync) {
>+			if (bundleEventSync != null) {
>+				framework.bundleEventSync.removeListener(this);
>+				bundleEventSync = null;
>+			}
> 		}
> 
> 		/* service's registered by the bundle, if any, are unregistered. */
>@@ -348,8 +356,8 @@
> 			Debug.println("removeServiceListener[" + bundle + "](" + listenerName + ")"); //$NON-NLS-1$ //$NON-NLS-2$ //$NON-NLS-3$
> 		}
> 
>-		if (serviceEvent != null) {
>-			synchronized (framework.serviceEvent) {
>+		synchronized (framework.serviceEvent) {
>+			if (serviceEvent != null) {
> 				serviceEvent.removeListener(listener);
> 			}
> 		}
>@@ -423,14 +431,14 @@
> 		if (listener instanceof SynchronousBundleListener) {
> 			framework.checkAdminPermission(getBundle(), AdminPermission.LISTENER);
> 
>-			if (bundleEventSync != null) {
>-				synchronized (framework.bundleEventSync) {
>+			synchronized (framework.bundleEventSync) {
>+				if (bundleEventSync != null) {
> 					bundleEventSync.removeListener(listener);
> 				}
> 			}
> 		} else {
>-			if (bundleEvent != null) {
>-				synchronized (framework.bundleEvent) {
>+			synchronized (framework.bundleEvent) {
>+				if (bundleEvent != null) {
> 					bundleEvent.removeListener(listener);
> 				}
> 			}
>@@ -489,8 +497,8 @@
> 			Debug.println("removeFrameworkListener[" + bundle + "](" + listenerName + ")"); //$NON-NLS-1$ //$NON-NLS-2$ //$NON-NLS-3$
> 		}
> 
>-		if (frameworkEvent != null) {
>-			synchronized (framework.frameworkEvent) {
>+		synchronized (framework.frameworkEvent) {
>+			if (frameworkEvent != null) {
> 				frameworkEvent.removeListener(listener);
> 			}
> 		}

Actions: View | Diff

Attachments on bug 149863: 45857 | 47796 | 48040 | 48041