From 0a80067c60043da428c473c331ed7b602c6e960b Mon Sep 17 00:00:00 2001 From: Colin Ian King Date: Fri, 11 Aug 2017 13:17:57 -0400 Subject: [PATCH] LU-9869 lnet: fix incorrect arguments order calling lstcon_session_new The arguments args->lstio_ses_force and args->lstio_ses_timeout are in the incorrect order. Fix this by swapping them around. Detected by CoverityScan, CID#1226833 ("Arguments in wrong order") Test-Parameters: trivial testlist=lnet-selftest Change-Id: If11c574655425db5bbf21ba2264be8d83a7e8bf8 Signed-off-by: Colin Ian King Signed-off-by: James Simmons Reviewed-on: https://review.whamcloud.com/28487 Tested-by: Jenkins Tested-by: Maloo Reviewed-by: Sonia Sharma Reviewed-by: Olaf Weber Reviewed-by: Oleg Drokin --- lnet/selftest/conctl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lnet/selftest/conctl.c b/lnet/selftest/conctl.c index fafadc9..ef878b6 100644 --- a/lnet/selftest/conctl.c +++ b/lnet/selftest/conctl.c @@ -69,8 +69,8 @@ lst_session_new_ioctl(struct lstio_session_new_args *args) rc = lstcon_session_new(name, args->lstio_ses_key, args->lstio_ses_feats, - args->lstio_ses_force, args->lstio_ses_timeout, + args->lstio_ses_force, args->lstio_ses_idp); LIBCFS_FREE(name, args->lstio_ses_nmlen + 1); -- 1.8.3.1