From 0fbd635caa86a027dbe88c4739dc00a27e29135e Mon Sep 17 00:00:00 2001 From: Artur Wojcik Date: Thu, 10 Dec 2009 12:03:37 -0700 Subject: [PATCH] Fix for possible NULL pointer dereference. Pointer 'this' returned from call to function 'malloc' at line 3795 may be NULL and will be dereferenced at line 3796. Artur Wojcik Signed-off-by: Dan Williams --- super-intel.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/super-intel.c b/super-intel.c index 2e119f83..5e8378f9 100644 --- a/super-intel.c +++ b/super-intel.c @@ -3804,6 +3804,11 @@ static struct mdinfo *container_content_imsm(struct supertype *st) } this = malloc(sizeof(*this)); + if (!this) { + fprintf(stderr, Name ": failed to allocate %lu bytes\n", + sizeof(*this)); + break; + } memset(this, 0, sizeof(*this)); this->next = rest; @@ -3821,7 +3826,7 @@ static struct mdinfo *container_content_imsm(struct supertype *st) ord = get_imsm_ord_tbl_ent(dev, slot); for (d = super->disks; d ; d = d->next) if (d->index == idx) - break; + break; if (d == NULL) skip = 1; -- 2.47.2