From: Gian-Carlo DeFazio <defazio1@llnl.gov>
Date: Thu, 8 May 2025 17:12:02 +0000 (-0700)
Subject: LU-18988 kfilnd: add return value after LBUGs
X-Git-Tag: 2.16.56~40
X-Git-Url: https://git.whamcloud.com/?a=commitdiff_plain;h=9558f28a46b4d7a5010e38571c3e6479d559fab9;p=fs%2Flustre-release.git

LU-18988 kfilnd: add return value after LBUGs

To get rid of compiler warnings of the kind
"control reaches end of non-void function",
add return statements after LBUGs.

Signed-off-by: Gian-Carlo DeFazio <defazio1@llnl.gov>
Change-Id: Ie8596c7527fa68f88e807ca290157b4a4dc891cf
Reviewed-on: https://review.whamcloud.com/c/fs/lustre-release/+/59157
Reviewed-by: Chris Horn <chris.horn@hpe.com>
Reviewed-by: Serguei Smirnov <ssmirnov@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
---

diff --git a/lnet/klnds/kfilnd/kfilnd_tn.c b/lnet/klnds/kfilnd/kfilnd_tn.c
index 758c9d9..f22820c 100644
--- a/lnet/klnds/kfilnd/kfilnd_tn.c
+++ b/lnet/klnds/kfilnd/kfilnd_tn.c
@@ -567,12 +567,14 @@ static int kfilnd_tn_state_send_failed(struct kfilnd_transaction *tn,
 					"Unexpected error during cancel tagged receive: rc=%d",
 					rc);
 			LBUG();
+			return -EINVAL;
 		}
 		break;
 
 	default:
 		KFILND_TN_ERROR(tn, "Invalid %s event", tn_event_to_str(event));
 		LBUG();
+		return -EINVAL;
 	}
 }
 
@@ -633,6 +635,7 @@ static int kfilnd_tn_state_tagged_recv_posted(struct kfilnd_transaction *tn,
 	default:
 		KFILND_TN_ERROR(tn, "Invalid %s event", tn_event_to_str(event));
 		LBUG();
+		return -EINVAL;
 	}
 }