OSDN Git Service

android/avrcp: Fix clang warning
authorAndrei Emeltchenko <andrei.emeltchenko@intel.com>
Mon, 31 Mar 2014 12:25:11 +0000 (15:25 +0300)
committerLuiz Augusto von Dentz <luiz.von.dentz@intel.com>
Wed, 2 Apr 2014 11:25:41 +0000 (14:25 +0300)
Check len before memcpy(). Fixes:
...
android/avrcp-lib.c:885:3: warning: Null pointer passed as an argument
to a 'nonnull' parameter
                memcpy(&ptr[4], text[i], len);
                ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
...

android/avrcp-lib.c

index ec5ffdb..c01ccbb 100644 (file)
@@ -1964,7 +1964,10 @@ int avrcp_get_player_attribute_text_rsp(struct avrcp *session,
                ptr[0] = attrs[i];
                put_be16(AVRCP_CHARSET_UTF8, &ptr[1]);
                ptr[3] = len;
-               memcpy(&ptr[4], text[i], len);
+
+               if (len)
+                       memcpy(&ptr[4], text[i], len);
+
                ptr += 4 + len;
                length += 4 + len;
        }