Commit Briefs

7847bd6699 Martijn van Duren

Domains are case insensitive.

Keep that in mind when comparing i- and d-tag

diff | patch | tree

022782a165 Martijn van Duren

osmtpd_ltok_skip_sig_h_tag_value can be further out than end, because a header can contain a ';'

diff | patch | tree

04225ff918 Martijn van Duren

FWS after h=-tag value could lead to a end mismatch.

Use osmtpd_ltok_skip_sig_h_tag_value for checking and original code for copying data

diff | patch | tree

09150ec450 Martijn van Duren

osmtpd_ltok_skip_key_h_tag_value should continue working on ptr, not prev

diff | patch | tree

2b49c69507 Martijn van Duren

Use osmtpd_ltok_skip_sig_i_tag_value() instead of parsing it manually where not needed

diff | patch | tree

088e9e4957 Martijn van Duren

Start the osmtpd_ltok_skip_sig_i_tag_value domain-name scan after '@'

diff | patch | tree

2817edcae0 Martijn van Duren

osmtpd_ltok_skip_domain has obs-domain, which gives the wrong result.

Use the appropriate osmtpd_ltok_skip_sig_d_tag_value

diff | patch | tree

bf67701d74 Martijn van Duren

signature pkey also needs to be freed after we're done with it.

diff | patch | tree

8986823b9e Martijn van Duren

Move ar_data free to the end of the fuction, so that failed queries also get released

diff | patch | tree

93003f16d4 Martijn van Duren

Free ar_data, as suggested by asr_run(3)

diff | patch | tree

749b2bf780 Martijn van Duren

semicolon is part of qp-section/safe-char. Ignore it if we overflow

diff | patch | tree

8cc3b72a5a Martijn van Duren

header->buf also needs to be freed

diff | patch | tree

5d58b443f9 Martijn van Duren

Add DKIM_UNKNOWN check for whitelines backlog as well

diff | patch | tree

42479031f4 Martijn van Duren

Rewrite osmtpd_ltok_skip_hyphenated_word. It didn't work quite right.

While here remove unneeded #include <stdio.h>

diff | patch | tree

efee726e37 Martijn van Duren

Fix calculating subdomain matching for i tag

diff | patch | tree

f85da7301b Martijn van Duren

Fix type-O

diff | patch | tree

31ffb91656 Martijn van Duren

rewrite dkim_body_parse, so that we don't copy the line.

This means that we don't limit the line length anymore, but might cost a little extra computing overhead. Let's see how bad this can get.

diff | patch | tree

6eeac6a109 Martijn van Duren

And of course I tested with the wrong file.

v=-tags should also increment key, similar to the other tags.

diff | patch | tree

2e9f7a5ec6 Martijn van Duren

Fix syntax validator in t=-tag (used wrong function)

While here fix unfold some line wrappings no longer needed

diff | patch | tree

475eae6523 Martijn van Duren

For the key h tag we should skip over the alg, not the entire value when

inside the loop. While here reorder some line folding since the mheader->ltok API name change.

diff | patch | tree

2b1224a85b Martijn van Duren

Put v=-tag handling for key into switch.

Shorter and better legibility

diff | patch | tree

7e24437dc3 Martijn van Duren

Always compile test failure issue 1000000

diff | patch | tree

d842ff09fd Martijn van Duren

Add more error checking in key parsing

diff | patch | tree

bccb8fb35c Martijn van Duren

Shrink unknown key tag parsing, similar to what happened in unknown sig tag

diff | patch | tree

4e2dd90f10 Martijn van Duren

Rework the entire mheader file

- Rename mheader to ltok (lexical token) - Make all skip functions const - Add a few missing functions

diff | patch | tree