Check should (indeed) be for GNUTLS_E_AGAIN.

Lasa Martxel wrote:
Hello,

I have found the following code in daemon.c file, lines 1449 to 1467:

      if (0 >= res)
        {
          if (GNUTLS_E_INTERRUPTED != res)
            {
              urh->app.celi &= ~MHD_EPOLL_STATE_WRITE_READY;

              if (GNUTLS_E_INTERRUPTED != res)

(GNUTLS_E_INTERRUPTED != res) is checked twice.

In the read part (a few lines above), GNUTLS_E_INTERRUPTED != res is checked first and then GNUTLS_E_AGAIN != res. It looks like something similar should be done here, but I’m not sure.

Thanks,

Martxel

1 files changed, 4 insertions, 0 deletions

diff --git a/ChangeLog b/ChangeLog
index 7a895911..1227571f 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@
+Wed Dec 19 00:06:03 CET 2018
+        Check for GNUTLS_E_AGAIN instead of GNUTLS_E_INTERRUPTED when
+        giving up on a TLS connection. -LM/CG
+
 Thu Dec 13 22:48:14 CET 2018
         Fix connection timeout logic if in thread-per-connection mode the
         working thread takes longer than the timeout to queue the response. -CG