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 <vsubbu@nvidia.com>
Reviewed-on: http://git-master/r/1280700
GVS: Gerrit_Virtual_Submit
Reviewed-by: Terje Bergstrom <tbergstrom@nvidia.com>
Tested-by: Terje Bergstrom <tbergstrom@nvidia.com>

1 files changed, 10 insertions, 3 deletions

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;
 }