projects
/
collectd.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
collectd-tg: Fix sleep interval.
[collectd.git]
/
src
/
utils_mount.c
diff --git
a/src/utils_mount.c
b/src/utils_mount.c
index
6555b41
..
e527c25
100644
(file)
--- a/
src/utils_mount.c
+++ b/
src/utils_mount.c
@@
-24,15
+24,21
@@
# include "config.h"
#endif
# include "config.h"
#endif
-#include "common.h"
+#define _GNU_SOURCE
+
+#include "collectd.h"
+
+#include "utils_mount.h"
+
#if HAVE_XFS_XQM_H
# include <xfs/xqm.h>
#define XFS_SUPER_MAGIC_STR "XFSB"
#define XFS_SUPER_MAGIC2_STR "BSFX"
#endif
#if HAVE_XFS_XQM_H
# include <xfs/xqm.h>
#define XFS_SUPER_MAGIC_STR "XFSB"
#define XFS_SUPER_MAGIC2_STR "BSFX"
#endif
-#include "plugin.h"
-#include "utils_mount.h"
+#include "common.h" /* sstrncpy() et alii */
+#include "plugin.h" /* ERROR() macro */
+
#if HAVE_GETVFSSTAT
# if HAVE_SYS_TYPES_H
#if HAVE_GETVFSSTAT
# if HAVE_SYS_TYPES_H
@@
-211,7
+217,6
@@
uuidcache_init(void)
FILE *procpt;
char uuid[16], *label = NULL;
char device[110];
FILE *procpt;
char uuid[16], *label = NULL;
char device[110];
- int firstPass;
int handleOnFirst;
if(uuidCache) {
int handleOnFirst;
if(uuidCache) {
@@
-223,7
+228,7
@@
uuidcache_init(void)
return;
}
return;
}
- for(firstPass = 1; firstPass >= 0; firstPass--) {
+ for(
int
firstPass = 1; firstPass >= 0; firstPass--) {
fseek(procpt, 0, SEEK_SET);
while(fgets(line, sizeof(line), procpt)) {
if(sscanf(line, " %d %d %d %[^\n ]",
fseek(procpt, 0, SEEK_SET);
while(fgets(line, sizeof(line), procpt)) {
if(sscanf(line, " %d %d %d %[^\n ]",
@@
-313,14
+318,13
@@
static char *
get_spec_by_uuid(const char *s)
{
char uuid[16];
get_spec_by_uuid(const char *s)
{
char uuid[16];
- int i;
if(strlen(s) != 36
|| s[8] != '-' || s[13] != '-' || s[18] != '-' || s[23] != '-') {
goto bad_uuid;
}
if(strlen(s) != 36
|| s[8] != '-' || s[13] != '-' || s[18] != '-' || s[23] != '-') {
goto bad_uuid;
}
- for(i=0; i<16; i++) {
+ for(i
nt i
=0; i<16; i++) {
if(*s == '-') {
s++;
}
if(*s == '-') {
s++;
}
@@
-377,7
+381,6
@@
static char *get_device_name(const char *optstr)
static cu_mount_t *cu_mount_listmntent (void)
{
cu_mount_t *last = *list;
static cu_mount_t *cu_mount_listmntent (void)
{
cu_mount_t *last = *list;
- struct tabmntent *p;
struct mntent *mnt;
struct tabmntent *mntlist;
struct mntent *mnt;
struct tabmntent *mntlist;
@@
-389,7
+392,7
@@
static cu_mount_t *cu_mount_listmntent (void)
#endif /* COLLECT_DEBUG */
}
#endif /* COLLECT_DEBUG */
}
- for(p = mntlist; p; p = p->next) {
+ for(
struct tabmntent *
p = mntlist; p; p = p->next) {
char *loop = NULL, *device = NULL;
mnt = p->ment;
char *loop = NULL, *device = NULL;
mnt = p->ment;
@@
-446,7
+449,6
@@
static cu_mount_t *cu_mount_getfsstat (void)
STRUCT_STATFS *buf;
int num;
STRUCT_STATFS *buf;
int num;
- int i;
cu_mount_t *first = NULL;
cu_mount_t *last = NULL;
cu_mount_t *first = NULL;
cu_mount_t *last = NULL;
@@
-463,10
+465,8
@@
static cu_mount_t *cu_mount_getfsstat (void)
return (NULL);
}
return (NULL);
}
- if ((buf = (STRUCT_STATFS *) malloc (bufsize * sizeof (STRUCT_STATFS)))
- == NULL)
+ if ((buf = calloc (bufsize, sizeof (*buf))) == NULL)
return (NULL);
return (NULL);
- memset (buf, '\0', bufsize * sizeof (STRUCT_STATFS));
/* The bufsize needs to be passed in bytes. Really. This is not in the
* manpage.. -octo */
/* The bufsize needs to be passed in bytes. Really. This is not in the
* manpage.. -octo */
@@
-481,12
+481,11
@@
static cu_mount_t *cu_mount_getfsstat (void)
return (NULL);
}
return (NULL);
}
- for (i = 0; i < num; i++)
+ for (i
nt i
= 0; i < num; i++)
{
{
- if ((new =
malloc (sizeof (cu_mount_t
))) == NULL)
+ if ((new =
calloc (1, sizeof (*new
))) == NULL)
break;
break;
- memset (new, '\0', sizeof (cu_mount_t));
-
+
/* Copy values from `struct mnttab' */
new->dir = sstrdup (buf[i].f_mntonname);
new->spec_device = sstrdup (buf[i].f_mntfromname);
/* Copy values from `struct mnttab' */
new->dir = sstrdup (buf[i].f_mntonname);
new->spec_device = sstrdup (buf[i].f_mntfromname);
@@
-537,10
+536,9
@@
static cu_mount_t *cu_mount_gen_getmntent (void)
while (getmntent (fp, &mt) == 0)
{
while (getmntent (fp, &mt) == 0)
{
- if ((new =
malloc (sizeof (cu_mount_t
))) == NULL)
+ if ((new =
calloc (1, sizeof (*new
))) == NULL)
break;
break;
- memset (new, '\0', sizeof (cu_mount_t));
-
+
/* Copy values from `struct mnttab' */
new->dir = sstrdup (mt.mnt_mountp);
new->spec_device = sstrdup (mt.mnt_special);
/* Copy values from `struct mnttab' */
new->dir = sstrdup (mt.mnt_mountp);
new->spec_device = sstrdup (mt.mnt_special);
@@
-595,9
+593,8
@@
static cu_mount_t *cu_mount_getmntent (void)
while (getmntent_r (fp, &me, mntbuf, sizeof (mntbuf) ))
{
while (getmntent_r (fp, &me, mntbuf, sizeof (mntbuf) ))
{
- if ((new =
malloc (sizeof (cu_mount_t
))) == NULL)
+ if ((new =
calloc (1, sizeof (*new
))) == NULL)
break;
break;
- memset (new, '\0', sizeof (cu_mount_t));
/* Copy values from `struct mntent *' */
new->dir = sstrdup (me.mnt_dir);
/* Copy values from `struct mntent *' */
new->dir = sstrdup (me.mnt_dir);
@@
-652,10
+649,9
@@
static cu_mount_t *cu_mount_getmntent (void)
while ((me = getmntent (fp)) != NULL)
{
while ((me = getmntent (fp)) != NULL)
{
- if ((new =
malloc (sizeof (cu_mount_t
))) == NULL)
+ if ((new =
calloc (1, sizeof (*new
))) == NULL)
break;
break;
- memset (new, '\0', sizeof (cu_mount_t));
-
+
/* Copy values from `struct mntent *' */
new->dir = sstrdup (me->mnt_dir);
new->spec_device = sstrdup (me->mnt_fsname);
/* Copy values from `struct mntent *' */
new->dir = sstrdup (me->mnt_dir);
new->spec_device = sstrdup (me->mnt_fsname);
@@
-742,10
+738,9
@@
cu_mount_t *cu_mount_getlist(cu_mount_t **list)
void cu_mount_freelist (cu_mount_t *list)
{
void cu_mount_freelist (cu_mount_t *list)
{
- cu_mount_t *this;
cu_mount_t *next;
cu_mount_t *next;
- for (this = list; this != NULL; this = next)
+ for (
cu_mount_t *
this = list; this != NULL; this = next)
{
next = this->next;
{
next = this->next;
@@
-759,7
+754,7
@@
void cu_mount_freelist (cu_mount_t *list)
} /* void cu_mount_freelist(cu_mount_t *list) */
char *
} /* void cu_mount_freelist(cu_mount_t *list) */
char *
-cu_mount_checkoption(char *line, char *keyword, int full)
+cu_mount_checkoption(char *line, c
onst c
har *keyword, int full)
{
char *line2, *l2, *p1, *p2;
int l;
{
char *line2, *l2, *p1, *p2;
int l;
@@
-800,7
+795,7
@@
cu_mount_checkoption(char *line, char *keyword, int full)
} /* char *cu_mount_checkoption(char *line, char *keyword, int full) */
char *
} /* char *cu_mount_checkoption(char *line, char *keyword, int full) */
char *
-cu_mount_getoptionvalue(char *line, char *keyword)
+cu_mount_getoptionvalue(char *line, c
onst c
har *keyword)
{
char *r;
{
char *r;
@@
-816,13
+811,13
@@
cu_mount_getoptionvalue(char *line, char *keyword)
if((p-r) == 1) {
return NULL;
}
if((p-r) == 1) {
return NULL;
}
- m =
(char *)
smalloc(p-r+1);
+ m = smalloc(p-r+1);
sstrncpy(m, r, p-r+1);
return m;
}
}
return r;
sstrncpy(m, r, p-r+1);
return m;
}
}
return r;
-} /* char *cu_mount_getoptionvalue(char *line, char *keyword) */
+} /* char *cu_mount_getoptionvalue(char *line, c
onst c
har *keyword) */
int
cu_mount_type(const char *type)
int
cu_mount_type(const char *type)