From f5ca9b91046f43410934ddf872bf4504cd9e9fe1 Mon Sep 17 00:00:00 2001 From: Nikolay Sivov Date: Fri, 8 Apr 2016 16:56:40 +0300 Subject: [PATCH] comctl32/listview: Handle DPA_SetPtr() failure (Coverity). Signed-off-by: Nikolay Sivov Signed-off-by: Alexandre Julliard --- dlls/comctl32/listview.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/dlls/comctl32/listview.c b/dlls/comctl32/listview.c index b467b362364..758b64016ba 100644 --- a/dlls/comctl32/listview.c +++ b/dlls/comctl32/listview.c @@ -3123,7 +3123,11 @@ static RANGES ranges_clone(RANGES ranges) RANGE *newrng = Alloc(sizeof(RANGE)); if (!newrng) goto fail; *newrng = *((RANGE*)DPA_GetPtr(ranges->hdpa, i)); - DPA_SetPtr(clone->hdpa, i, newrng); + if (!DPA_SetPtr(clone->hdpa, i, newrng)) + { + Free(newrng); + goto fail; + } } return clone;