From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ferruh Yigit Subject: Re: [PATCH] app/testbbdev: fix sprintf with snprintf Date: Fri, 8 Feb 2019 14:13:59 +0000 Message-ID: <0eaf883b-58b4-aeba-a929-8dd5cb81f5c2@intel.com> References: <1549264554-2501-1-git-send-email-pallantlax.poornima@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Cc: reshma.pattan@intel.com, amr.mokhtar@intel.com, stable@dpdk.org To: Pallantla Poornima , dev@dpdk.org Return-path: In-Reply-To: <1549264554-2501-1-git-send-email-pallantlax.poornima@intel.com> Content-Language: en-US List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On 2/4/2019 7:15 AM, Pallantla Poornima wrote: > @@ -825,7 +825,7 @@ > > test_bbdev_driver_init(void) > > TEST_ASSERT_FAIL(rte_bbdev_release(NULL), > "Failed to uninitialize bbdev driver with NULL bbdev"); > - sprintf(name_tmp, "%s", "invalid_name"); > + snprintf(name_tmp, sizeof(name_tmp), "%s", "invalid_name"); > dev2 = rte_bbdev_get_named_dev(name_tmp); > TEST_ASSERT_FAIL(rte_bbdev_release(dev2), > "Failed to uninitialize bbdev driver with invalid name"); Can use strlcpy() instead of snprintf() here.