test/handshake_helper.c:138: error: NULL_DEREFERENCE
  pointer `ex_data` last assigned on line 113 could be null and is dereferenced at line 138, column 9.
Showing all 22 steps of the trace


test/handshake_helper.c:110:1: start of procedure select_server_ctx()
108.    * An empty SNI extension also returns SSL_TSLEXT_ERR_NOACK.
109.    */
110. > static int select_server_ctx(SSL *s, void *arg, int ignore)
111.   {
112.       const char *servername = SSL_get_servername(s, TLSEXT_NAMETYPE_host_name);

test/handshake_helper.c:112:5: 
110.   static int select_server_ctx(SSL *s, void *arg, int ignore)
111.   {
112. >     const char *servername = SSL_get_servername(s, TLSEXT_NAMETYPE_host_name);
113.       HANDSHAKE_EX_DATA *ex_data =
114.           (HANDSHAKE_EX_DATA*)(SSL_get_ex_data(s, ex_data_idx));

ssl/ssl_lib.c:2598:1: start of procedure SSL_get_servername()
  2596.    */
  2597.   
  2598. > const char *SSL_get_servername(const SSL *s, const int type)
  2599.   {
  2600.       if (type != TLSEXT_NAMETYPE_host_name)

ssl/ssl_lib.c:2600:9: Taking false branch
  2598.   const char *SSL_get_servername(const SSL *s, const int type)
  2599.   {
  2600.       if (type != TLSEXT_NAMETYPE_host_name)
                  ^
  2601.           return NULL;
  2602.   

ssl/ssl_lib.c:2603:12: Condition is true
  2601.           return NULL;
  2602.   
  2603.       return s->session && !s->ext.hostname ?
                     ^
  2604.           s->session->ext.hostname : s->ext.hostname;
  2605.   }

ssl/ssl_lib.c:2603:27: Condition is true
  2601.           return NULL;
  2602.   
  2603.       return s->session && !s->ext.hostname ?
                                    ^
  2604.           s->session->ext.hostname : s->ext.hostname;
  2605.   }

ssl/ssl_lib.c:2603:12: 
  2601.           return NULL;
  2602.   
  2603. >     return s->session && !s->ext.hostname ?
  2604.           s->session->ext.hostname : s->ext.hostname;
  2605.   }

ssl/ssl_lib.c:2603:5: 
  2601.           return NULL;
  2602.   
  2603. >     return s->session && !s->ext.hostname ?
  2604.           s->session->ext.hostname : s->ext.hostname;
  2605.   }

ssl/ssl_lib.c:2605:1: return from a call to SSL_get_servername
  2603.       return s->session && !s->ext.hostname ?
  2604.           s->session->ext.hostname : s->ext.hostname;
  2605. > }
  2606.   
  2607.   int SSL_get_servername_type(const SSL *s)

