From: Jeremy Harris Date: Mon, 18 Sep 2017 16:47:52 +0000 (+0100) Subject: SOCKS: fix unitialized-pointer X-Git-Tag: exim-4_90_RC1~65 X-Git-Url: https://git.exim.org/exim.git/commitdiff_plain/848214f788cb561935b8200d6e795df2de1cb8fa?hp=89dd53dfbf0166d98032ad1f72428ceda6baf52a SOCKS: fix unitialized-pointer --- diff --git a/doc/doc-txt/ChangeLog b/doc/doc-txt/ChangeLog index e777a2fb2..8f1b6b7ad 100644 --- a/doc/doc-txt/ChangeLog +++ b/doc/doc-txt/ChangeLog @@ -163,6 +163,9 @@ JH/26 Fix DKIM bug: when the pseudoheader generated for signing was exactly HS/02 Fix Bug 2130: large writes from the transport subprocess where chunked and confused the parent. +JH/27 Fix SOCKS bug: an unitialized pointer was deref'd by the transport process + which could crash as a result. This could lead to undeliverable messages. + Exim version 4.89 ----------------- diff --git a/src/src/transports/smtp_socks.c b/src/src/transports/smtp_socks.c index 555843068..92d13659d 100644 --- a/src/src/transports/smtp_socks.c +++ b/src/src/transports/smtp_socks.c @@ -285,7 +285,7 @@ for(;;) sob = &proxies[idx]; /* bodge up a host struct for the proxy */ - proxy.address = sob->proxy_host; + proxy.address = proxy.name = sob->proxy_host; proxy_af = Ustrchr(sob->proxy_host, ':') ? AF_INET6 : AF_INET; if ((fd = smtp_sock_connect(&proxy, proxy_af, sob->port,