diff options
| author |   Jason Gunthorpe <jgunthorpe@obsidianresearch.com> | 2017-03-17 12:48:15 -0600 |
|---|---|---|
| committer |   Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2017-03-21 06:44:33 +0100 |
| commit | a0d78193dcf2bfa7c3d169c55a7a30a5a89fca7a (patch) | |
| tree | 1b2c23c01c1a415cf4bad27b0e30d2e6cae3b074 /drivers/infiniband/core/user_mad.c | |
| parent | platform/chrome: cros_ec_dev - utilize new cdev_device_add helper function (diff) | |
| download | wireguard-linux-a0d78193dcf2bfa7c3d169c55a7a30a5a89fca7a.tar.xz wireguard-linux-a0d78193dcf2bfa7c3d169c55a7a30a5a89fca7a.zip | |
IB/ucm: utilize new cdev_device_add helper function
The use after free is not triggerable here because the cdev holds
the module lock and the only device_unregister is only triggered by
module unload, however make the change for consistency.
To make this work the cdev_del needs to move out of the struct device
release function.
This cleans up the error path significantly and thus also fixes a minor
bug where the devnum would not be released if cdev_add failed.
Signed-off-by: Jason Gunthorpe <jgunthorpe@obsidianresearch.com>
Signed-off-by: Logan Gunthorpe <logang@deltatee.com>
Reviewed-by: Logan Gunthorpe <logang@deltatee.com>
Reviewed-by: Leon Romanovsky <leonro@mellanox.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/infiniband/core/user_mad.c')
0 files changed, 0 insertions, 0 deletions