On Sunday, July 16, 2017 4:29:52 PM PDT Lionel Landwerlin wrote: > v2: Use sizeof() (Chris) > > CID: 1415113 > Reported-by: Grazvydas Ignotas <nota...@gmail.com> > Signed-off-by: Lionel Landwerlin <lionel.g.landwer...@intel.com> > --- > src/intel/vulkan/anv_device.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/src/intel/vulkan/anv_device.c b/src/intel/vulkan/anv_device.c > index 34d4a675481..7e3eae43081 100644 > --- a/src/intel/vulkan/anv_device.c > +++ b/src/intel/vulkan/anv_device.c > @@ -891,8 +891,8 @@ void anv_GetPhysicalDeviceProperties( > .sparseProperties = {0}, /* Broadwell doesn't do sparse. */ > }; > > - strncpy(pProperties->deviceName, pdevice->name, > - VK_MAX_PHYSICAL_DEVICE_NAME_SIZE); > + snprintf(pProperties->deviceName, sizeof(pProperties->deviceName), > + "%s", pdevice->name); > memcpy(pProperties->pipelineCacheUUID, > pdevice->pipeline_cache_uuid, VK_UUID_SIZE); > } > -- > 2.13.2 > _______________________________________________ > mesa-dev mailing list > mesa-dev@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/mesa-dev >
I guess we haven't landed strlcpy wrappers yet, or we could use that. Reviewed-by: Kenneth Graunke <kenn...@whitecape.org>
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev