[haiku-commits] r38178 - haiku/trunk/src/add-ons/kernel/network/protocols/udp

  • From: axeld@xxxxxxxxxxxxxxxx
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Tue, 17 Aug 2010 11:51:23 +0200 (CEST)

Author: axeld
Date: 2010-08-17 11:51:22 +0200 (Tue, 17 Aug 2010)
New Revision: 38178
Changeset: http://dev.haiku-os.org/changeset/38178
Ticket: http://dev.haiku-os.org/ticket/6446

Modified:
   haiku/trunk/src/add-ons/kernel/network/protocols/udp/udp.cpp
Log:
* UDP really wanted to make sure that the error buffer was freed, and just did
  it once more. This should be the final nail on bug #6446.


Modified: haiku/trunk/src/add-ons/kernel/network/protocols/udp/udp.cpp
===================================================================
--- haiku/trunk/src/add-ons/kernel/network/protocols/udp/udp.cpp        
2010-08-17 09:15:33 UTC (rev 38177)
+++ haiku/trunk/src/add-ons/kernel/network/protocols/udp/udp.cpp        
2010-08-17 09:51:22 UTC (rev 38178)
@@ -1254,14 +1254,13 @@
                case B_NET_ERROR_QUENCH:
                default:
                        // ignore them
-                       break;
+                       gBufferModule->free(buffer);
+                       return B_OK;
        }
 
-       if (notifyError != B_OK)
-               sUdpEndpointManager->ReceiveError(notifyError, buffer);
+       ASSERT(notifyError != B_OK);
 
-       gBufferModule->free(buffer);
-       return B_OK;
+       return sUdpEndpointManager->ReceiveError(notifyError, buffer);
 }
 
 


Other related posts:

  • » [haiku-commits] r38178 - haiku/trunk/src/add-ons/kernel/network/protocols/udp - axeld