diff mbox

[rdma-core,2/8] srp_daemon: Fix missing free_config for ibsrpdm

Message ID 1501099152-6344-3-git-send-email-jgunthorpe@obsidianresearch.com (mailing list archive)
State Accepted
Headers show

Commit Message

Jason Gunthorpe July 26, 2017, 7:59 p.m. UTC 
Doesn't really matter since this just exits the process right after.

Signed-off-by: Jason Gunthorpe <jgunthorpe@obsidianresearch.com>
---
 srp_daemon/srp_daemon.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Bart Van Assche July 26, 2017, 8:37 p.m. UTC | #1 
On Wed, 2017-07-26 at 13:59 -0600, Jason Gunthorpe wrote:
> Doesn't really matter since this just exits the process right after.

Reviewed-by: Bart Van Assche <bart.vanassche@wdc.com>--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/srp_daemon/srp_daemon.c b/srp_daemon/srp_daemon.c
index ec394b686d55ff..836e2e5d1487b5 100644
--- a/srp_daemon/srp_daemon.c
+++ b/srp_daemon/srp_daemon.c
@@ -2053,7 +2053,7 @@  static int ibsrpdm(int argc, char *argv[])
 	ret = set_conf_dev_and_port(umad_dev, config);
 	if (ret) {
 		pr_err("Failed to build config\n");
-		return 1;
+		goto out;
 	}
 
 	umad_init();
@@ -2076,7 +2076,7 @@  static int ibsrpdm(int argc, char *argv[])
 	free_res(res);
 umad_done:
 	umad_done();
-
+out:
 	free_config(config);
 
 	return ret;