Commit d9135820 authored by Daniel Verkamp's avatar Daniel Verkamp Committed by Jim Harris
Browse files

blob: copy xattr name with memcpy() 



We know exactly how long the name is, so there is no need to use a C
string function to copy it.

Change-Id: I21b5f1e318555b46729582ab6a1e6bd163c85205
Signed-off-by: default avatarDaniel Verkamp <daniel.verkamp@intel.com>
Reviewed-on: https://review.gerrithub.io/406984


Tested-by: default avatarSPDK Automated Test System <sys_sgsw@intel.com>
Reviewed-by: default avatarBen Walker <benjamin.walker@intel.com>
Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
parent ec7ef532

lib/blob/blobstore.c

+1 −1
Original line number Diff line number Diff line
@@ -290,7 +290,7 @@ _spdk_blob_deserialize_xattr(struct spdk_blob *blob, 
		free(xattr);

		return -ENOMEM;

	}

	strncpy(xattr->name, desc_xattr->name, desc_xattr->name_length);

	memcpy(xattr->name, desc_xattr->name, desc_xattr->name_length);

	xattr->name[desc_xattr->name_length] = '\0';



	xattr->value = malloc(desc_xattr->value_length);