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


test/handshake_helper.c:140:1: start of procedure early_select_server_ctx()
138.   }
139.   
140. > static int early_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:3889:1: start of procedure SSL_get_ex_data()
  3887.   }
  3888.   
  3889. > void *SSL_get_ex_data(const SSL *s, int idx)
  3890.   {
  3891.       return (CRYPTO_get_ex_data(&s->ex_data, idx));

ssl/ssl_lib.c:3891:5: 
  3889.   void *SSL_get_ex_data(const SSL *s, int idx)
  3890.   {
  3891. >     return (CRYPTO_get_ex_data(&s->ex_data, idx));
  3892.   }
  3893.   

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

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

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

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

ssl/ssl_lib.c:3892:1: return from a call to SSL_get_ex_data
  3890.   {
  3891.       return (CRYPTO_get_ex_data(&s->ex_data, idx));
  3892. > }
  3893.   
  3894.   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_early_get0_ext(s, TLSEXT_TYPE_server_name, &p, &remaining) ||
                ^
153.           remaining <= 2)
154.           return 0;

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

test/handshake_helper.c:195:16: Taking true branch
193.           ex_data->servername = SSL_TEST_SERVERNAME_SERVER1;
194.           return 1;
195.       } else if (ignore) {
                      ^
196.           ex_data->servername = SSL_TEST_SERVERNAME_SERVER1;
197.           return 1;

test/handshake_helper.c:196:9: 
194.           return 1;
195.       } else if (ignore) {
196. >         ex_data->servername = SSL_TEST_SERVERNAME_SERVER1;
197.           return 1;
198.       }

static int early_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_early_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 early_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_early_get0_ext(s, TLSEXT_TYPE_server_name, &p, &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}']