From: JINMEI Tatuya <jinmei@isc.org>
Date: Wed, 28 Nov 2012 22:06:36 +0000 (-0800)
Subject: [2420] added note about current limitation of adding rrset and sig separately.
X-Git-Tag: bind10-1.0.0-beta-release~52^2~1^2~2^2
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=0c36cd61ca034017db27d2f03d7a791c4717813a;p=thirdparty%2Fkea.git

[2420] added note about current limitation of adding rrset and sig separately.
---

diff --git a/src/lib/datasrc/memory/zone_data_updater.h b/src/lib/datasrc/memory/zone_data_updater.h
index 43fb1e3d23..32320ef081 100644
--- a/src/lib/datasrc/memory/zone_data_updater.h
+++ b/src/lib/datasrc/memory/zone_data_updater.h
@@ -119,7 +119,8 @@ public:
     /// the zone; it's unusual that \c rrset is NULL, but is still possible
     /// if these RRsets are given separately to the loader, or if even the
     /// zone is half broken and really contains an RRSIG that doesn't have
-    /// any covered RRset.  This implementation supports these cases.
+    /// any covered RRset.  This implementation supports these cases (but
+    /// see the note below).
     ///
     /// There is one tricky case: Due to a limitation of the current
     /// implementation, it cannot accept an RRSIG for NSEC3 without the covered
@@ -128,6 +129,12 @@ public:
     /// should be very rare in practice, and hopefully wouldn't be a real
     /// issue.
     ///
+    /// \note Due to limitations of the current implementation, if a
+    /// (non RRSIG) RRset and its RRSIG are added separately in different
+    /// calls to this method, the second attempt will be rejected due to
+    /// an \c AddError exception.  This will be loosened in Trac
+    /// ticket #2441.
+    ///
     /// \throw NullRRset Both \c rrset and sig_rrset is NULL
     /// \throw AddError any of a variety of validation checks fail for the
     /// \c rrset and its associated \c sig_rrset.