RE: [PATCH rdma-rc] RDMA/ucma: Ensure that CM_ID exists prior to access it

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> From: Leon Romanovsky <leonro@xxxxxxxxxxxx>
> 
> Prior to access UCMA commands, the context should be initialized and
> connected to CM_ID with ucma_create_id(). In case user skips this
> step, he can provide non-valid ctx without CM_ID and cause to multiple
> NULL dereferences.

I think the issue is that there's a window between the creation of a new ucma_context and it being associated with an rdma_cm_id.  A caller could exploit this window to cause a NULL pointer dereference.

The ctx itself would be valid, but the cm_id still not set.

The use of an invalid ctx should be trapped already.

The change itself looks reasonable.

- Sean
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]     [Yosemite Photos]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux