app/bbdev: fix null termination in argument parsing

use snprintf instead of strncpy to ensure null termination when
copying test_vector_filename from cmd arguments.

Coverity issue: 257001
Fixes: f714a18885a6 ("app/testbbdev: add test application for bbdev")

Signed-off-by: Amr Mokhtar <amr.mokhtar@intel.com>
This commit is contained in:
Amr Mokhtar 2018-01-31 12:40:40 +00:00 committed by Thomas Monjalon
parent a9bb0c44c7
commit 9efeea0a19

View File

@ -226,8 +226,9 @@ parse_args(int argc, char **argv, struct test_params *tp)
TEST_ASSERT(strlen(optarg) > 0,
"Config file name is null");
strncpy(tp->test_vector_filename, optarg,
sizeof(tp->test_vector_filename));
snprintf(tp->test_vector_filename,
sizeof(tp->test_vector_filename),
"%s", optarg);
break;
case 'l':
TEST_ASSERT(strlen(optarg) > 0,