⬢ DragonFlyBSD Kernel Audit
DF-1149 / fix.diff
← back to finding ↓ download raw 
 1
 2
 3
 4
 5
 6
 7
 8
 9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
diff --git a/sys/dev/drm/radeon/ci_dpm.c b/sys/dev/drm/radeon/ci_dpm.c
--- a/sys/dev/drm/radeon/ci_dpm.c
+++ b/sys/dev/drm/radeon/ci_dpm.c
@@ -2687,7 +2687,8 @@
 	int ret = -EINVAL;
 
 	table->UvdLevelCount =
-		rdev->pm.dpm.dyn_state.uvd_clock_voltage_dependency_table.count;
+		min_t(u8, rdev->pm.dpm.dyn_state.uvd_clock_voltage_dependency_table.count,
+		      SMU7_MAX_LEVELS_UVD);
 
 	for (count = 0; count < table->UvdLevelCount; count++) {
 		table->UvdLevel[count].VclkFrequency =
@@ -2730,7 +2731,8 @@
 	int ret = -EINVAL;
 
 	table->VceLevelCount =
-		rdev->pm.dpm.dyn_state.vce_clock_voltage_dependency_table.count;
+		min_t(u8, rdev->pm.dpm.dyn_state.vce_clock_voltage_dependency_table.count,
+		      SMU7_MAX_LEVELS_VCE);
 
 	for (count = 0; count < table->VceLevelCount; count++) {
 		table->VceLevel[count].Frequency =
@@ -2762,8 +2764,9 @@
 	struct atom_clock_dividers dividers;
 	int ret = -EINVAL;
 
-	table->AcpLevelCount = (u8)
-		(rdev->pm.dpm.dyn_state.acp_clock_voltage_dependency_table.count);
+	table->AcpLevelCount =
+		min_t(u8, rdev->pm.dpm.dyn_state.acp_clock_voltage_dependency_table.count,
+		      SMU7_MAX_LEVELS_ACP);
 
 	for (count = 0; count < table->AcpLevelCount; count++) {
 		table->AcpLevel[count].Frequency =
@@ -2795,7 +2798,8 @@
 	int ret = -EINVAL;
 
 	table->SamuLevelCount =
-		rdev->pm.dpm.dyn_state.samu_clock_voltage_dependency_table.count;
+		min_t(u8, rdev->pm.dpm.dyn_state.samu_clock_voltage_dependency_table.count,
+		      SMU7_MAX_LEVELS_SAMU);
 
 	for (count = 0; count < table->SamuLevelCount; count++) {
 		table->SamuLevel[count].Frequency =