staging/lustre/lnet: Fix allocation size for sv_cpt_data
authorOleg Drokin <green@linuxhacker.ru>
Sun, 19 Feb 2017 21:35:59 +0000 (16:35 -0500)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sun, 12 Mar 2017 05:41:42 +0000 (06:41 +0100)
commit dc7ffefdcc28a45214aa707fdc3df6a5e611ba09 upstream.

This is unbreaking another of those "stealth" janitor
patches that got in and subtly broke some things.

sv_cpt_data is a pointer to pointer, so need to
dereference it twice to allocate the correct structure size.

Fixes: 9899cb68c6c2 ("Staging: lustre: rpc: Use sizeof type *pointer instead of sizeof type.")
CC: Sandhya Bankar <bankarsandhya512@gmail.com>
Signed-off-by: Oleg Drokin <green@linuxhacker.ru>
Reviewed-by: James Simmons <jsimmons@infradead.org>
Reviewed-by: Doug Oucharek <doug.s.oucharek@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/lustre/lnet/selftest/rpc.c

index f5619d8..0256d65 100644 (file)
@@ -252,7 +252,7 @@ srpc_service_init(struct srpc_service *svc)
        svc->sv_shuttingdown = 0;
 
        svc->sv_cpt_data = cfs_percpt_alloc(lnet_cpt_table(),
-                                           sizeof(*svc->sv_cpt_data));
+                                           sizeof(**svc->sv_cpt_data));
        if (!svc->sv_cpt_data)
                return -ENOMEM;