changeset 1324:806c5e4d1265

6513803: httpserver regression test Test13 failing and causing NullPointerException Summary: check for NPEs Reviewed-by: chegar
author michaelm
date Mon, 29 Jun 2009 13:10:33 +0100
parents 5a3a5388756c
children b6721df9ae0a 89b14d3740dc
files test/com/sun/net/httpserver/Test1.java test/com/sun/net/httpserver/Test12.java test/com/sun/net/httpserver/Test13.java test/com/sun/net/httpserver/Test3.java test/com/sun/net/httpserver/Test4.java test/com/sun/net/httpserver/Test5.java test/com/sun/net/httpserver/Test9.java test/com/sun/net/httpserver/Test9a.java test/com/sun/net/httpserver/TestLogging.java
diffstat 9 files changed, 44 insertions(+), 22 deletions(-) [+]
line wrap: on
line diff
--- a/test/com/sun/net/httpserver/Test1.java	Thu Jun 25 17:01:56 2009 -0400
+++ b/test/com/sun/net/httpserver/Test1.java	Mon Jun 29 13:10:33 2009 +0100
@@ -86,9 +86,12 @@
             System.out.println ("OK");
         } finally {
             delay();
-            s1.stop(2);
-            s2.stop(2);
-            executor.shutdown ();
+            if (s1 != null)
+                s1.stop(2);
+            if (s2 != null)
+                s2.stop(2);
+            if (executor != null)
+                executor.shutdown ();
         }
     }
 
--- a/test/com/sun/net/httpserver/Test12.java	Thu Jun 25 17:01:56 2009 -0400
+++ b/test/com/sun/net/httpserver/Test12.java	Mon Jun 29 13:10:33 2009 +0100
@@ -85,9 +85,12 @@
             System.out.println ("OK");
         } finally {
             delay();
-            s1.stop(2);
-            s2.stop(2);
-            executor.shutdown ();
+            if (s1 != null)
+                s1.stop(2);
+            if (s2 != null)
+                s2.stop(2);
+            if (executor != null)
+                executor.shutdown ();
         }
     }
 
--- a/test/com/sun/net/httpserver/Test13.java	Thu Jun 25 17:01:56 2009 -0400
+++ b/test/com/sun/net/httpserver/Test13.java	Mon Jun 29 13:10:33 2009 +0100
@@ -81,9 +81,12 @@
             System.out.println ("OK");
         } finally {
             delay();
-            s1.stop(2);
-            s2.stop(2);
-            executor.shutdown ();
+            if (s1 != null)
+                s1.stop(2);
+            if (s2 != null)
+                s2.stop(2);
+            if (executor != null)
+                executor.shutdown ();
         }
     }
 
--- a/test/com/sun/net/httpserver/Test3.java	Thu Jun 25 17:01:56 2009 -0400
+++ b/test/com/sun/net/httpserver/Test3.java	Mon Jun 29 13:10:33 2009 +0100
@@ -62,8 +62,10 @@
             System.out.println ("OK");
         } finally {
             delay();
-            server.stop(2);
-            exec.shutdown();
+            if (server != null)
+                server.stop(2);
+            if (exec != null)
+                exec.shutdown();
         }
     }
 
--- a/test/com/sun/net/httpserver/Test4.java	Thu Jun 25 17:01:56 2009 -0400
+++ b/test/com/sun/net/httpserver/Test4.java	Mon Jun 29 13:10:33 2009 +0100
@@ -60,8 +60,10 @@
             System.out.println ("OK");
         } finally {
             delay();
-            server.stop(2);
-            exec.shutdown();
+            if (server != null)
+                server.stop(2);
+            if (exec != null)
+                exec.shutdown();
         }
     }
 
--- a/test/com/sun/net/httpserver/Test5.java	Thu Jun 25 17:01:56 2009 -0400
+++ b/test/com/sun/net/httpserver/Test5.java	Mon Jun 29 13:10:33 2009 +0100
@@ -61,8 +61,10 @@
             System.out.println ("OK");
         } finally {
             delay ();
-            server.stop(2);
-            exec.shutdown();
+            if (server != null)
+                server.stop(2);
+            if (exec != null)
+                exec.shutdown();
         }
     }
 
--- a/test/com/sun/net/httpserver/Test9.java	Thu Jun 25 17:01:56 2009 -0400
+++ b/test/com/sun/net/httpserver/Test9.java	Mon Jun 29 13:10:33 2009 +0100
@@ -97,9 +97,12 @@
             System.out.println ("OK");
         } finally {
             delay();
-            s1.stop(2);
-            s2.stop(2);
-            executor.shutdown ();
+            if (s1 != null)
+                s1.stop(2);
+            if (s2 != null)
+                s2.stop(2);
+            if (executor != null)
+                executor.shutdown ();
         }
     }
 
--- a/test/com/sun/net/httpserver/Test9a.java	Thu Jun 25 17:01:56 2009 -0400
+++ b/test/com/sun/net/httpserver/Test9a.java	Mon Jun 29 13:10:33 2009 +0100
@@ -92,8 +92,10 @@
             System.out.println ("OK");
         } finally {
             delay();
-            server.stop(2);
-            executor.shutdown ();
+            if (server != null)
+                server.stop(2);
+            if (executor != null)
+                executor.shutdown();
         }
     }
 
--- a/test/com/sun/net/httpserver/TestLogging.java	Thu Jun 25 17:01:56 2009 -0400
+++ b/test/com/sun/net/httpserver/TestLogging.java	Mon Jun 29 13:10:33 2009 +0100
@@ -89,8 +89,10 @@
             System.out.println ("OK");
         } finally {
             delay();
-            s1.stop(2);
-            executor.shutdown ();
+            if (s1 != null)
+                s1.stop(2);
+            if (executor != null)
+                executor.shutdown();
         }
     }
 }