diff options
author | Gustavo A. R. Silva <gustavo@embeddedor.com> | 2019-06-10 16:06:13 -0500 |
---|---|---|
committer | Dan Williams <dan.j.williams@intel.com> | 2019-08-29 13:49:00 -0700 |
commit | 2b90cb223320a93b1be6c2616efe6f9ff14d8b28 (patch) | |
tree | 7b732e876a6233be6268797ae4db50f24ad71f18 /drivers/nvdimm/btt.h | |
parent | 5518ba4ebd7d1e15fc558af05fa600198535f074 (diff) | |
download | lwn-2b90cb223320a93b1be6c2616efe6f9ff14d8b28.tar.gz lwn-2b90cb223320a93b1be6c2616efe6f9ff14d8b28.zip |
libnvdimm, region: Use struct_size() in kzalloc()
One of the more common cases of allocation size calculations is finding
the size of a structure that has a zero-sized array at the end, along
with memory for some number of elements for that array. For example:
struct nd_region {
...
struct nd_mapping mapping[0];
};
instance = kzalloc(sizeof(struct nd_region) + sizeof(struct nd_mapping) *
count, GFP_KERNEL);
Instead of leaving these open-coded and prone to type mistakes, we can
now use the new struct_size() helper:
instance = kzalloc(struct_size(instance, mapping, count), GFP_KERNEL);
This code was detected with the help of Coccinelle.
Signed-off-by: Gustavo A. R. Silva <gustavo@embeddedor.com>
Reviewed-by: Vishal Verma <vishal.l.verma@intel.com>
Reviewed-by: Kees Cook <keescook@chromium.org>
Link: https://lore.kernel.org/r/20190610210613.GA21989@embeddedor
Signed-off-by: Dan Williams <dan.j.williams@intel.com>
Diffstat (limited to 'drivers/nvdimm/btt.h')
0 files changed, 0 insertions, 0 deletions