Commit 633b3895 authored by Alessandro Ghedini's avatar Alessandro Ghedini Committed by Daniel Stenberg
Browse files

polarssl: fix ALPN protocol negotiation 

Correctly check for strncmp() return value (it returns 0 if the strings
match).
parent b723ec99

lib/vtls/polarssl.c

+2 −2
Original line number Diff line number Diff line
@@ -468,11 +468,11 @@ polarssl_connect_step2(struct connectdata *conn, 
    if(next_protocol != NULL) {

      infof(data, "ALPN, server accepted to use %s\n", next_protocol);



      if(strncmp(next_protocol, NGHTTP2_PROTO_VERSION_ID,

      if(!strncmp(next_protocol, NGHTTP2_PROTO_VERSION_ID,

                  NGHTTP2_PROTO_VERSION_ID_LEN)) {

        conn->negnpn = NPN_HTTP2;

      }

      else if(strncmp(next_protocol, ALPN_HTTP_1_1, ALPN_HTTP_1_1_LENGTH)) {

      else if(!strncmp(next_protocol, ALPN_HTTP_1_1, ALPN_HTTP_1_1_LENGTH)) {

        conn->negnpn = NPN_HTTP1_1;

      }

    }