Merge pull request #890 from dgarske/fix_sb_sni

Fix for scan build warning for `TLSX_SNI_GetRequest` possible use of null ptr
2017-05-03 10:03:59 -07:00
parent 1dc5a0fba2 338194be25
commit 86f10086bb
1 changed files with 3 additions and 2 deletions
--- a/src/tls.c
+++ b/src/tls.c
@@ -1699,9 +1699,10 @@ word16 TLSX_SNI_GetRequest(TLSX* extensions, byte type, void** data)
    if (sni && sni->status != WOLFSSL_SNI_NO_MATCH) {
        switch (sni->type) {
            case WOLFSSL_SNI_HOST_NAME:
-                if (data)
+                if (data) {
                    *data = sni->data.host_name;
-                return (word16)XSTRLEN((char*)*data);
+                    return (word16)XSTRLEN((char*)*data);
+                }
        }
    }