From b92c7a84cde64dfbb4e32911fafce64888d75e61 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 Lustre-change: https://review.whamcloud.com/28487 Lustre-commit: 0a80067c60043da428c473c331ed7b602c6e960b Change-Id: If11c574655425db5bbf21ba2264be8d83a7e8bf8 Signed-off-by: Colin Ian King Signed-off-by: James Simmons Reviewed-by: Sonia Sharma Reviewed-by: Olaf Weber Signed-off-by: Minh Diep Reviewed-on: https://review.whamcloud.com/28763 Tested-by: Jenkins Tested-by: Maloo Reviewed-by: John L. Hammond --- 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 f9e4c0d..17a95ce 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