From: Russell Bryant <russell@russellbryant.com>
Date: Fri, 28 Mar 2008 22:45:43 +0000 (+0000)
Subject: Note a minor race condition that I noticed while reviewing Jeff's changes
X-Git-Tag: 1.6.2.0-beta1~2588
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=16b2720cd4dfdc9a741abfc2cf1080a2642d2507;p=thirdparty%2Fasterisk.git

Note a minor race condition that I noticed while reviewing Jeff's changes
to this code.


git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@111908 65c4cc65-6c06-0410-ace0-fbb531ad65f3
---

diff --git a/main/dnsmgr.c b/main/dnsmgr.c
index a258328501..ac13a8ab3c 100644
--- a/main/dnsmgr.c
+++ b/main/dnsmgr.c
@@ -21,6 +21,11 @@
  * \brief Background DNS update manager
  *
  * \author Kevin P. Fleming <kpfleming@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"