Fix Static Analysis Issues for HOST thermal Utility Module.

Below mentioned Static Analysis issues have been fixed. +-----------------------------------------+ | 1. Unchecked return value from library | | | | 2. Ignoring number of bytes read | +-----------------------------------------+ Tracked-On: OAM-115786 Signed-off-by: RajaniRanjan <[email protected]> Signed-off-by: Vilas R K <[email protected]>
projectceladon · Mar 4, 2024 · 1b75b0c · 1b75b0c
1 parent 7616257
commit 1b75b0c
Showing 1 changed file with 12 additions and 4 deletions.
diff --git a/vm_thermal_utility/thermal_sysfsread.c b/vm_thermal_utility/thermal_sysfsread.c
@@ -76,10 +76,18 @@ void read_sysfs_values(char *base_path, char *filename, void *buf, int len, int
 		return;
 	}
 
-	if (flag==0)
-		fread(buf, len, 1, fp);
-	else
-    		fscanf (fp, "%d", (int*)buf);  /* read/validate value */
+	if (flag==0) {
+	    if (fread(buf, len, 1, fp) == EOF) {
+                fclose (fp);
+                return;
+	    }
+	}
+	else {
+            if (fscanf (fp, "%d", (int*)buf) < 0) {
+                fclose (fp);
+                return;
+            }
+	}
 	fclose (fp);
 	return;
 }