Skip to content

Commit

Permalink
Merge branch 'develop'
Browse files Browse the repository at this point in the history
Updating to 1.3.1-16
  • Loading branch information
stevejenkins committed Apr 11, 2016
2 parents 568a662 + c92c873 commit 0563569
Show file tree
Hide file tree
Showing 6 changed files with 122 additions and 6 deletions.
15 changes: 15 additions & 0 deletions PATCHES/opendmarc.ticket115.patch
Original file line number Diff line number Diff line change
@@ -0,0 +1,15 @@
Patch thanks to Sebastian A. Siewior

Index: opendmarc-1.3.1+dfsg/libopendmarc/opendmarc_dns.c
===================================================================
--- opendmarc-1.3.1+dfsg.orig/libopendmarc/opendmarc_dns.c 2015-02-23 15:31:51.000000000 -0500
+++ opendmarc-1.3.1+dfsg/libopendmarc/opendmarc_dns.c 2015-02-25 17:59:28.302473233 -0500
@@ -217,7 +217,7 @@
_res.options |= RES_USE_DNSSEC;
#endif
(void) opendmarc_policy_library_dns_hook(&_res.nscount,
- &_res.nsaddr_list);
+ _res.nsaddr_list);
answer_len = res_query(bp, C_IN, T_TXT, answer_buf, sizeof answer_buf);
#endif /* HAVE_RES_NINIT */
if (answer_len < 0)
13 changes: 13 additions & 0 deletions PATCHES/opendmarc.ticket131.patch
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
Index: opendmarc-1.3.1/libopendmarc/opendmarc_policy.c
===================================================================
--- opendmarc-1.3.1.orig/libopendmarc/opendmarc_policy.c
+++ opendmarc-1.3.1/libopendmarc/opendmarc_policy.c
@@ -495,7 +495,7 @@ opendmarc_policy_store_dkim(DMARC_POLICY
* If so and if we have not already found
* a best match, make this the temporary best match.
*/
- if (opendmarc_policy_check_alignment(dp, pctx->from_domain, pctx->adkim))
+ if (opendmarc_policy_check_alignment(dp, pctx->from_domain, pctx->adkim) == 0)
{
if (pctx->dkim_domain != NULL)
{
45 changes: 45 additions & 0 deletions PATCHES/opendmarc.ticket138.patch
Original file line number Diff line number Diff line change
@@ -0,0 +1,45 @@
Index: opendmarc-code/opendmarc/opendmarc-config.h
===================================================================
--- opendmarc-code.orig/opendmarc/opendmarc-config.h 2015-06-29 09:50:43.247001227 -0300
+++ opendmarc-code/opendmarc/opendmarc-config.h 2015-06-29 09:59:37.092292267 -0300
@@ -32,6 +32,7 @@
{ "FailureReportsOnNone", CONFIG_TYPE_BOOLEAN, FALSE },
{ "FailureReportsSentBy", CONFIG_TYPE_STRING, FALSE },
{ "HistoryFile", CONFIG_TYPE_STRING, FALSE },
+ { "HoldQuarantinedMessages", CONFIG_TYPE_BOOLEAN, FALSE },
{ "IgnoreAuthenticatedClients", CONFIG_TYPE_BOOLEAN, FALSE },
{ "IgnoreHosts", CONFIG_TYPE_STRING, FALSE },
{ "IgnoreMailFrom", CONFIG_TYPE_STRING, FALSE },
Index: opendmarc-code/opendmarc/opendmarc.c
===================================================================
--- opendmarc-code.orig/opendmarc/opendmarc.c 2015-06-29 09:50:43.247001227 -0300
+++ opendmarc-code/opendmarc/opendmarc.c 2015-06-29 09:58:41.148988316 -0300
@@ -156,6 +156,7 @@
_Bool conf_spfselfvalidate;
#endif /* WITH_SPF */
_Bool conf_ignoreauthclients;
+ _Bool conf_holdquarantinedmessages;
unsigned int conf_refcnt;
unsigned int conf_dnstimeout;
struct config * conf_data;
@@ -1285,6 +1286,10 @@
&conf->conf_recordall,
sizeof conf->conf_recordall);

+ (void) config_get(data, "HoldQuarantinedMessages",
+ &conf->conf_holdquarantinedmessages,
+ sizeof conf->conf_holdquarantinedmessages);
+
(void) config_get(data, "IgnoreAuthenticatedClients",
&conf->conf_ignoreauthclients,
sizeof conf->conf_ignoreauthclients);
@@ -2971,7 +2976,8 @@
case DMARC_POLICY_QUARANTINE: /* Explicit quarantine */
aresult = "fail";

- if (conf->conf_rejectfail && random() % 100 < pct)
+ if (conf->conf_rejectfail && conf->conf_holdquarantinedmessages &&
+ random() % 100 < pct)
{
snprintf(replybuf, sizeof replybuf,
"quarantined by DMARC policy for %s",
21 changes: 21 additions & 0 deletions PATCHES/opendmarc.ticket139.patch
Original file line number Diff line number Diff line change
@@ -0,0 +1,21 @@
Description: add rDNS to FailureReport Source-IP field
as suggested on dmarc-discuss
URL: http://lists.dmarc.org/pipermail/dmarc-discuss/2015-September/003237.html
Author: A. Schulze
---
This patch header follows DEP-3: http://dep.debian.net/deps/dep3/
Index: opendmarc-1.3.1/opendmarc/opendmarc.c
===================================================================
--- opendmarc-1.3.1.orig/opendmarc/opendmarc.c
+++ opendmarc-1.3.1/opendmarc/opendmarc.c
@@ -2865,8 +2865,8 @@ mlfi_eom(SMFICTX *ctx)
dfc->mctx_envfrom);

dmarcf_dstring_printf(dfc->mctx_afrf,
- "Source-IP: %s\n",
- cc->cctx_ipstr);
+ "Source-IP: %s (%s)\n",
+ cc->cctx_ipstr, cc->cctx_host);

dmarcf_dstring_printf(dfc->mctx_afrf,
"Reported-Domain: %s\n\n",
2 changes: 0 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,6 @@ This git repo contains:

**CONTRIBUTORS:** Please do pull requests in the "develop" branch only. The "master" branch is just for release versions. Thanks!

For the SystemV-compatible version of this repo, visit http://github.com/stevejenkins/OpenDMARC-SysV/

For official source code and documentation, please visit http://www.trusteddomain.org/opendmarc/

Find/follow/friend me at:
Expand Down
32 changes: 28 additions & 4 deletions SPECS/opendmarc.spec
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
Summary: A Domain-based Message Authentication, Reporting & Conformance (DMARC) milter and library
Name: opendmarc
Version: 1.3.1
Release: 13%{?dist}
Release: 16%{?dist}
Group: System Environment/Daemons
License: BSD and Sendmail
URL: http://www.trusteddomain.org/%{name}.html
Expand Down Expand Up @@ -36,7 +36,17 @@ Requires(post): policycoreutils

BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)

# Patch0: %{name}.patchname.patch
# https://sourceforge.net/p/opendmarc/tickets/115/
Patch0: %{name}.ticket115.patch

# https://sourceforge.net/p/opendmarc/tickets/131/
Patch1: %{name}.ticket131.patch

# https://sourceforge.net/p/opendmarc/tickets/138/
Patch2: %{name}.ticket138.patch

# https://sourceforge.net/p/opendmarc/tickets/139/
Patch3: %{name}.ticket139.patch

%description
%{upname} (Domain-based Message Authentication, Reporting & Conformance)
Expand Down Expand Up @@ -68,11 +78,16 @@ required for developing applications against libopendmarc.
%prep
%setup -q
%if %systemd
# Apply systemd patches
# Apply systemd-only patches
#%patch0 -p1
%else
# Apply SysV patches
# Apply SysV-only patches
#%patch0 -p1
# Apply Global patches
%patch0 -p1
%patch1 -p1
%patch2 -p1
%patch3 -p1
%endif

%build
Expand Down Expand Up @@ -233,6 +248,15 @@ exit 0
%{_libdir}/*.so

%changelog
* Mon Apr 11 2016 Steve Jenkins <[email protected]> - 1.3.1-16
- Added patches for SourceForge tickets 115, 131, 138, 139

* Thu Feb 04 2016 Fedora Release Engineering <[email protected]> - 1.3.1-15
- Rebuilt for https://fedoraproject.org/wiki/Fedora_24_Mass_Rebuild

* Wed Jun 17 2015 Fedora Release Engineering <[email protected]> - 1.3.1-14
- Rebuilt for https://fedoraproject.org/wiki/Fedora_23_Mass_Rebuild

* Wed Apr 29 2015 Steve Jenkins <[email protected]> - 1.3.1-13
- Replaced various commands with rpm macros
- Included support for systemd macros (#1216881)
Expand Down

0 comments on commit 0563569

Please sign in to comment.