Abort sending AddName/Refresh requests when the server is shutting down

Registering three names (the same host both as workstation and server, plus the
workgroup name) with five retries each at two second intervals takes 30 seconds,
and with more names to announce it'd only get worse.

Right now, if the server is being shut down shortly after having been started,
we have to wait for all those AddName requests to be processes, which seems a
bit pointless.

Author: buttercookie42

src/main/java/org/filesys/netbios/server/NetBIOSNameServer.java

@@ -198,7 +198,7 @@ else if (m_hshutdown == true)
                     boolean txsts = true;
                     int retry = 0;
 
-                    while (req.hasErrorStatus() == false && retry++ < reqRetry) {
+                    retryLoop: while (req.hasErrorStatus() == false && retry++ < reqRetry) {
 
                         //	Debug
                         if (Debug.EnableInfo && hasDebug())
@@ -210,6 +210,9 @@ else if (m_hshutdown == true)
                             //	Add name request
                             case ADD_NAME:
 
+                            	// The server is shutting down, we can stop handling ADD_NAME
+                            	if (m_shutdown)
+                            		break retryLoop;
                                 //	Check if a WINS server is configured
                                 if (hasPrimaryWINSServer())
                                     txsts = sendAddName(req, getPrimaryWINSServer(), false);
@@ -230,6 +233,9 @@ else if (m_hshutdown == true)
                             //	Refresh name request
                             case REFRESH_NAME:
 
+                            	// The server is shutting down, we can stop handling REFRESH_NAME
+                            	if (m_shutdown)
+                            		break retryLoop;
                                 //	Check if a WINS server is configured
                                 if (hasPrimaryWINSServer())
                                     txsts = sendRefreshName(req, getPrimaryWINSServer(), false);
@@ -1617,6 +1623,9 @@ public void shutdownServer(boolean immediate) {
                 Debug.println(ex);
         }
 
+        //	Indicate that the server is closing
+        m_shutdown = true;
+
         //	If the shutdown is not immediate then release all of the names registered by this server
         if (isActive() && immediate == false) {
 
@@ -1663,9 +1672,6 @@ public void shutdownServer(boolean immediate) {
                 Debug.println(ex);
         }
 
-        //	Indicate that the server is closing
-        m_shutdown = true;
-
         try {
 
             //	Close the server socket so that any pending receive is cancelled