diff options
author | Brian Norris <computersforpeace@gmail.com> | 2015-11-11 16:47:52 -0800 |
---|---|---|
committer | Brian Norris <computersforpeace@gmail.com> | 2015-11-12 10:10:01 -0800 |
commit | 938672338991ac8bef5cb025b187ab8d6230ac6c (patch) | |
tree | bea42fb1f87a4ab9910d14c82156d42d87b61b28 | |
parent | 215a02fd30871d0d888d27a3154588b66f5dbec2 (diff) |
mtd: zero out mtd_partition struct before using it
It's easier to guarantee we've cleared out all unused fields with
memset() than by manually initializing each field.
Signed-off-by: Brian Norris <computersforpeace@gmail.com>
Reviewed-by: Boris Brezillon <boris.brezillon@free-electrons.com>
-rw-r--r-- | drivers/mtd/mtdpart.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/mtd/mtdpart.c b/drivers/mtd/mtdpart.c index f8ba153f63bf..46dfbf5629c3 100644 --- a/drivers/mtd/mtdpart.c +++ b/drivers/mtd/mtdpart.c @@ -596,11 +596,10 @@ int mtd_add_partition(struct mtd_info *master, const char *name, if (length <= 0) return -EINVAL; + memset(&part, 0, sizeof(part)); part.name = name; part.size = length; part.offset = offset; - part.mask_flags = 0; - part.ecclayout = NULL; new = allocate_partition(master, &part, -1, offset); if (IS_ERR(new)) |