From: Dave Jones <davej@redhat.com>
Date: Fri, 31 Jan 2014 02:17:30 +0000 (-0500)
Subject: drm/radeon/dpm: fix uninitialized read from stack in kv_dpm_late_enable
X-Git-Url: http://drtracing.org/?a=commitdiff_plain;h=7c7e867cf0f74c4c79649605b964510041f1a8bf;p=deliverable%2Flinux.git

drm/radeon/dpm: fix uninitialized read from stack in kv_dpm_late_enable

If we take the false branch of the if quoted in the diff below, we
end up doing a return ret, without ever having initialized it.

Picked up by coverity.

Signed-off-by: Dave Jones <davej@fedoraproject.org>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
---

diff --git a/drivers/gpu/drm/radeon/kv_dpm.c b/drivers/gpu/drm/radeon/kv_dpm.c
index b6e01d5d2cce..351db361239d 100644
--- a/drivers/gpu/drm/radeon/kv_dpm.c
+++ b/drivers/gpu/drm/radeon/kv_dpm.c
@@ -1223,7 +1223,7 @@ int kv_dpm_enable(struct radeon_device *rdev)
 
 int kv_dpm_late_enable(struct radeon_device *rdev)
 {
-	int ret;
+	int ret = 0;
 
 	if (rdev->irq.installed &&
 	    r600_is_internal_thermal_sensor(rdev->pm.int_thermal_type)) {