summaryrefslogtreecommitdiff
path: root/src/android/mm
diff options
context:
space:
mode:
authorUmang Jain <umang.jain@ideasonboard.com>2021-08-18 14:08:41 +0530
committerUmang Jain <umang.jain@ideasonboard.com>2021-08-19 19:38:30 +0530
commitcdb70b5c4012e8bb87d4fe9008f466fab13ef062 (patch)
tree876da02ec87651bfac968a3cd021cc9dcd59662e /src/android/mm
parent31078711d6c3639073db97322c6f7d98dacbbefe (diff)
libcamera: ipc_unixsocket: Do not run memcpy with null arguments
In IPCUnixSocket, a payload can be sent/received with empty fd vector, which leads to passing a nullptr in memcpy() in both sendData() and recvData(). Add a null check for fd vector's data pointer to avoid invoking memcpy() with nullptr. The issue is noticed by running a test manually testing the vimc IPA code paths in isolated mode. It is only noticed when the test is compiled with -Db_sanitize=address,undefined meson built-in option. ipc_unixsocket.cpp:268:8: runtime error: null pointer passed as argument 2, which is declared to never be null ipc_unixsocket.cpp:312:8: runtime error: null pointer passed as argument 1, which is declared to never be null Signed-off-by: Umang Jain <umang.jain@ideasonboard.com> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Reviewed-by: Paul Elder <paul.elder@ideasonboard.com>
Diffstat (limited to 'src/android/mm')
0 files changed, 0 insertions, 0 deletions