Rollback last modifications. It already breaks one unit test.

ldns as a DNS tooling library should be able to handle uncomplient resource records.

diff --git a/error.c b/error.c
index 27277b8f390ef32f2ea13f309a2a434c79f0a645..ff240dcc82a15c47118708dab59ca89f14f0e668 100644 (file)
--- a/error.c
+++ b/error.c
@@ -88,7 +88,6 @@ ldns_lookup_table ldns_error_str[] = {
        { LDNS_STATUS_DNSSEC_NSEC_RR_NOT_COVERED, "RR not covered by the given NSEC RRs" },
        { LDNS_STATUS_DNSSEC_NSEC_WILDCARD_NOT_COVERED, "wildcard not covered by the given NSEC RRs" },
        { LDNS_STATUS_DNSSEC_NSEC3_ORIGINAL_NOT_FOUND, "original of NSEC3 hashed name could not be found" },
-       { LDNS_STATUS_WIRE_MISSING_RDATA_FIELDS, "More rdata fields were expected while parsing the wireformat data" },
        { 0, NULL }
 };
 

diff --git a/ldns/error.h b/ldns/error.h
index 4b2b62d7510a4f84fcc206f0259eb24a97c90da3..e17846f38b3b1bc4c0dbbd4dc02144a86f8fccd4 100644 (file)
--- a/ldns/error.h
+++ b/ldns/error.h
@@ -98,8 +98,7 @@ enum ldns_enum_status {
        LDNS_STATUS_DNSSEC_EXISTENCE_DENIED,
        LDNS_STATUS_DNSSEC_NSEC_RR_NOT_COVERED,
        LDNS_STATUS_DNSSEC_NSEC_WILDCARD_NOT_COVERED,
-       LDNS_STATUS_DNSSEC_NSEC3_ORIGINAL_NOT_FOUND,
-       LDNS_STATUS_WIRE_MISSING_RDATA_FIELDS
+       LDNS_STATUS_DNSSEC_NSEC3_ORIGINAL_NOT_FOUND
 };
 typedef enum ldns_enum_status ldns_status;
 

diff --git a/wire2host.c b/wire2host.c
index 5eb2acbd401ed10e2b9bae875248dd6665fdf6a2..e87fcdf5df64725d891079752778411bbd6ddb82 100644 (file)
--- a/wire2host.c
+++ b/wire2host.c
@@ -271,9 +271,6 @@ ldns_wire2rdf(ldns_rr *rr, const uint8_t *wire, size_t max, size_t *pos)
                        cur_rdf = NULL;
                }
        }
-       if (rdf_index < ldns_rr_descriptor_minimum(descriptor)) {
-               return LDNS_STATUS_WIRE_MISSING_RDATA_FIELDS;
-       }
 
        return LDNS_STATUS_OK;
 }
@@ -319,10 +316,10 @@ ldns_wire2rr(ldns_rr **rr_p, const uint8_t *wire, size_t max,
                status = ldns_wire2rdf(rr, wire, max, pos);
 
                LDNS_STATUS_CHECK_GOTO(status, status_error);
-               ldns_rr_set_question(rr, false);
+        ldns_rr_set_question(rr, false);
        } else {
-               ldns_rr_set_question(rr, true);
-       }
+        ldns_rr_set_question(rr, true);
+    }
 
        *rr_p = rr;
        return LDNS_STATUS_OK;