From 6dd0d0326e76d1688c6241fc7e6ef366db9ea230 Mon Sep 17 00:00:00 2001
From: Peter van Dijk <peter.van.dijk@powerdns.com>
Date: Fri, 1 Oct 2021 17:03:24 +0200
Subject: [PATCH] pdnsutil check-zone: warn about @ in SOA RNAME

Co-authored-by: Kees Monshouwer <mind04@monshouwer.org>
---
 pdns/pdnsutil.cc | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/pdns/pdnsutil.cc b/pdns/pdnsutil.cc
index a4ed9a8195..ea8e305536 100644
--- a/pdns/pdnsutil.cc
+++ b/pdns/pdnsutil.cc
@@ -370,6 +370,13 @@ static int checkZone(DNSSECKeeper &dk, UeberBackend &B, const DNSName& zone, con
         cout<<"[Warning] SOA autocomplete is deprecated, missing field(s) in SOA content: "<<rr.qname<<" IN " <<rr.qtype.toString()<< " '" << rr.content<<"'"<<endl;
       }
 
+      if(parts.size() >= 2) {
+        if(parts[1].find('@') != string::npos) {
+          cout<<"[Warning] Found @-sign in SOA RNAME, should probably be a dot (.): "<<rr.qname<<" IN " <<rr.qtype.toString()<< " '" << rr.content<<"'"<<endl;
+          numwarnings++;
+        }
+      }
+
       ostringstream o;
       o<<rr.content;
       for(int pleft=parts.size(); pleft < 7; ++pleft) {
-- 
2.47.2