[Bug 910] New: "dkim_canon = simple" uses relaxed canonicalisation

View: New views
2 Messages — Rating Filter:   Alert me  

[Bug 910] New: "dkim_canon = simple" uses relaxed canonicalisation

by Bugzilla from bugzilla.exim.simon@arlott.org :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

------- You are receiving this mail because: -------
You are on the CC list for the bug.

http://bugs.exim.org/show_bug.cgi?id=910
           Summary: "dkim_canon = simple" uses relaxed canonicalisation
           Product: Exim
           Version: N/A
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: bug
          Priority: critical
         Component: Unfiled
        AssignedTo: tom@...
        ReportedBy: bugzilla.exim.simon@...
                CC: exim-dev@...


The code in src/dkim.c incorrectly handles both "relaxed" and "simple" as
"relaxed":
  if (Ustrcmp(dkim_canon, "relaxed") == 0)
    pdkim_canon = PDKIM_CANON_RELAXED;
  else if (Ustrcmp(dkim_canon, "simple") == 0)
    pdkim_canon = PDKIM_CANON_RELAXED;
  else {
    log_write(0, LOG_MAIN, "DKIM: unknown canonicalization method '%s',
defaulting to 'relaxed'.\n",dkim_canon);
    pdkim_canon = PDKIM_CANON_RELAXED;
  }


--
Configure bugmail: http://bugs.exim.org/userprefs.cgi?tab=email

--
## List details at http://lists.exim.org/mailman/listinfo/exim-dev Exim details at http://www.exim.org/ ##

[Bug 910] "dkim_canon = simple" uses relaxed canonicalisation

by Bugzilla from nigel@exim.org :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

------- You are receiving this mail because: -------
You are on the CC list for the bug.

http://bugs.exim.org/show_bug.cgi?id=910

Nigel Metheringham <nigel@...> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED




--- Comment #1 from Nigel Metheringham <nigel@...>  2009-11-12 13:02:17 ---
CVS commit by nm4:

Correct issue with relaxed/simple handling. Fixes: bug#910

--- CVS commit summary ---
1.590  1.591  +2 -0  -  exim/exim-doc/doc-txt/ChangeLog
1.8    1.9    +1 -1  -  exim/exim-src/src/dkim.c


--
Configure bugmail: http://bugs.exim.org/userprefs.cgi?tab=email

--
## List details at http://lists.exim.org/mailman/listinfo/exim-dev Exim details at http://www.exim.org/ ##