lkml.org 
[lkml]   [2008]   [Feb]   [15]   [last100]   RSS-feed
Views: [more markup]  [less markup]  [headers]  [forward] 
 
Messages in this thread
Complete thread (EXPERIMENTAL)
Patch in this message
/
Subject[patch 1/6] Generic thermal management: validate input parameters
DateFri, 15 Feb 2008 10:52:44 +0530
From"Thomas, Sujith" <>
Digg This
From: Thomas Sujith <sujith.thomas@intel.com>
Added sanity check to make sure that thermal zone
and cooling device exists.

Signed-off-by: Thomas Sujith <sujith.thomas@intel.com>
---
 drivers/thermal/thermal.c |   13 ++++++++++++-
 1 files changed, 12 insertions(+), 1 deletion(-)
Index: linux-2.6.24/drivers/thermal/thermal.c

[view this diff only]
===================================================================
--- linux-2.6.24.orig/drivers/thermal/thermal.c
+++ linux-2.6.24/drivers/thermal/thermal.c
@@ -306,12 +306,23 @@ int thermal_zone_bind_cooling_device(str
 {
 	struct thermal_cooling_device_instance *dev;
 	struct thermal_cooling_device_instance *pos;
+	struct thermal_zone_device *pos1;
+	struct thermal_cooling_device *pos2;
 	int result;
 
 	if (trip >= tz->trips || (trip < 0 && trip !=
THERMAL_TRIPS_NONE))
 		return -EINVAL;
 
-	if (!tz || !cdev)
+	list_for_each_entry(pos1, &thermal_tz_list, node) {
+		if (pos1 == tz)
+			break;
+	}
+	list_for_each_entry(pos2, &thermal_cdev_list, node) {
+		if (pos2 == cdev)
+			break;
+	}
+
+	if (tz != pos1 || cdev != pos2)
 		return -EINVAL;
 
 	dev =
[unhandled content-type:application/octet-stream]
\
ISP Services
Valid XHTML 1.0!\ /
Valid CSS! Last update: 2008-02-15 06:29    [W:0.521 / U:0.240 seconds]
©2003-2005 Jasper Spaans