client/gatt: Fix scan-build warning
authorLuiz Augusto von Dentz <luiz.von.dentz@intel.com>
Fri, 23 Sep 2022 20:47:10 +0000 (13:47 -0700)
committerAyush Garg <ayush.garg@samsung.com>
Mon, 15 May 2023 09:25:55 +0000 (14:55 +0530)
This fixes the following warning:

client/gatt.c:2146:2: warning: Null pointer passed to 2nd parameter
expecting 'nonnull' [core.NonNullParamChecker]
        memcpy(*dst_value + offset, src_val, src_len);
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Signed-off-by: Manika Shrivastava <manika.sh@samsung.com>
Signed-off-by: Ayush Garg <ayush.garg@samsung.com>
client/gatt.c

index 0793ec9..c9525f2 100755 (executable)
@@ -2142,7 +2142,8 @@ static int write_value(size_t *dst_len, uint8_t **dst_value, uint8_t *src_val,
                *dst_value = g_realloc(*dst_value, *dst_len);
        }
 
-       memcpy(*dst_value + offset, src_val, src_len);
+       if (src_val && src_len)
+               memcpy(*dst_value + offset, src_val, src_len);
 
        return 0;
 }