From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from m12-11.163.com (m12-11.163.com [220.181.12.11]) by dpdk.org (Postfix) with ESMTP id 61DC11B964 for ; Sat, 22 Dec 2018 13:41:26 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=163.com; s=s110527; h=From:Subject:Date:Message-Id; bh=GBSPWpYn+HihgdjcIz 7x6RcXp9+UfNJgsDBodbTLva8=; b=OcTnF5c5/Tw9id7dB2UPFCHPXHaJNO/lEL tKL7O8/RNEx2Eq643StDqTJUyq4jTxfFAQ8hlcfE3xRxQEg7Dznoovwoqzoj3Rex dRo99b96wGBK+daXth7FdQvrdBDY4a1Dn4FvCcsiNfcdyBSIdjve9rTi1X2Fclpe J1aWhb1E4= Received: from localhost (unknown [14.17.22.40]) by smtp7 (Coremail) with SMTP id C8CowAAHAuXpMB5clTAsBQ--.61104S2; Sat, 22 Dec 2018 20:41:13 +0800 (CST) From: Haiyang Tan To: John Daley , Hyong Youb Kim Cc: dev@dpdk.org, Haiyang Tan Date: Sat, 22 Dec 2018 04:41:03 -0800 Message-Id: <20181222124103.116759-1-haiyangtan@tencent.com> X-Mailer: git-send-email 2.14.1 X-CM-TRANSID: C8CowAAHAuXpMB5clTAsBQ--.61104S2 X-Coremail-Antispam: 1Uf129KBjvJXoW7Cr4fZrWfZw18Kry3JF4kCrg_yoW8Xw13pa 1fWw1UKw4DJa1Y9rsrZan7G3Z8u3WIyFyUGF92ywnYkr1UAr1rJFyDCFyIyr1kKFW3ZF17 XFW2vr18W3Z5ZrJanT9S1TB71UUUUUUqnTZGkaVYY2UrUUUUjbIjqfuFe4nvWSU5nxnvy2 9KBjDUYxBIdaVFxhVjvjDU0xZFpf9x07jwManUUUUU= X-Originating-IP: [14.17.22.40] Sender: thy295@163.com X-CM-SenderInfo: xwk1jmqv6rljoofrz/xtbBEAklP1UMJ-Iu-gAAsf Subject: [dpdk-dev] [PATCH] net/enic: fix possible uninitialized variable X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 22 Dec 2018 12:41:51 -0000 The uninitialized field 'extra_flag' of hash parameter may enable certain feature silently. Typically, if bit0 of 'extra_flag' set, the hardware transactional memory support will be enabled unexpectedly. Signed-off-by: Haiyang Tan --- drivers/net/enic/enic_clsf.c | 17 +++++++++-------- 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/drivers/net/enic/enic_clsf.c b/drivers/net/enic/enic_clsf.c index 9d95201ec..f9707c78f 100644 --- a/drivers/net/enic/enic_clsf.c +++ b/drivers/net/enic/enic_clsf.c @@ -475,14 +475,15 @@ void enic_clsf_destroy(struct enic *enic) int enic_clsf_init(struct enic *enic) { char clsf_name[RTE_HASH_NAMESIZE]; - struct rte_hash_parameters hash_params = { - .name = clsf_name, - .entries = ENICPMD_CLSF_HASH_ENTRIES, - .key_len = sizeof(struct rte_eth_fdir_filter), - .hash_func = DEFAULT_HASH_FUNC, - .hash_func_init_val = 0, - .socket_id = SOCKET_ID_ANY, - }; + struct rte_hash_parameters hash_params = { 0 }; + + hash_params.name = clsf_name; + hash_params.entries = ENICPMD_CLSF_HASH_ENTRIES; + hash_params.key_len = sizeof(struct rte_eth_fdir_filter); + hash_params.hash_func = DEFAULT_HASH_FUNC; + hash_params.hash_func_init_val = 0; + hash_params.socket_id = SOCKET_ID_ANY; + snprintf(clsf_name, RTE_HASH_NAMESIZE, "enic_clsf_%s", enic->bdf_name); enic->fdir.hash = rte_hash_create(&hash_params); memset(&enic->fdir.stats, 0, sizeof(enic->fdir.stats)); -- 2.14.1