Your message dated Sat, 02 Jul 2016 19:46:23 -0400
with message-id <2339603.cyzuTPKQDK@kitterma-e6430>
and subject line Re: opendkim "final" script does not work
has caused the Debian Bug report #770778,
regarding opendkim "final" script does not work
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact [email protected]
immediately.)


-- 
770778: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=770778
Debian Bug Tracking System
Contact [email protected] with problems
--- Begin Message ---
Package: opendkim
Version: 2.0.1+dfsg-1

opendkim to asserts an error if a FinalPolicyScript is set.

In "opendkim.c" line 5038 there is this:

         if (dkimf_lua_final_hook(NULL, NULL,
                                  conf->conf_finalscript,
                                  &lres) != 0)

I believe those parameters are incorrect. The second NULL should not be there. It should probably be changed to something like the following which is similar to what the other hook calls use:

         if (dkimf_lua_final_hook(NULL,
                                  conf->conf_finalscript,
                                  str, &lres) != 0)

Debian squeeze-lts

--- End Message ---
--- Begin Message ---
On Sun, 23 Nov 2014 19:19:36 -0500 Chris <[email protected]> wrote:
> Package: opendkim
> Version: 2.0.1+dfsg-1
> 
> opendkim to asserts an error if a FinalPolicyScript is set.
> 
> In "opendkim.c" line 5038 there is this:
> 
>           if (dkimf_lua_final_hook(NULL, NULL,
>                                    conf->conf_finalscript,
>                                    &lres) != 0)
> 
> I believe those parameters are incorrect. The second NULL should not be 
> there. It should probably be changed to something like the following 
> which is similar to what the other hook calls use:
> 
>           if (dkimf_lua_final_hook(NULL,
>                                    conf->conf_finalscript,
>                                    str, &lres) != 0)
> 
> Debian squeeze-lts

I checked the code in testing and this has been changed.

Scott K

--- End Message ---

Reply via email to