From patchwork Wed Nov 2 03:24:31 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Ajit Khaparde X-Patchwork-Id: 16891 Return-Path: X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [IPv6:::1]) by dpdk.org (Postfix) with ESMTP id E87C02BEB; Wed, 2 Nov 2016 04:24:35 +0100 (CET) Received: from mail-gw1-out.broadcom.com (mail-gw1-out.broadcom.com [216.31.210.62]) by dpdk.org (Postfix) with ESMTP id 4A35A2716 for ; Wed, 2 Nov 2016 04:24:33 +0100 (CET) X-IronPort-AV: E=Sophos;i="5.31,582,1473145200"; d="scan'208";a="109171422" Received: from mail-irv-17.broadcom.com ([10.15.198.34]) by mail-gw1-out.broadcom.com with ESMTP; 01 Nov 2016 22:48:38 -0700 Received: from C02PT1RBG8WP.vpn.broadcom.net (unknown [10.10.118.229]) by mail-irv-17.broadcom.com (Postfix) with ESMTP id 72E2E9D4A6; Tue, 1 Nov 2016 20:24:32 -0700 (PDT) Date: Tue, 1 Nov 2016 22:24:31 -0500 From: Ajit Khaparde To: dev@dpdk.org Message-ID: <20161102032431.GA69905@C02PT1RBG8WP.vpn.broadcom.net> MIME-Version: 1.0 Content-Disposition: inline User-Agent: Mutt/1.5.23 (2014-03-12) Subject: [dpdk-dev] [PATCH] bnxt: use unsigned short instead of signed integer in bnxt_alloc_vnic_attributes X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Prevent the arithmetic in bnxt_alloc_vnic_attributes from causing any unintentional havoc because of the usage of a signed variable. Coverity: 137874 Signed-off-by: Ajit Khaparde --- drivers/net/bnxt/bnxt_vnic.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/bnxt/bnxt_vnic.c b/drivers/net/bnxt/bnxt_vnic.c index 205a940..4cdca35 100644 --- a/drivers/net/bnxt/bnxt_vnic.c +++ b/drivers/net/bnxt/bnxt_vnic.c @@ -179,7 +179,7 @@ int bnxt_alloc_vnic_attributes(struct bnxt *bp) HW_HASH_INDEX_SIZE * sizeof(*vnic->rss_table) + HW_HASH_KEY_SIZE); uint16_t max_vnics; - int i; + uint16_t i; if (BNXT_PF(bp)) { struct bnxt_pf_info *pf = &bp->pf;