[freenet-cvs] r15032 - trunk/plugins/UPnP

toad at freenetproject.org toad at freenetproject.org
Sat Sep 8 12:06:25 UTC 2007


Author: toad
Date: 2007-09-08 12:06:24 +0000 (Sat, 08 Sep 2007)
New Revision: 15032

Modified:
   trunk/plugins/UPnP/UPnP.java
Log:
Consistently use lock object not this

Modified: trunk/plugins/UPnP/UPnP.java
===================================================================
--- trunk/plugins/UPnP/UPnP.java	2007-09-08 12:04:09 UTC (rev 15031)
+++ trunk/plugins/UPnP/UPnP.java	2007-09-08 12:06:24 UTC (rev 15032)
@@ -153,7 +153,7 @@
 	
 	private void registerPortMappings() {
 		Set ports;
-		synchronized(this) {
+		synchronized(lock) {
 			ports = portsToForward;
 		}
 		if(ports == null) return;
@@ -204,7 +204,7 @@
 	
 	public void unregisterPortMappings() {
 		Set ports;
-		synchronized(this) {
+		synchronized(lock) {
 			ports = portsForwarded;
 		}
 		this.unregisterPorts(ports);
@@ -422,7 +422,7 @@
 		add.setArgumentValue("NewLeaseDuration", 0);
 		
 		if(add.postControlAction()) {
-			synchronized(this) {
+			synchronized(lock) {
 				portsForwarded.add(fp);
 			}
 			return true;
@@ -444,7 +444,7 @@
 		remove.setArgumentValue("NewProtocol", protocol);
 		
 		boolean retval = remove.postControlAction();
-		synchronized(this) {
+		synchronized(lock) {
 			portsForwarded.remove(fp);
 		}
 		




More information about the cvs mailing list