Commit 4c355005 authored Sep 12, 2012 by Mike Marciniszyn Committed by Roland Dreier Sep 14, 2012

IB/qib: Fix failure of compliance test C14-024#06_LocalPortNum



Commit 3236b2d4 ("IB/qib: MADs with misset M_Keys should return
failure") introduced a return code assignment that unfortunately
introduced an unconditional exit for the routine due to missing braces.

This patch adds the braces to correct the original patch.

Reviewed-by: Dean Luick <dean.luick@intel.com>
Signed-off-by: Mike Marciniszyn <mike.marciniszyn@intel.com>
Signed-off-by: Roland Dreier <roland@purestorage.com>

parent fea7a08a

drivers/infiniband/hw/qib/qib_mad.c

+2 −1

Original line number	Diff line number	Diff line
		@@ -471,11 +471,12 @@ static int subn_get_portinfo(struct ib_smp smp, struct ib_device ibdev,
		if (port_num != port) {
		ibp = to_iport(ibdev, port_num);
		ret = check_mkey(ibp, smp, 0);
		if (ret)
		if (ret) {
		ret = IB_MAD_RESULT_FAILURE;
		goto bail;
		}
		}
		}

		dd = dd_from_ibdev(ibdev);
		/* IB numbers ports from 1, hdw from 0 */