coverity 177409: dont bother assigning ignored second SSL_shutdown return value
authorAndy Green <andy@warmcat.com>
Mon, 6 Mar 2017 06:01:46 +0000 (14:01 +0800)
committerAndy Green <andy@warmcat.com>
Mon, 6 Mar 2017 06:46:06 +0000 (14:46 +0800)
No security impact

lib/libwebsockets.c

index 3f9612b..9443589 100755 (executable)
@@ -458,7 +458,7 @@ just_kill_connection:
                        lwsl_info("%s: shutting down SSL connection: %p (ssl %p, sock %d, state %d)\n", __func__, wsi, wsi->ssl, (int)(long)wsi->desc.sockfd, wsi->state);
                        n = SSL_shutdown(wsi->ssl);
                        if (n == 0) /* Complete bidirectional SSL shutdown */
-                               n = SSL_shutdown(wsi->ssl);
+                               SSL_shutdown(wsi->ssl);
                        n = shutdown(wsi->desc.sockfd, SHUT_WR);
                }
                else