On Thu, 8 Sep 2005, Tony Finch wrote:
> When putting together the patch I noticed something odd in
> rf_get_errors_address:
>
> DEBUG(D_route|D_verify)
> debug_printf("------ Verifying errors address %s ------\n", s);
> if (verify_address(snew, NULL, vopt_is_recipient | vopt_qualify, -1, -1, -1,
> NULL, NULL, NULL) == OK) *errors_to = snew->address;
> DEBUG(D_route|D_verify)
> debug_printf("------ End verifying errors address %s ------\n", s);
>
> Shouldn't the errors_to address be verified as a sender address?
Why? It's where you are going to send error messages. Oh, I see, you are
thinking of it as a replacement sender, which in a way it is. I dunno.
It's been that way for a long time and of course I can't remember if
there is/was a reason. There might have been, and then again I might
just have set all verifies to recipients when I introduced the
distinction, and not changed that one.
The only thing to do is to change it, and see if any tests break...
--
Philip Hazel University of Cambridge Computing Service,
ph10@??? Cambridge, England. Phone: +44 1223 334714.
Get the Exim 4 book: http://www.uit.co.uk/exim-book
