From 4099785e2d60377459845b8fa886ab6be1a1d8e4 Mon Sep 17 00:00:00 2001 From: Vijayakumar Date: Thu, 5 Jan 2017 16:20:51 +0530 Subject: gpu: nvgpu: free boardobj if construct hits an err During construct of some VFE/CLK boardobjs, some data is filled after a boardobj allocation is done. Free up boardobj memory if an error is encountered in the data filling. Coverity ID 490171 Coverity ID 490172 Bug 200192125 Change-Id: I20621f7f9f9e379b8dced4905cd417c2ffa905b0 Signed-off-by: Vijayakumar Reviewed-on: http://git-master/r/1280700 GVS: Gerrit_Virtual_Submit Reviewed-by: Terje Bergstrom Tested-by: Terje Bergstrom --- drivers/gpu/nvgpu/perf/vfe_var.c | 13 ++++++++++--- 1 file changed, 10 insertions(+), 3 deletions(-) (limited to 'drivers/gpu/nvgpu/perf') diff --git a/drivers/gpu/nvgpu/perf/vfe_var.c b/drivers/gpu/nvgpu/perf/vfe_var.c index f4c3f276..f43e68e2 100644 --- a/drivers/gpu/nvgpu/perf/vfe_var.c +++ b/drivers/gpu/nvgpu/perf/vfe_var.c @@ -1,5 +1,5 @@ /* - * Copyright (c) 2016, NVIDIA CORPORATION. All rights reserved. + * Copyright (c) 2017, NVIDIA CORPORATION. All rights reserved. * * This program is free software; you can redistribute it and/or modify it * under the terms and conditions of the GNU General Public License, @@ -668,12 +668,19 @@ static u32 vfe_var_construct_single_sensed_fuse(struct gk20a *g, if (pvfevar->vfield_info.fuse.segment_count == 0) { gk20a_err(dev_from_gk20a(g), "unable to get fuse reg info %x", pvfevar->vfield_info.v_field_id); - return -EINVAL; + status = -EINVAL; + goto exit; } if (pvfevar->vfield_ver_info.fuse.segment_count == 0) { gk20a_err(dev_from_gk20a(g), "unable to get fuse reg info %x", pvfevar->vfield_ver_info.v_field_id_ver); - return -EINVAL; + status = -EINVAL; + goto exit; + } +exit: + if (status) { + if (*ppboardobj != NULL) + (*ppboardobj)->destruct(*ppboardobj); } return status; } -- cgit v1.2.2