[dpdk-dev] i40e: fix unintended sign extension
Commit Message
Coverity issue reported like
CID 119268 (#1 of 1): Unintended sign extension
(SIGN_EXTENSION)sign_extension: Suspicious implicit sign extension:
vsi_id with type unsigned short (16 bits, unsigned) is promoted in
vsi_id << 23 to type int (32 bits, signed), then sign-extended to type
unsigned long (64 bits, unsigned). If vsi_id << 23 is greater than
0x7FFFFFFF, the upper bits of the result will all be 1.
Fixes: 88ebc2b7f976 ("i40e: extend flow director to support VF")
Signed-off-by: Jingjing Wu <jingjing.wu@intel.com>
---
drivers/net/i40e/i40e_fdir.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Comments
> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Jingjing Wu
> Sent: Tuesday, December 15, 2015 4:23 PM
> To: dev@dpdk.org
> Subject: [dpdk-dev] [PATCH] i40e: fix unintended sign extension
>
> Coverity issue reported like
> CID 119268 (#1 of 1): Unintended sign extension
> (SIGN_EXTENSION)sign_extension: Suspicious implicit sign extension:
> vsi_id with type unsigned short (16 bits, unsigned) is promoted in vsi_id
> << 23 to type int (32 bits, signed), then sign-extended to type unsigned
> long (64 bits, unsigned). If vsi_id << 23 is greater than 0x7FFFFFFF, the
> upper bits of the result will all be 1.
>
> Fixes: 88ebc2b7f976 ("i40e: extend flow director to support VF")
> Signed-off-by: Jingjing Wu <jingjing.wu@intel.com>
Acked-by: John McNamara <john.mcnamara@intel.com>
> Coverity issue reported like
> > CID 119268 (#1 of 1): Unintended sign extension
> > (SIGN_EXTENSION)sign_extension: Suspicious implicit sign extension:
> > vsi_id with type unsigned short (16 bits, unsigned) is promoted in vsi_id
> > << 23 to type int (32 bits, signed), then sign-extended to type unsigned
> > long (64 bits, unsigned). If vsi_id << 23 is greater than 0x7FFFFFFF, the
> > upper bits of the result will all be 1.
> >
> > Fixes: 88ebc2b7f976 ("i40e: extend flow director to support VF")
> > Signed-off-by: Jingjing Wu <jingjing.wu@intel.com>
>
> Acked-by: John McNamara <john.mcnamara@intel.com>
Applied, thanks
@@ -1091,7 +1091,7 @@ i40e_fdir_filter_programming(struct i40e_pf *pf,
/* Use LAN VSI Id by default */
vsi_id = pf->main_vsi->vsi_id;
fdirdp->qindex_flex_ptype_vsi |=
- rte_cpu_to_le_32((vsi_id <<
+ rte_cpu_to_le_32(((uint32_t)vsi_id <<
I40E_TXD_FLTR_QW0_DEST_VSI_SHIFT) &
I40E_TXD_FLTR_QW0_DEST_VSI_MASK);