Commit f426faac authored by Stephen Kitt's avatar Stephen Kitt Committed by Herbert Xu

crypto: chelsio - remove extra allocation for chtls_dev

chtls_uld_add allocates room for info->nports net_device structs
following the chtls_dev struct, presumably because it was originally
intended that the ports array would be stored there. This is suggested
by the assignment which was present in initial versions and removed by
c4e84858 ("crypto: chelsio - remove redundant assignment to
cdev->ports"):

	cdev->ports = (struct net_device **)(cdev + 1);

This assignment was never used, being overwritten by lldi->ports
immediately afterwards, and I couldn't find any uses of the memory
allocated past the end of the struct.
Signed-off-by: default avatarStephen Kitt <steve@sk2.org>
Signed-off-by: default avatarHerbert Xu <herbert@gondor.apana.org.au>
parent 7f19380b
...@@ -229,8 +229,7 @@ static void *chtls_uld_add(const struct cxgb4_lld_info *info) ...@@ -229,8 +229,7 @@ static void *chtls_uld_add(const struct cxgb4_lld_info *info)
struct chtls_dev *cdev; struct chtls_dev *cdev;
int i, j; int i, j;
cdev = kzalloc(sizeof(*cdev) + info->nports * cdev = kzalloc(sizeof(*cdev), GFP_KERNEL);
(sizeof(struct net_device *)), GFP_KERNEL);
if (!cdev) if (!cdev)
goto out; goto out;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment