Commit 26c1c601 authored by Oran Agra's avatar Oran Agra
Browse files

fix recently broken TLS build error, and add coverage for CI (#7833)


(cherry picked from commit a735bf5c)
parent 2ccbba65
...@@ -9,7 +9,8 @@ jobs: ...@@ -9,7 +9,8 @@ jobs:
steps: steps:
- uses: actions/checkout@v2 - uses: actions/checkout@v2
- name: make - name: make
run: make # build with TLS just for compilatoin coverage
run: make BUILD_TLS=yes
- name: test - name: test
run: | run: |
sudo apt-get install tcl8.5 sudo apt-get install tcl8.5
......
...@@ -454,7 +454,7 @@ void updateSSLEvent(tls_connection *conn) { ...@@ -454,7 +454,7 @@ void updateSSLEvent(tls_connection *conn) {
} }
static void tlsHandleEvent(tls_connection *conn, int mask) { static void tlsHandleEvent(tls_connection *conn, int mask) {
int ret; int ret, conn_error;
TLSCONN_DEBUG("tlsEventHandler(): fd=%d, state=%d, mask=%d, r=%d, w=%d, flags=%d", TLSCONN_DEBUG("tlsEventHandler(): fd=%d, state=%d, mask=%d, r=%d, w=%d, flags=%d",
fd, conn->c.state, mask, conn->c.read_handler != NULL, conn->c.write_handler != NULL, fd, conn->c.state, mask, conn->c.read_handler != NULL, conn->c.write_handler != NULL,
...@@ -464,7 +464,7 @@ static void tlsHandleEvent(tls_connection *conn, int mask) { ...@@ -464,7 +464,7 @@ static void tlsHandleEvent(tls_connection *conn, int mask) {
switch (conn->c.state) { switch (conn->c.state) {
case CONN_STATE_CONNECTING: case CONN_STATE_CONNECTING:
int conn_error = connGetSocketError((connection *) conn); conn_error = connGetSocketError((connection *) conn);
if (conn_error) { if (conn_error) {
conn->c.last_errno = conn_error; conn->c.last_errno = conn_error;
conn->c.state = CONN_STATE_ERROR; conn->c.state = CONN_STATE_ERROR;
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment