You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spamassassin.apache.org by gb...@apache.org on 2023/09/04 09:36:11 UTC

svn commit: r1912075 - /spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/DMARC.pm

Author: gbechis
Date: Mon Sep  4 09:36:11 2023
New Revision: 1912075

URL: http://svn.apache.org/viewvc?rev=1912075&view=rev
Log:
avoid a warning

Modified:
    spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/DMARC.pm

Modified: spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/DMARC.pm
URL: http://svn.apache.org/viewvc/spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/DMARC.pm?rev=1912075&r1=1912074&r2=1912075&view=diff
==============================================================================
--- spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/DMARC.pm (original)
+++ spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/DMARC.pm Mon Sep  4 09:36:11 2023
@@ -364,12 +364,17 @@ sub _check_dmarc {
       $old_arc_index = $spf_parse->{arc_index};
     }
 
-    my @arc_seals = sort { ( $a->{arc_verifier}{seals}{tags_by_name}{i}{value} // 0 ) <=> ( $b->{arc_verifier}{seals}{tags_by_name}{i}{value} // 0 ) } @{$pms->{arc_verifier}{seals}};
-    foreach my $seals ( @arc_seals ) {
-      if(exists($seals->{tags_by_name}{d}) and exists($pms->{arc_author_domains}->{$mfrom_domain})) {
-        dbg("Evaluate DMARC using AAR dkim information for index $seals->{tags_by_name}{i}{value}");
-        $dmarc->dkim(domain => $mfrom_domain, selector => $seals->{tags_by_name}{s}{value}, result => $seals->{verify_result});
-        last;
+    my @tmp_arc_seals;
+    my @arc_seals;
+    if(defined $pms->{arc_verifier}{seals}) {
+      @tmp_arc_seals = @{$pms->{arc_verifier}{seals}};
+      @arc_seals = sort { ( $a->{arc_verifier}{seals}{tags_by_name}{i}{value} // 0 ) <=> ( $b->{arc_verifier}{seals}{tags_by_name}{i}{value} // 0 ) } @tmp_arc_seals;
+      foreach my $seals ( @arc_seals ) {
+        if(exists($seals->{tags_by_name}{d}) and exists($pms->{arc_author_domains}->{$mfrom_domain})) {
+          dbg("Evaluate DMARC using AAR dkim information for index $seals->{tags_by_name}{i}{value}");
+          $dmarc->dkim(domain => $mfrom_domain, selector => $seals->{tags_by_name}{s}{value}, result => $seals->{verify_result});
+          last;
+        }
       }
     }