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


test/handshake_helper.c:140:1: start of procedure client_hello_select_server_ctx()
138.   }
139.   
140. > static int client_hello_select_server_ctx(SSL *s, void *arg, int ignore)
141.   {
142.       const char *servername;

test/handshake_helper.c:145:5: 
143.       const unsigned char *p;
144.       size_t len, remaining;
145. >     HANDSHAKE_EX_DATA *ex_data =
146.           (HANDSHAKE_EX_DATA*)(SSL_get_ex_data(s, ex_data_idx));
147.   

ssl/ssl_lib.c:3908:1: start of procedure SSL_get_ex_data()
  3906.   }
  3907.   
  3908. > void *SSL_get_ex_data(const SSL *s, int idx)
  3909.   {
  3910.       return CRYPTO_get_ex_data(&s->ex_data, idx);

ssl/ssl_lib.c:3910:5: 
  3908.   void *SSL_get_ex_data(const SSL *s, int idx)
  3909.   {
  3910. >     return CRYPTO_get_ex_data(&s->ex_data, idx);
  3911.   }
  3912.   

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:3911:1: return from a call to SSL_get_ex_data
  3909.   {
  3910.       return CRYPTO_get_ex_data(&s->ex_data, idx);
  3911. > }
  3912.   
  3913.   int SSL_CTX_set_ex_data(SSL_CTX *s, int idx, void *arg)

test/handshake_helper.c:152:10: Taking false branch
150.        * was written, so parsing the normal case is a bit complex.
151.        */
152.       if (!SSL_client_hello_get0_ext(s, TLSEXT_TYPE_server_name, &p,
                ^
153.                                      &remaining) ||
154.           remaining <= 2)

test/handshake_helper.c:154:9: Taking false branch
152.       if (!SSL_client_hello_get0_ext(s, TLSEXT_TYPE_server_name, &p,
153.                                      &remaining) ||
154.           remaining <= 2)
               ^
155.           return 0;
156.       /* Extract the length of the supplied list of names. */

test/handshake_helper.c:157:5: 
155.           return 0;
156.       /* Extract the length of the supplied list of names. */
157. >     len = (*(p++) << 8);
158.       len += *(p++);
159.       if (len + 2 != remaining)

test/handshake_helper.c:158:5: 
156.       /* Extract the length of the supplied list of names. */
157.       len = (*(p++) << 8);
158. >     len += *(p++);
159.       if (len + 2 != remaining)
160.           return 0;

test/handshake_helper.c:159:9: Taking false branch
157.       len = (*(p++) << 8);
158.       len += *(p++);
159.       if (len + 2 != remaining)
               ^
160.           return 0;
161.       remaining = len;

test/handshake_helper.c:161:5: 
159.       if (len + 2 != remaining)
160.           return 0;
161. >     remaining = len;
162.       /*
163.        * The list in practice only has a single element, so we only consider

test/handshake_helper.c:166:9: Taking false branch
164.        * the first one.
165.        */
166.       if (remaining == 0 || *p++ != TLSEXT_NAMETYPE_host_name)
               ^
167.           return 0;
168.       remaining--;

test/handshake_helper.c:166:27: Taking false branch
164.        * the first one.
165.        */
166.       if (remaining == 0 || *p++ != TLSEXT_NAMETYPE_host_name)
                                 ^
167.           return 0;
168.       remaining--;

test/handshake_helper.c:168:5: 
166.       if (remaining == 0 || *p++ != TLSEXT_NAMETYPE_host_name)
167.           return 0;
168. >     remaining--;
169.       /* Now we can finally pull out the byte array with the actual hostname. */
170.       if (remaining <= 2)

test/handshake_helper.c:170:9: Taking false branch
168.       remaining--;
169.       /* Now we can finally pull out the byte array with the actual hostname. */
170.       if (remaining <= 2)
               ^
171.           return 0;
172.       len = (*(p++) << 8);

test/handshake_helper.c:172:5: 
170.       if (remaining <= 2)
171.           return 0;
172. >     len = (*(p++) << 8);
173.       len += *(p++);
174.       if (len + 2 > remaining)

test/handshake_helper.c:173:5: 
171.           return 0;
172.       len = (*(p++) << 8);
173. >     len += *(p++);
174.       if (len + 2 > remaining)
175.           return 0;

test/handshake_helper.c:174:9: Taking false branch
172.       len = (*(p++) << 8);
173.       len += *(p++);
174.       if (len + 2 > remaining)
               ^
175.           return 0;
176.       remaining = len;

test/handshake_helper.c:176:5: 
174.       if (len + 2 > remaining)
175.           return 0;
176. >     remaining = len;
177.       servername = (const char *)p;
178.   

test/handshake_helper.c:177:5: 
175.           return 0;
176.       remaining = len;
177. >     servername = (const char *)p;
178.   
179.       if (len == strlen("server2") && strncmp(servername, "server2", len) == 0) {

test/handshake_helper.c:179:9: Taking false branch
177.       servername = (const char *)p;
178.   
179.       if (len == strlen("server2") && strncmp(servername, "server2", len) == 0) {
               ^
180.           SSL_CTX *new_ctx = arg;
181.           SSL_set_SSL_CTX(s, new_ctx);

test/handshake_helper.c:192:16: Taking true branch
190.           ex_data->servername = SSL_TEST_SERVERNAME_SERVER2;
191.           return 1;
192.       } else if (len == strlen("server1") &&
                      ^
193.                  strncmp(servername, "server1", len) == 0) {
194.           ex_data->servername = SSL_TEST_SERVERNAME_SERVER1;

test/handshake_helper.c:193:16: Taking true branch
191.           return 1;
192.       } else if (len == strlen("server1") &&
193.                  strncmp(servername, "server1", len) == 0) {
                      ^
194.           ex_data->servername = SSL_TEST_SERVERNAME_SERVER1;
195.           return 1;

test/handshake_helper.c:194:9: 
192.       } else if (len == strlen("server1") &&
193.                  strncmp(servername, "server1", len) == 0) {
194. >         ex_data->servername = SSL_TEST_SERVERNAME_SERVER1;
195.           return 1;
196.       } else if (ignore) {

static int client_hello_select_server_ctx(SSL *s, void *arg, int ignore)
{
    const char *servername;
    const unsigned char *p;
    size_t len, remaining;
    HANDSHAKE_EX_DATA *ex_data =
        (HANDSHAKE_EX_DATA*)(SSL_get_ex_data(s, ex_data_idx));
    if (!SSL_client_hello_get0_ext(s, TLSEXT_TYPE_server_name, &p,
                                   &remaining) ||
        remaining <= 2)
        return 0;
    len = (*(p++) << 8);
    len += *(p++);
    if (len + 2 != remaining)
        return 0;
    remaining = len;
    if (remaining == 0 || *p++ != TLSEXT_NAMETYPE_host_name)
        return 0;
    remaining--;
    if (remaining <= 2)
        return 0;
    len = (*(p++) << 8);
    len += *(p++);
    if (len + 2 > remaining)
        return 0;
    remaining = len;
    servername = (const char *)p;
    if (len == strlen("server2") && strncmp(servername, "server2", len) == 0) {
        SSL_CTX *new_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 1;
    } else if (len == strlen("server1") &&
               strncmp(servername, "server1", len) == 0) {
        ex_data->servername = SSL_TEST_SERVERNAME_SERVER1;
        return 1;
    } else if (ignore) {
        ex_data->servername = SSL_TEST_SERVERNAME_SERVER1;
        return 1;
    }
    return 0;
}
['static int client_hello_select_server_ctx(SSL *s, void *arg, int ignore)\n{\n    const char *servername;\n    const unsigned char *p;\n    size_t len, remaining;\n    HANDSHAKE_EX_DATA *ex_data =\n        (HANDSHAKE_EX_DATA*)(SSL_get_ex_data(s, ex_data_idx));\n    if (!SSL_client_hello_get0_ext(s, TLSEXT_TYPE_server_name, &p,\n                                   &remaining) ||\n        remaining <= 2)\n        return 0;\n    len = (*(p++) << 8);\n    len += *(p++);\n    if (len + 2 != remaining)\n        return 0;\n    remaining = len;\n    if (remaining == 0 || *p++ != TLSEXT_NAMETYPE_host_name)\n        return 0;\n    remaining--;\n    if (remaining <= 2)\n        return 0;\n    len = (*(p++) << 8);\n    len += *(p++);\n    if (len + 2 > remaining)\n        return 0;\n    remaining = len;\n    servername = (const char *)p;\n    if (len == strlen("server2") && strncmp(servername, "server2", len) == 0) {\n        SSL_CTX *new_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 1;\n    } else if (len == strlen("server1") &&\n               strncmp(servername, "server1", len) == 0) {\n        ex_data->servername = SSL_TEST_SERVERNAME_SERVER1;\n        return 1;\n    } else if (ignore) {\n        ex_data->servername = SSL_TEST_SERVERNAME_SERVER1;\n        return 1;\n    }\n    return 0;\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}']