You are viewing a plain text version of this content. The canonical link for it is here.
Posted to derby-commits@db.apache.org by kr...@apache.org on 2012/04/04 08:55:01 UTC

svn commit: r1309264 - /db/derby/code/trunk/java/drda/org/apache/derby/drda/NetServlet.java

Author: kristwaa
Date: Wed Apr  4 06:55:01 2012
New Revision: 1309264

URL: http://svn.apache.org/viewvc?rev=1309264&view=rev
Log:
DERBY-5328: The private fields of the NetServlet can be changed by multiple threads, giving rise to race conditions and corruptions.

Made doGet synchronized.

Patch file: derby-5328-02-aa-synchronized_doget.diff

Modified:
    db/derby/code/trunk/java/drda/org/apache/derby/drda/NetServlet.java

Modified: db/derby/code/trunk/java/drda/org/apache/derby/drda/NetServlet.java
URL: http://svn.apache.org/viewvc/db/derby/code/trunk/java/drda/org/apache/derby/drda/NetServlet.java?rev=1309264&r1=1309263&r2=1309264&view=diff
==============================================================================
--- db/derby/code/trunk/java/drda/org/apache/derby/drda/NetServlet.java (original)
+++ db/derby/code/trunk/java/drda/org/apache/derby/drda/NetServlet.java Wed Apr  4 06:55:01 2012
@@ -130,7 +130,8 @@ public class NetServlet extends HttpServ
 		Get the form of NetServlet. Provides buttons and forms to control the
 		Network server.
 	*/
-	public void doGet (HttpServletRequest request, HttpServletResponse response)
+    public synchronized void doGet (HttpServletRequest request,
+                                    HttpServletResponse response)
 	        throws ServletException, IOException
 	{
 		String logOnMessage;