diff mbox series

[rdma-next] RDMA/odp: Add missing cast for 32 bit

Message ID 20190908080726.30017-1-leon@kernel.org (mailing list archive)
State Mainlined
Commit b97b218b302d7ccc82381d3c8254396804e7d819
Delegated to: Jason Gunthorpe
Headers show
Series [rdma-next] RDMA/odp: Add missing cast for 32 bit | expand

Commit Message

Leon Romanovsky Sept. 8, 2019, 8:07 a.m. UTC
From: Jason Gunthorpe <jgg@mellanox.com>

length is a size_t which is unsigned int on 32 bit:

../drivers/infiniband/core/umem_odp.c: In function 'ib_init_umem_odp':
../include/linux/overflow.h:59:15: warning: comparison of distinct pointer types lacks a cast
   59 |  (void) (&__a == &__b);   \
      |               ^~
../drivers/infiniband/core/umem_odp.c:220:7: note: in expansion of macro 'check_add_overflow'

Fixes: 204e3e5630c5 ("RDMA/odp: Check for overflow when computing the umem_odp end")
Signed-off-by: Jason Gunthorpe <jgg@mellanox.com>
Signed-off-by: Leon Romanovsky <leonro@mellanox.com>
---
 drivers/infiniband/core/umem_odp.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Jason Gunthorpe Sept. 12, 2019, 1:39 p.m. UTC | #1
On Sun, Sep 08, 2019 at 11:07:26AM +0300, Leon Romanovsky wrote:
> From: Jason Gunthorpe <jgg@mellanox.com>
> 
> length is a size_t which is unsigned int on 32 bit:
> 
> ../drivers/infiniband/core/umem_odp.c: In function 'ib_init_umem_odp':
> ../include/linux/overflow.h:59:15: warning: comparison of distinct pointer types lacks a cast
>    59 |  (void) (&__a == &__b);   \
>       |               ^~
> ../drivers/infiniband/core/umem_odp.c:220:7: note: in expansion of macro 'check_add_overflow'
> 
> Fixes: 204e3e5630c5 ("RDMA/odp: Check for overflow when computing the umem_odp end")
> Signed-off-by: Jason Gunthorpe <jgg@mellanox.com>
> Signed-off-by: Leon Romanovsky <leonro@mellanox.com>
> ---
>  drivers/infiniband/core/umem_odp.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Applied to for-next

Jason
diff mbox series

Patch

diff --git a/drivers/infiniband/core/umem_odp.c b/drivers/infiniband/core/umem_odp.c
index 32fb0b579dec..bd01c77aeef0 100644
--- a/drivers/infiniband/core/umem_odp.c
+++ b/drivers/infiniband/core/umem_odp.c
@@ -293,7 +293,7 @@  static inline int ib_init_umem_odp(struct ib_umem_odp *umem_odp,
 		umem_odp->interval_tree.start =
 			ALIGN_DOWN(umem_odp->umem.address, page_size);
 		if (check_add_overflow(umem_odp->umem.address,
-				       umem_odp->umem.length,
+				       (unsigned long)umem_odp->umem.length,
 				       &umem_odp->interval_tree.last))
 			return -EOVERFLOW;
 		umem_odp->interval_tree.last =