[asterisk-commits] russell: trunk r111908 - /trunk/main/dnsmgr.c

[SVN commits to the Asterisk project]

Author: russell
Date: Fri Mar 28 17:45:43 2008
New Revision: 111908

URL: http://svn.digium.com/view/asterisk?view=rev&rev=111908
Log:
Note a minor race condition that I noticed while reviewing Jeff's changes
to this code.

Modified:
    trunk/main/dnsmgr.c

Modified: trunk/main/dnsmgr.c
URL: http://svn.digium.com/view/asterisk/trunk/main/dnsmgr.c?view=diff&rev=111908&r1=111907&r2=111908
==============================================================================
--- trunk/main/dnsmgr.c (original)
+++ trunk/main/dnsmgr.c Fri Mar 28 17:45:43 2008
@@ -21,6 +21,11 @@
  * \brief Background DNS update manager
  *
  * \author Kevin P. Fleming <kpfleming at digium.com> 
+ *
+ * \bug There is a minor race condition.  In the event that an IP address
+ * of a dnsmgr managed host changes, there is the potential for the consumer
+ * of that address to access the in_addr data at the same time that the dnsmgr
+ * thread is in the middle of updating it to the new address.
  */
 
 #include "asterisk.h"