609. [bug] getrrsetbyname() would crash lwresd if the server

found more SIGs than answers. [RT #554]

609. [bug] getrrsetbyname() would crash lwresd if the server
found more SIGs than answers. [RT #554]
bf00f50c · Brian Wellington · b74896ea · bf00f50c · bf00f50c
Commit bf00f50c authored Dec 12, 2000 by Brian Wellington
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 2 deletions

CHANGES CHANGES +3 -0

bin/named/lwdgrbn.c bin/named/lwdgrbn.c +2 -2

No files found.
--- a/CHANGES
+++ b/CHANGES

+ 609.	[bug]		getrrsetbyname() would crash lwresd if the server
+			found more SIGs than answers. [RT #554]
+
 608.	[func]		dnssec-signzone now adds a comment to the zone
 			with the time the file was signed.


--- a/bin/named/lwdgrbn.c
+++ b/bin/named/lwdgrbn.c
@@ -15,7 +15,7 @@
 * WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
 */

-/* $Id: lwdgrbn.c,v 1.3 2000/12/11 23:09:38 marka Exp $ */
+/* $Id: lwdgrbn.c,v 1.4 2000/12/12 07:25:04 bwelling Exp $ */

 #include <config.h>

@@ -167,7 +167,7 @@ lookup_done(isc_task_t *task, isc_event_t *event) {
 		     result == ISC_R_SUCCESS;
 		     result = dns_rdataset_next(sigrdataset))
 		{
-			INSIST(i < grbn->nrdatas);
+			INSIST(i < grbn->nsigs);
 			dns_rdataset_current(sigrdataset, &rdata);
 			dns_rdata_toregion(&rdata, &r);
 			grbn->sigs[i] = r.base;