net/mlx4: fix minor resource leak during init
authorAdrien Mazarguil <adrien.mazarguil@6wind.com>
Tue, 22 May 2018 15:36:59 +0000 (17:36 +0200)
committerShahaf Shuler <shahafs@mellanox.com>
Sun, 17 Jun 2018 08:04:48 +0000 (10:04 +0200)
commit66f2ac0f8b857d880185fa32ba7229181be4d536
treea3cd7c91b0840040d41ea6923409633138283230
parentaf87183b82da32b2ca832a51acdb5a2a6117c718
net/mlx4: fix minor resource leak during init

Temporary IB device context and list are not freed in case of a successful
initialization of the device.

This issue is caused by the two following commits, the first of which
causes initialization to return early, while the second one goes a bit
overboard while switching to negative errno values; an internal variable
(err) is needed to tell success from failure at the end of the function
since rte_errno is not reliable enough.

Fixes: f2318196c71a ("net/mlx4: remove limitation on number of instances")
Fixes: 9d14b27308a0 ("net/mlx4: standardize on negative errno values")
Cc: stable@dpdk.org
Signed-off-by: Adrien Mazarguil <adrien.mazarguil@6wind.com>
drivers/net/mlx4/mlx4.c