test/handshake_helper.c:113:5: 
111.   {
112.       const char *servername = SSL_get_servername(s, TLSEXT_NAMETYPE_host_name);
113. >     HANDSHAKE_EX_DATA *ex_data =
114.           (HANDSHAKE_EX_DATA*)(SSL_get_ex_data(s, ex_data_idx));
115.   

ssl/ssl_lib.c:4045:1: start of procedure SSL_get_ex_data()
  4043.   }
  4044.   
  4045. > void *SSL_get_ex_data(const SSL *s, int idx)
  4046.   {
  4047.       return CRYPTO_get_ex_data(&s->ex_data, idx);

ssl/ssl_lib.c:4047:5: 
  4045.   void *SSL_get_ex_data(const SSL *s, int idx)
  4046.   {
  4047. >     return CRYPTO_get_ex_data(&s->ex_data, idx);
  4048.   }
  4049.   

crypto/ex_data.c:393:1: start of procedure CRYPTO_get_ex_data()
    391.    * particular index in the class used by this variable
    392.    */
    393. > void *CRYPTO_get_ex_data(const CRYPTO_EX_DATA *ad, int idx)
    394.   {
    395.       if (ad->sk == NULL || idx >= sk_void_num(ad->sk))

crypto/ex_data.c:395:9: Taking true branch
    393.   void *CRYPTO_get_ex_data(const CRYPTO_EX_DATA *ad, int idx)
    394.   {
    395.       if (ad->sk == NULL || idx >= sk_void_num(ad->sk))
                   ^
    396.           return NULL;
    397.       return sk_void_value(ad->sk, idx);

crypto/ex_data.c:396:9: 
    394.   {
    395.       if (ad->sk == NULL || idx >= sk_void_num(ad->sk))
    396. >         return NULL;
    397.       return sk_void_value(ad->sk, idx);
    398.   }

crypto/ex_data.c:398:1: return from a call to CRYPTO_get_ex_data
    396.           return NULL;
    397.       return sk_void_value(ad->sk, idx);
    398. > }

ssl/ssl_lib.c:4048:1: return from a call to SSL_get_ex_data
  4046.   {
  4047.       return CRYPTO_get_ex_data(&s->ex_data, idx);
  4048. > }
  4049.   
  4050.   int SSL_CTX_set_ex_data(SSL_CTX *s, int idx, void *arg)

test/handshake_helper.c:116:9: Taking false branch
114.           (HANDSHAKE_EX_DATA*)(SSL_get_ex_data(s, ex_data_idx));
115.   
116.       if (servername == NULL) {
               ^
117.           ex_data->servername = SSL_TEST_SERVERNAME_SERVER1;
118.           return SSL_TLSEXT_ERR_NOACK;

test/handshake_helper.c:121:9: Taking false branch
119.       }
120.   
121.       if (strcmp(servername, "server2") == 0) {
               ^
122.           SSL_CTX *new_ctx = (SSL_CTX*)arg;
123.           SSL_set_SSL_CTX(s, new_ctx);

test/handshake_helper.c:134:16: Taking false branch
132.           ex_data->servername = SSL_TEST_SERVERNAME_SERVER2;
133.           return SSL_TLSEXT_ERR_OK;
134.       } else if (strcmp(servername, "server1") == 0) {
                      ^
135.           ex_data->servername = SSL_TEST_SERVERNAME_SERVER1;
136.           return SSL_TLSEXT_ERR_OK;

test/handshake_helper.c:137:16: Taking true branch
135.           ex_data->servername = SSL_TEST_SERVERNAME_SERVER1;
136.           return SSL_TLSEXT_ERR_OK;
137.       } else if (ignore) {
                      ^
138.           ex_data->servername = SSL_TEST_SERVERNAME_SERVER1;
139.           return SSL_TLSEXT_ERR_NOACK;

test/handshake_helper.c:138:9: 
136.           return SSL_TLSEXT_ERR_OK;
137.       } else if (ignore) {
138. >         ex_data->servername = SSL_TEST_SERVERNAME_SERVER1;
139.           return SSL_TLSEXT_ERR_NOACK;
140.       } else {

static int select_server_ctx(SSL *s, void *arg, int ignore)
{
    const char *servername = SSL_get_servername(s, TLSEXT_NAMETYPE_host_name);
    HANDSHAKE_EX_DATA *ex_data =
        (HANDSHAKE_EX_DATA*)(SSL_get_ex_data(s, ex_data_idx));
    if (servername == NULL) {
        ex_data->servername = SSL_TEST_SERVERNAME_SERVER1;
        return SSL_TLSEXT_ERR_NOACK;
    }
    if (strcmp(servername, "server2") == 0) {
        SSL_CTX *new_ctx = (SSL_CTX*)arg;
        SSL_set_SSL_CTX(s, new_ctx);
        SSL_clear_options(s, 0xFFFFFFFFL);
        SSL_set_options(s, SSL_CTX_get_options(new_ctx));
        ex_data->servername = SSL_TEST_SERVERNAME_SERVER2;
        return SSL_TLSEXT_ERR_OK;
    } else if (strcmp(servername, "server1") == 0) {
        ex_data->servername = SSL_TEST_SERVERNAME_SERVER1;
        return SSL_TLSEXT_ERR_OK;
    } else if (ignore) {
        ex_data->servername = SSL_TEST_SERVERNAME_SERVER1;
        return SSL_TLSEXT_ERR_NOACK;
    } else {
        return SSL_TLSEXT_ERR_ALERT_FATAL;
    }
}
['static int select_server_ctx(SSL *s, void *arg, int ignore)\n{\n    const char *servername = SSL_get_servername(s, TLSEXT_NAMETYPE_host_name);\n    HANDSHAKE_EX_DATA *ex_data =\n        (HANDSHAKE_EX_DATA*)(SSL_get_ex_data(s, ex_data_idx));\n    if (servername == NULL) {\n        ex_data->servername = SSL_TEST_SERVERNAME_SERVER1;\n        return SSL_TLSEXT_ERR_NOACK;\n    }\n    if (strcmp(servername, "server2") == 0) {\n        SSL_CTX *new_ctx = (SSL_CTX*)arg;\n        SSL_set_SSL_CTX(s, new_ctx);\n        SSL_clear_options(s, 0xFFFFFFFFL);\n        SSL_set_options(s, SSL_CTX_get_options(new_ctx));\n        ex_data->servername = SSL_TEST_SERVERNAME_SERVER2;\n        return SSL_TLSEXT_ERR_OK;\n    } else if (strcmp(servername, "server1") == 0) {\n        ex_data->servername = SSL_TEST_SERVERNAME_SERVER1;\n        return SSL_TLSEXT_ERR_OK;\n    } else if (ignore) {\n        ex_data->servername = SSL_TEST_SERVERNAME_SERVER1;\n        return SSL_TLSEXT_ERR_NOACK;\n    } else {\n        return SSL_TLSEXT_ERR_ALERT_FATAL;\n    }\n}', 'const char *SSL_get_servername(const SSL *s, const int type)\n{\n    if (type != TLSEXT_NAMETYPE_host_name)\n        return NULL;\n    return s->session && !s->ext.hostname ?\n        s->session->ext.hostname : s->ext.hostname;\n}', 'void *SSL_get_ex_data(const SSL *s, int idx)\n{\n    return CRYPTO_get_ex_data(&s->ex_data, idx);\n}', 'void *CRYPTO_get_ex_data(const CRYPTO_EX_DATA *ad, int idx)\n{\n    if (ad->sk == NULL || idx >= sk_void_num(ad->sk))\n        return NULL;\n    return sk_void_value(ad->sk, idx);\n}']