[packages/exim] Rel 6; upstream fixes and also fix logging of message-id
arekm
arekm at pld-linux.org
Sun Sep 5 13:51:21 CEST 2021
commit 16a62dd6f421d9ec2274d9d75793c7d4163b0459
Author: Arkadiusz Miśkiewicz <arekm at maven.pl>
Date: Sun Sep 5 13:51:12 2021 +0200
Rel 6; upstream fixes and also fix logging of message-id
exim.spec | 6 ++-
messageid.patch | 155 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++
2 files changed, 159 insertions(+), 2 deletions(-)
---
diff --git a/exim.spec b/exim.spec
index 456777c..3226683 100644
--- a/exim.spec
+++ b/exim.spec
@@ -24,7 +24,7 @@ Summary(pl.UTF-8): Agent Transferu Poczty Uniwersytetu w Cambridge
Summary(pt_BR.UTF-8): Servidor de correio eletrônico exim
Name: exim
Version: 4.94.2
-Release: 5
+Release: 6
Epoch: 2
License: GPL v2+
Group: Networking/Daemons/SMTP
@@ -50,7 +50,7 @@ Source15: %{name}4-smtp.pamd
Source16: %{name}on.png
# sh branch.sh
Patch100: %{name}-git.patch
-# Patch100-md5: 113794adc1a2d54a61193e61cc8f3ead
+# Patch100-md5: 706af51f29db2a91422081491fb85d87
Patch0: %{name}4-EDITME.patch
Patch1: %{name}4-monitor-EDITME.patch
Patch2: %{name}4-cflags.patch
@@ -58,6 +58,7 @@ Patch3: exim-defs.patch
Patch4: %{name}4-Makefile-Default.patch
# http://marc.merlins.org/linux/exim/files/sa-exim-cvs/localscan_dlopen_exim_4.20_or_better.patch
Patch5: localscan_dlopen_%{name}_4.20_or_better.patch
+Patch6: messageid.patch
Patch8: %{name}-spam-timeout.patch
@@ -182,6 +183,7 @@ Pliki nagłówkowe dla Exima.
%patch3 -p1
%patch4 -p1
%patch5 -p1
+%patch6 -p2
%patch8 -p1
diff --git a/messageid.patch b/messageid.patch
new file mode 100644
index 0000000..78c0631
--- /dev/null
+++ b/messageid.patch
@@ -0,0 +1,155 @@
+From 889894461aa958da4604299acc161c29e2aa603c Mon Sep 17 00:00:00 2001
+From: Jeremy Harris <jgh146exb at wizmail.org>
+Date: Sun, 5 Sep 2021 11:32:09 +0100
+Subject: [PATCH] Fix validation of domain-literals in Message_ID: headers.
+ Bug 2805
+
+---
+ src/src/parse.c | 34 +++++++++++++++++++++-------------
+ src/src/receive.c | 3 ++-
+ 5 files changed, 43 insertions(+), 20 deletions(-)
+
+diff --git a/src/src/parse.c b/src/src/parse.c
+index 58f8941..42f1234 100644
+--- a/src/src/parse.c
++++ b/src/src/parse.c
+@@ -224,16 +224,20 @@ If allow_domain_literals is TRUE, a "domain" may also be an IP address enclosed
+ in []. Make sure the output is set to the null string if there is a syntax
+ error as well as if there is no domain at all.
+
++Optionally, msg_id domain literals ( printable-ascii enclosed in [] )
++are permitted.
++
+ Arguments:
+ s current character pointer
+ t where to put the domain
++ msg_id_literals flag for relaxed domain-literal processing
+ errorptr put error message here on failure (*t will be 0 on exit)
+
+ Returns: new character pointer
+ */
+
+ static const uschar *
+-read_domain(const uschar *s, uschar *t, uschar **errorptr)
++read_domain(const uschar *s, uschar *t, BOOL msg_id_literals, uschar **errorptr)
+ {
+ uschar *tt = t;
+ s = skip_comment(s);
+@@ -259,7 +263,11 @@ if (*s == '[')
+ t += 5;
+ s += 5;
+ }
+- while (*s == '.' || *s == ':' || isxdigit(*s)) *t++ = *s++;
++
++ if (msg_id_literals)
++ while (*s >= 33 && *s <= 90 || *s >= 94 && *s <= 126) *t++ = *s++;
++ else
++ while (*s == '.' || *s == ':' || isxdigit(*s)) *t++ = *s++;
+
+ if (*s == ']') *t++ = *s++; else
+ {
+@@ -267,7 +275,7 @@ if (*s == '[')
+ *tt = 0;
+ }
+
+- if (!allow_domain_literals)
++ if (!allow_domain_literals && !msg_id_literals)
+ {
+ *errorptr = US"domain literals not allowed";
+ *tt = 0;
+@@ -500,7 +508,7 @@ BOOL commas = FALSE;
+ while (*s == '@')
+ {
+ *t++ = '@';
+- s = read_domain(s+1, t, errorptr);
++ s = read_domain(s+1, t, FALSE, errorptr);
+ if (*t == 0) return s;
+ t += Ustrlen((const uschar *)t);
+ if (*s != ',') break;
+@@ -559,7 +567,7 @@ if (*errorptr == NULL)
+ t += Ustrlen((const uschar *)t);
+ *t++ = *s++;
+ *domainptr = t;
+- s = read_domain(s, t, errorptr);
++ s = read_domain(s, t, FALSE, errorptr);
+ }
+ return s;
+ }
+@@ -649,7 +657,7 @@ if (*s != '@' && *s != '<')
+ {
+ if (*s == 0 || *s == ';')
+ {
+- if (*t == 0) FAILED(US"empty address");
++ if (!*t) FAILED(US"empty address");
+ endptr = last_comment_position;
+ goto PARSE_SUCCEEDED; /* Bare local part */
+ }
+@@ -740,7 +748,7 @@ if (*s == '<')
+ }
+
+ endptr = s;
+- if (*errorptr != NULL) goto PARSE_FAILED;
++ if (*errorptr) goto PARSE_FAILED;
+ while (bracket_count-- > 0) if (*s++ != '>')
+ {
+ *errorptr = s[-1] == 0
+@@ -759,14 +767,14 @@ should be the domain. However, for flexibility we allow for a route-address
+ not enclosed in <> as well, which is indicated by an empty first local
+ part preceding '@'. The source routing is, however, ignored. */
+
+-else if (*t == 0)
++else if (!*t)
+ {
+ uschar *domainptr = yield;
+ s = read_route(s, t, errorptr);
+- if (*errorptr != NULL) goto PARSE_FAILED;
++ if (*errorptr) goto PARSE_FAILED;
+ *t = 0; /* Ensure route is ignored - probably overkill */
+ s = read_addr_spec(s, t, 0, errorptr, &domainptr);
+- if (*errorptr != NULL) goto PARSE_FAILED;
++ if (*errorptr) goto PARSE_FAILED;
+ *domain = domainptr - yield;
+ endptr = last_comment_position;
+ if (*domain == 0) FAILED(US"domain missing in source-routed address");
+@@ -779,8 +787,8 @@ else
+ t += Ustrlen((const uschar *)t);
+ *t++ = *s++;
+ *domain = t - yield;
+- s = read_domain(s, t, errorptr);
+- if (*t == 0) goto PARSE_FAILED;
++ s = read_domain(s, t, TRUE, errorptr);
++ if (!*t) goto PARSE_FAILED;
+ endptr = last_comment_position;
+ }
+
+@@ -789,7 +797,7 @@ through for other cases. Endptr may have been moved over whitespace, so
+ move it back past white space if necessary. */
+
+ PARSE_SUCCEEDED:
+-if (*s != 0)
++if (*s)
+ {
+ if (f.parse_found_group && *s == ';')
+ {
+diff --git a/src/src/receive.c b/src/src/receive.c
+index c2b313c..5471aa7 100644
+--- a/src/src/receive.c
++++ b/src/src/receive.c
+@@ -1663,7 +1663,6 @@ int process_info_len = Ustrlen(process_info);
+ int error_rc = error_handling == ERRORS_SENDER
+ ? errors_sender_rc : EXIT_FAILURE;
+ int header_size = 256;
+-int start, end, domain;
+ int id_resolution = 0;
+ int had_zero = 0;
+ int prevlines_length = 0;
+@@ -4084,6 +4083,8 @@ if ( LOGGING(msg_id) && msgid_header
+ uschar * old_id;
+ BOOL save_allow_domain_literals = allow_domain_literals;
+ allow_domain_literals = TRUE;
++ int start, end, domain;
++
+ old_id = parse_extract_address(Ustrchr(msgid_header->text, ':') + 1,
+ &errmsg, &start, &end, &domain, FALSE);
+ allow_domain_literals = save_allow_domain_literals;
+
================================================================
---- gitweb:
http://git.pld-linux.org/gitweb.cgi/packages/exim.git/commitdiff/16a62dd6f421d9ec2274d9d75793c7d4163b0459
More information about the pld-cvs-commit
mailing list