- dnsbl.c:dnsbl_positive(): fixed weird logic that could result in 'text_type' being...

  with two commas which later would defeat the purpose of *(strrchr(text_type, ',')) = '\0'

git-svn-id: svn://svn.ircd-hybrid.org/svnroot/hopm/trunk@6232 82007160-df01-0410-b94d-b575c5fd34c7

diff --git a/src/dnsbl.c b/src/dnsbl.c
index f8ea9a471f23de34c9682d81084cab616fe0c2e0..d73cd4ff48d78df480c575e6d3823eec692c37d0 100644 (file)
--- a/src/dnsbl.c
+++ b/src/dnsbl.c
@@ -111,11 +111,8 @@ dnsbl_positive(struct scan_struct *ss, struct BlacklistConf *bl, unsigned char t
 
       if (item->number & type)
       {
-        strncat(text_type, item->type, sizeof(text_type) - strlen(text_type) - 2);
-        text_type[sizeof(text_type) - 2] = '\0';
-
-        strncat(text_type, ", ", sizeof(text_type) - strlen(text_type) - 1);
-        text_type[sizeof(text_type) - 1] = '\0';
+        strlcat(text_type, item->type, sizeof(text_type));
+        strlcat(text_type, ", ", sizeof(text_type));
       }
     }