On 5/14/2018 6:01 AM, Andy Green wrote:
> Signed-off-by: Andy Green <a...@warmcat.com>
> Fixes: f714a18885a6 ("app/testbbdev: add test application for bbdev")
> Cc: sta...@dpdk.org
> ---
> app/test-bbdev/test_bbdev_vector.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/app/test-bbdev/test_bbdev_vector.c
> b/app/test-bbdev/test_bbdev_vector.c
> index a37e35f4d..c574f2135 100644
> --- a/app/test-bbdev/test_bbdev_vector.c
> +++ b/app/test-bbdev/test_bbdev_vector.c
> @@ -892,7 +892,7 @@ test_bbdev_vector_read(const char *filename,
> }
>
> memset(entry, 0, strlen(line) + 1);
> - strncpy(entry, line, strlen(line));
> + strcpy(entry, line);
agreed that according above code strcpy is OK but still why change it? Is this
fixing a build error?
>
> /* check if entry ends with , or = */
> if (entry[strlen(entry) - 1] == ','
>
