sendout.c:_sendbundle_setup_creds(): fix false assert() (Jan Chaloupka)..
commitb1e97ae938f390a2b3d799ee45640aaf421cee75
authorSteffen (Daode) Nurpmeso <sdaoden@users.sf.net>
Fri, 1 Aug 2014 09:40:38 +0000 (1 11:40 +0200)
committerSteffen (Daode) Nurpmeso <sdaoden@users.sf.net>
Fri, 1 Aug 2014 09:42:54 +0000 (1 11:42 +0200)
tree40a07e71cd923bd510ff855aad1f97fc3c7c7f62
parent7b3ca095e7148705de77801a499d95c5d72c629e
sendout.c:_sendbundle_setup_creds(): fix false assert() (Jan Chaloupka)..

Again about the still messy state of myorigin()/myaddrs().
With the new URL and credential scheme i've already introduced the
temporary __sendout_ident bypass hack, but that only comes into
play if *smtp-hostname* is set.  If neither that nor *hostname* is
set, *from* is required to exist as a non-NULL entry.  I have
indeed identified this precondition, but used assert(3) to verify
it's true, which is of course completely stupid given that we talk
about something that comes in from the user!

So today Jan Chaloupka (jchaloup AT redhat DOT com) reported on
nail-devel@ that Heirloom mailx can be crashed by setting *smtp*
in combination with with *from* effectively set to a NULL string.
I first was optimistic, but it turns out S-nail can, too.
sendout.c