]> git.ipfire.org Git - thirdparty/knot-resolver.git/commitdiff
lib/dnssec: wrong prepend_asterisk() usage was fixed, cleanup
authorGrigorii Demidov <grigorii.demidov@nic.cz>
Mon, 18 Jan 2016 09:27:19 +0000 (10:27 +0100)
committerGrigorii Demidov <grigorii.demidov@nic.cz>
Mon, 18 Jan 2016 09:27:19 +0000 (10:27 +0100)
lib/dnssec/nsec3.c

index 5ea26792a1c8938003bd07b0553c9f9deb07afb2..a3cb0ab63e92a25f44a02bc1fe49268df420988a 100644 (file)
@@ -353,7 +353,7 @@ fail:
  *
  * @param tgt  Target buffer to write domain name into.
  * @param name Name to be added to the asterisk.
- * @return     0 or error code
+ * @return     Size of the resulting name or error code.
  */
 static int prepend_asterisk(uint8_t *tgt, size_t maxlen, const knot_dname_t *name)
 {
@@ -639,9 +639,10 @@ static int matches_closest_encloser_wildcard(const knot_pkt_t *pkt, knot_section
 
        uint8_t wildcard[KNOT_DNAME_MAXLEN];
        int ret = prepend_asterisk(wildcard, sizeof(wildcard), encloser);
-       if (ret < 3) {
+       if (ret < 0) {
                return ret;
        }
+       assert(ret >= 3);
 
        int flags;
        for (unsigned i = 0; i < sec->count; ++i) {