NFSD: Use struct_size() helper in alloc_session()
authorXiu Jianfeng <xiujianfeng@huawei.com>
Fri, 11 Nov 2022 09:18:35 +0000 (17:18 +0800)
committerChuck Lever <cel@kernel.org>
Sat, 10 Dec 2022 16:01:11 +0000 (11:01 -0500)
Use struct_size() helper to simplify the code, no functional changes.

Signed-off-by: Xiu Jianfeng <xiujianfeng@huawei.com>
Reviewed-by: Jeff Layton <jlayton@kernel.org>
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
fs/nfsd/nfs4state.c

index 2ec981f..97badca 100644 (file)
@@ -1833,13 +1833,12 @@ static struct nfsd4_session *alloc_session(struct nfsd4_channel_attrs *fattrs,
        int numslots = fattrs->maxreqs;
        int slotsize = slot_bytes(fattrs);
        struct nfsd4_session *new;
-       int mem, i;
+       int i;
 
-       BUILD_BUG_ON(NFSD_MAX_SLOTS_PER_SESSION * sizeof(struct nfsd4_slot *)
-                       + sizeof(struct nfsd4_session) > PAGE_SIZE);
-       mem = numslots * sizeof(struct nfsd4_slot *);
+       BUILD_BUG_ON(struct_size(new, se_slots, NFSD_MAX_SLOTS_PER_SESSION)
+                    > PAGE_SIZE);
 
-       new = kzalloc(sizeof(*new) + mem, GFP_KERNEL);
+       new = kzalloc(struct_size(new, se_slots, numslots), GFP_KERNEL);
        if (!new)
                return NULL;
        /* allocate each struct nfsd4_slot and data cache in one piece */