Control: found -1 0.6.75-1 On Sun, 2021-03-07 at 17:42 +0000, Surla, Sai Kalyan wrote:
> Already tried with version 0.6.75-1.
Thanks, marking the bug as found in that version.
> Also compiled the latest code available and tried with it, still the
> same results.
Thanks for testing this too.
> Please find the changes in the attached file. (readpst.c line no. : 1238)
It is traditional to provide changes in the patch format by using the
`diff -u` command or the corresponding commands from the version
control system that the upstream project is using.
Below is the output from the Mercurial diff for your change.
$ hg diff
diff -r 7200790e46ac src/readpst.c
--- a/src/readpst.c Tue Jun 16 17:18:28 2020 -0700
+++ b/src/readpst.c Mon Mar 08 09:20:50 2021 +0800
@@ -1235,7 +1235,7 @@
int header_match(char *header, char*field) {
int n = strlen(field);
- if (strncasecmp(header, field, n) == 0) return 1; // tag:{space}
+ if (strstr(header,field) != NULL || strncasecmp(header, field, n) == 0)
return 1; // tag:{space}
if ((field[n-1] == ' ') && (strncasecmp(header, field, n-1) == 0)) {
char *crlftab = "\r\n\t";
DEBUG_INFO(("Possible wrapped header = %s\n", header));
I am fairly certain that this is not the correct fix for this issue.
> ARC headers are kind of email authentication headers.
Thanks for the info.
> For some security reasons we cannot share the original
Understood.
> if possible we will try to share the inhouse sample pst.
That would be necessary to be able to fix the issue.
> Meanwhile our observation is if the headers start with the following
> headers (...) it is treated as bogus, this email is starting with
> some header which is not one of the listed.
That does look like what the code does indeed, probably the right fix
is to scan through all of the headers instead of just the first one.
--
bye,
pabs
https://wiki.debian.org/PaulWise
signature.asc
Description: This is a digitally signed message part
