diff options
| author | Deepak Nibade <dnibade@nvidia.com> | 2017-06-05 05:05:07 -0400 |
|---|---|---|
| committer | mobile promotions <svcmobile_promotions@nvidia.com> | 2017-06-06 11:13:43 -0400 |
| commit | fe868356339182f0aa21a4424f5ef08229f8ebd6 (patch) | |
| tree | 2eb8f340e882adb278bc352ecac0af75120ab6af /drivers/gpu/nvgpu/therm/thrmdev.h | |
| parent | d4fcf90047fcf5f576801b8c1cfbca5b9ac44844 (diff) | |
gpu: nvgpu: fix resource leak
We pass (struct device_attribute *) to gp10b_ecc_stat_create()
and gr_gp10b_ecc_stat_create() and then assign a memory
allocation to this pointer
But since this pointer is local copy to function, static
pointer variables are never set in gr_gp10b_create_sysfs()
This also results in a resource leak since we never free
the storage assigned to local variable
Fix this by adding correct parameter (struct device_attribute **)
so that the address of the allocation is returned to the caller
correctly
Bug 200291879
Coverity id : 2567934
Change-Id: I7ac79810d9e22a568ae3cd1a69eb3f59a8d356ac
Signed-off-by: Deepak Nibade <dnibade@nvidia.com>
Reviewed-on: http://git-master/r/1495905
Reviewed-by: Automatic_Commit_Validation_User
GVS: Gerrit_Virtual_Submit
Reviewed-by: Sachin Nikam <snikam@nvidia.com>
Reviewed-by: svccoveritychecker <svccoveritychecker@nvidia.com>
Diffstat (limited to 'drivers/gpu/nvgpu/therm/thrmdev.h')
0 files changed, 0 insertions, 0 deletions