From 338194be2530ef4ef844374dc1c5929b0d3a0892 Mon Sep 17 00:00:00 2001
From: David Garske <david@wolfssl.com>
Date: Wed, 3 May 2017 07:33:13 -0700
Subject: [PATCH] Fix for scan build warning for `TLSX_SNI_GetRequest` possible
 use of null pointer.

---
 src/tls.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/src/tls.c b/src/tls.c
index 5a144ee27..056ca0934 100755
--- 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);
+                }
         }
     }