X-Git-Url: https://git.tokkee.org/?a=blobdiff_plain;f=src%2Fdf.c;h=9d53b0b67eb7c326d3b1811682144d2a7cb6b14b;hb=ca65f66c3cdbbb8af941d3624b85757355f38ad8;hp=9185ba449560d17d13c1bb9c5a8b92927986d391;hpb=d48a9456e42af4ab5881ab055c95ceab71819bc1;p=collectd.git

diff --git a/src/df.c b/src/df.c
index 9185ba44..9d53b0b6 100644
--- a/src/df.c
+++ b/src/df.c
@@ -33,12 +33,14 @@
 #  include <sys/statvfs.h>
 # endif
 # define STATANYFS statvfs
+# define STATANYFS_STR "statvfs"
 # define BLOCKSIZE(s) ((s).f_frsize ? (s).f_frsize : (s).f_bsize)
 #elif HAVE_STATFS
 # if HAVE_SYS_STATFS_H
 #  include <sys/statfs.h>
 # endif
 # define STATANYFS statfs
+# define STATANYFS_STR "statfs"
 # define BLOCKSIZE(s) (s).f_bsize
 #else
 # error "No applicable input method."
@@ -204,7 +206,10 @@ static int df_read (void)
 
 	mnt_list = NULL;
 	if (cu_mount_getlist (&mnt_list) == NULL)
+	{
+		ERROR ("df plugin: cu_mount_getlist failed.");
 		return (-1);
+	}
 
 	for (mnt_ptr = mnt_list; mnt_ptr != NULL; mnt_ptr = mnt_ptr->next)
 	{
@@ -224,7 +229,8 @@ static int df_read (void)
 		if (STATANYFS (mnt_ptr->dir, &statbuf) < 0)
 		{
 			char errbuf[1024];
-			ERROR ("statv?fs failed: %s",
+			ERROR (STATANYFS_STR"(%s) failed: %s",
+					mnt_ptr->dir,
 					sstrerror (errno, errbuf,
 						sizeof (errbuf)));
 			continue;
@@ -274,7 +280,22 @@ static int df_read (void)
 			uint64_t blk_reserved;
 			uint64_t blk_used;
 
-			/* Sanity-check for the values in the struct */
+			/*
+			 * Sanity-check for the values in the struct
+			 */
+			/* Check for negative "available" byes. For example UFS can
+			 * report negative free space for user. Notice. blk_reserved
+			 * will start to diminish after this. */
+#if HAVE_STATVFS
+			/* Cast is needed to avoid compiler warnings.
+			 * ((struct statvfs).f_bavail is unsigned (POSIX)) */
+			if (((int64_t) statbuf.f_bavail) < 0)
+				statbuf.f_bavail = 0;
+#elif HAVE_STATFS
+			if (statbuf.f_bavail < 0)
+				statbuf.f_bavail = 0;
+#endif
+			/* Make sure that f_blocks >= f_bfree >= f_bavail */
 			if (statbuf.f_bfree < statbuf.f_bavail)
 				statbuf.f_bfree = statbuf.f_bavail;
 			if (statbuf.f_blocks < statbuf.f_bfree)