]> granicus.if.org Git - pdns/commitdiff
gmysql compiles
authorKees Monshouwer <mind04@monshouwer.org>
Mon, 1 Jun 2015 21:07:01 +0000 (23:07 +0200)
committermind04 <mind04@monshouwer.org>
Tue, 30 Jun 2015 06:12:44 +0000 (08:12 +0200)
modules/gmysqlbackend/smysql.cc
pdns/backends/gsql/gsqlbackend.cc
pdns/backends/gsql/gsqlbackend.hh
pdns/backends/gsql/ssql.hh

index 8d981ef5589707bd28d1840a2c45f2ac2cd04639..5dc519b0ae7d857be1316575b40527da5f889b75 100644 (file)
@@ -117,6 +117,9 @@ public:
     d_paridx++;
     return this;
   }
+  SSqlStatement* bind(const string& name, const DNSName& value) {
+    return bind(name, stripDot(value.toString()));
+  }
   SSqlStatement* bindNull(const string& name) { 
     if (d_paridx >= d_parnum)
       throw SSqlException("Attempt to bind more parameters than query has: " + d_query);
index 5ea37df8cf158ee57d6f215ffc3c4d1818ec7b94..c8975e9c5bf589e9daf2e2f89f92f734e5e8e1ef 100644 (file)
@@ -917,22 +917,20 @@ bool GSQLBackend::setDomainMetadata(const string& name, const std::string& kind,
   return true;
 }
 
-void GSQLBackend::lookup(const QType &qtype,const string &qname, DNSPacket *pkt_p, int domain_id)
+void GSQLBackend::lookup(const QType &qtype,const DNSName &qname, DNSPacket *pkt_p, int domain_id)
 {
-  string lcqname=toLower(qname);
-
   try {
     if(qtype.getCode()!=QType::ANY) {
       if(domain_id < 0) {
         d_query_stmt = d_NoIdQuery_stmt;
         d_query_stmt->
           bind("qtype", qtype.getName())->
-          bind("qname", lcqname);
+          bind("qname", qname); // DNSNameFIXME: toLower()?
       } else {
         d_query_stmt = d_IdQuery_stmt;
         d_query_stmt->
           bind("qtype", qtype.getName())->
-          bind("qname", lcqname)->
+          bind("qname", qname)->
           bind("domain_id", domain_id);
       }
     } else {
@@ -940,11 +938,11 @@ void GSQLBackend::lookup(const QType &qtype,const string &qname, DNSPacket *pkt_
       if(domain_id < 0) {
         d_query_stmt = d_ANYNoIdQuery_stmt;
         d_query_stmt->
-          bind("qname", lcqname);
+          bind("qname", qname);
       } else {
         d_query_stmt = d_ANYIdQuery_stmt;
         d_query_stmt->
-          bind("qname", lcqname)->
+          bind("qname", qname)->
           bind("domain_id", domain_id);
       }
     }
@@ -959,7 +957,7 @@ void GSQLBackend::lookup(const QType &qtype,const string &qname, DNSPacket *pkt_
   d_qname=qname;
 }
 
-bool GSQLBackend::list(const string &target, int domain_id, bool include_disabled)
+bool GSQLBackend::list(const DNSName &target, int domain_id, bool include_disabled)
 {
   DLOG(L<<"GSQLBackend constructing handle for list of domain id '"<<domain_id<<"'"<<endl);
 
index 45cb116d53e02db9e42b0e0b34ce376441969ac5..91ef318d0c6a4d206d6b135db505b2870f0a5111 100644 (file)
@@ -162,8 +162,8 @@ public:
     release(&d_DeleteCommentsQuery_stmt);
   }
 
-  void lookup(const QType &, const string &qdomain, DNSPacket *p=0, int zoneId=-1);
-  bool list(const string &target, int domain_id, bool include_disabled=false);
+  void lookup(const QType &, const DNSName &qdomain, DNSPacket *p=0, int zoneId=-1);
+  bool list(const DNSName &target, int domain_id, bool include_disabled=false);
   bool get(DNSResourceRecord &r);
   void getAllDomains(vector<DomainInfo> *domains, bool include_disabled=false);
   bool isMaster(const string &domain, const string &ip);
@@ -222,7 +222,7 @@ public:
   bool replaceComments(const uint32_t domain_id, const string& qname, const QType& qt, const vector<Comment>& comments);
 
 private:
-  string d_qname;
+  DNSName d_qname;
   SSql *d_db;
   SSqlStatement::result_t d_result;
 
index 7256d6a06ec2afc136b7c8fd3513088fae657504..955bc1f7965fed26017dec6d262f4a5bca93aa0b 100644 (file)
@@ -13,6 +13,7 @@
 #include <vector>
 #include "../../namespaces.hh"
 #include <inttypes.h>
+#include "../../dnsname.hh"
 
 class SSqlException 
 {