X-Git-Url: https://git.octo.it/?a=blobdiff_plain;f=src%2Fzfs_arc.c;h=d1ee111bb637f70db05bf8fd5b3186bdb4995c87;hb=711f5b6c86f51061c21bedcaa46214a01de0125c;hp=e589184c2cf0fe6b898762270a47462c33d299fa;hpb=196f5bd17880d91ba0da33a8f5f6168d039cfa0c;p=collectd.git diff --git a/src/zfs_arc.c b/src/zfs_arc.c index e589184c..d1ee111b 100644 --- a/src/zfs_arc.c +++ b/src/zfs_arc.c @@ -99,6 +99,11 @@ static void free_zfs_values(kstat_t *ksp) { } #elif defined(KERNEL_SOLARIS) + +#if HAVE_KSTAT_H +#include +#endif + extern kstat_ctl_t *kc; static long long get_zfs_value(kstat_t *ksp, char *name) { @@ -202,9 +207,20 @@ static int za_read(void) { fh = fopen(ZOL_ARCSTATS_FILE, "r"); if (fh == NULL) { - char errbuf[1024]; ERROR("zfs_arc plugin: Opening \"%s\" failed: %s", ZOL_ARCSTATS_FILE, - sstrerror(errno, errbuf, sizeof(errbuf))); + STRERRNO); + return -1; + } + + /* Ignore the first two lines because they contain information about the rest + * of the file. + * See kstat_seq_show_headers module/spl/spl-kstat.c of the spl kernel module. + */ + if ((fgets(buffer, sizeof(buffer), fh) == NULL) || + (fgets(buffer, sizeof(buffer), fh) == NULL)) { + ERROR("zfs_arc plugin: \"%s\" does not contain at least two lines.", + ZOL_ARCSTATS_FILE); + fclose(fh); return -1; } @@ -223,13 +239,13 @@ static int za_read(void) { ERROR("zfs_arc plugin: \"%s\" does not contain a single line.", ZOL_ARCSTATS_FILE); fclose(fh); - return -1; + return (-1); } if (fgets(buffer, sizeof(buffer), fh) == NULL) { ERROR("zfs_arc plugin: \"%s\" does not contain at least two lines.", ZOL_ARCSTATS_FILE); fclose(fh); - return -1; + return (-1); } while (fgets(buffer, sizeof(buffer), fh) != NULL) {