Commit f65fadb0 authored by Lu Hongfei's avatar Lu Hongfei Committed by Steve French
Browse files

ksmbd: Replace the ternary conditional operator with min() 



It would be better to replace the traditional ternary conditional
operator with min() in compare_sids.

Signed-off-by: default avatarLu Hongfei <luhongfei@vivo.com>
Acked-by: default avatarNamjae Jeon <linkinjeon@kernel.org>
Signed-off-by: default avatarSteve French <stfrench@microsoft.com>
parent 81a94b27

fs/smb/server/smbacl.c

+1 −1
Original line number Diff line number Diff line
@@ -97,7 +97,7 @@ int compare_sids(const struct smb_sid *ctsid, const struct smb_sid *cwsid) 
	/* compare all of the subauth values if any */

	num_sat = ctsid->num_subauth;

	num_saw = cwsid->num_subauth;

	num_subauth = num_sat < num_saw ? num_sat : num_saw;

	num_subauth = min(num_sat, num_saw);

	if (num_subauth) {

		for (i = 0; i < num_subauth; ++i) {

			if (ctsid->sub_auth[i] != cwsid->sub_auth[i]) {