diff options
author | Jeremy Harris <jgh146exb@wizmail.org> | 2019-05-09 14:10:12 +0100 |
---|---|---|
committer | Jeremy Harris <jgh146exb@wizmail.org> | 2019-05-09 14:10:12 +0100 |
commit | 345f272404cb1c454b7602214c76fd0ebf04d66e (patch) | |
tree | a757697dd8e0d60ce44036e01e465338fa966df2 /src | |
parent | eb2163cbc29a00b2ca214e507e4a57f374c2e8ed (diff) |
Avoid potential crash in close of a verify callout
Diffstat (limited to 'src')
-rw-r--r-- | src/src/verify.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/src/src/verify.c b/src/src/verify.c index 720f85d8c..184809fb5 100644 --- a/src/src/verify.c +++ b/src/src/verify.c @@ -1103,12 +1103,9 @@ no_conn: if (options & vopt_callout_recipsender) cancel_cutthrough_connection(TRUE, US"not usable for cutthrough"); if (sx.send_quit) - { - (void) smtp_write_command(&sx, SCMD_FLUSH, "QUIT\r\n"); - - /* Wait a short time for response, and discard it */ - smtp_read_response(&sx, sx.buffer, sizeof(sx.buffer), '2', 1); - } + if (smtp_write_command(&sx, SCMD_FLUSH, "QUIT\r\n") != -1) + /* Wait a short time for response, and discard it */ + smtp_read_response(&sx, sx.buffer, sizeof(sx.buffer), '2', 1); if (sx.cctx.sock >= 0) { |