Author: trociny
Date: Sun Jun 30 08:36:19 2013
New Revision: 252410
URL: http://svnweb.freebsd.org/changeset/base/252410

Log:
  MFC r250378:
  
  Make errbuf optional, so if a caller is not interested in an error
  message she can pass NULL (procstat(1) already does this).

Modified:
  stable/9/lib/libprocstat/libprocstat.c
Directory Properties:
  stable/9/lib/libprocstat/   (props changed)

Modified: stable/9/lib/libprocstat/libprocstat.c
==============================================================================
--- stable/9/lib/libprocstat/libprocstat.c      Sun Jun 30 07:46:22 2013        
(r252409)
+++ stable/9/lib/libprocstat/libprocstat.c      Sun Jun 30 08:36:19 2013        
(r252410)
@@ -902,7 +902,8 @@ procstat_get_pipe_info(struct procstat *
                return (procstat_get_pipe_info_sysctl(fst, ps, errbuf));
        } else {
                warnx("unknown access method: %d", procstat->type);
-               snprintf(errbuf, _POSIX2_LINE_MAX, "error");
+               if (errbuf != NULL)
+                       snprintf(errbuf, _POSIX2_LINE_MAX, "error");
                return (1);
        }
 }
@@ -931,7 +932,8 @@ procstat_get_pipe_info_kvm(kvm_t *kd, st
        return (0);
 
 fail:
-       snprintf(errbuf, _POSIX2_LINE_MAX, "error");
+       if (errbuf != NULL)
+               snprintf(errbuf, _POSIX2_LINE_MAX, "error");
        return (1);
 }
 
@@ -967,7 +969,8 @@ procstat_get_pts_info(struct procstat *p
                return (procstat_get_pts_info_sysctl(fst, pts, errbuf));
        } else {
                warnx("unknown access method: %d", procstat->type);
-               snprintf(errbuf, _POSIX2_LINE_MAX, "error");
+               if (errbuf != NULL)
+                       snprintf(errbuf, _POSIX2_LINE_MAX, "error");
                return (1);
        }
 }
@@ -995,7 +998,8 @@ procstat_get_pts_info_kvm(kvm_t *kd, str
        return (0);
 
 fail:
-       snprintf(errbuf, _POSIX2_LINE_MAX, "error");
+       if (errbuf != NULL)
+               snprintf(errbuf, _POSIX2_LINE_MAX, "error");
        return (1);
 }
 
@@ -1030,7 +1034,8 @@ procstat_get_sem_info(struct procstat *p
                return (procstat_get_sem_info_sysctl(fst, sem, errbuf));
        } else {
                warnx("unknown access method: %d", procstat->type);
-               snprintf(errbuf, _POSIX2_LINE_MAX, "error");
+               if (errbuf != NULL)
+                       snprintf(errbuf, _POSIX2_LINE_MAX, "error");
                return (1);
        }
 }
@@ -1076,7 +1081,8 @@ procstat_get_sem_info_kvm(kvm_t *kd, str
        return (0);
 
 fail:
-       snprintf(errbuf, _POSIX2_LINE_MAX, "error");
+       if (errbuf != NULL)
+               snprintf(errbuf, _POSIX2_LINE_MAX, "error");
        return (1);
 }
 
@@ -1111,7 +1117,8 @@ procstat_get_shm_info(struct procstat *p
                return (procstat_get_shm_info_sysctl(fst, shm, errbuf));
        } else {
                warnx("unknown access method: %d", procstat->type);
-               snprintf(errbuf, _POSIX2_LINE_MAX, "error");
+               if (errbuf != NULL)
+                       snprintf(errbuf, _POSIX2_LINE_MAX, "error");
                return (1);
        }
 }
@@ -1157,7 +1164,8 @@ procstat_get_shm_info_kvm(kvm_t *kd, str
        return (0);
 
 fail:
-       snprintf(errbuf, _POSIX2_LINE_MAX, "error");
+       if (errbuf != NULL)
+               snprintf(errbuf, _POSIX2_LINE_MAX, "error");
        return (1);
 }
 
@@ -1192,7 +1200,8 @@ procstat_get_vnode_info(struct procstat 
                return (procstat_get_vnode_info_sysctl(fst, vn, errbuf));
        } else {
                warnx("unknown access method: %d", procstat->type);
-               snprintf(errbuf, _POSIX2_LINE_MAX, "error");
+               if (errbuf != NULL)
+                       snprintf(errbuf, _POSIX2_LINE_MAX, "error");
                return (1);
        }
 }
@@ -1264,7 +1273,8 @@ procstat_get_vnode_info_kvm(kvm_t *kd, s
                        break;
                }
        if (i == NTYPES) {
-               snprintf(errbuf, _POSIX2_LINE_MAX, "?(%s)", tagstr);
+               if (errbuf != NULL)
+                       snprintf(errbuf, _POSIX2_LINE_MAX, "?(%s)", tagstr);
                return (1);
        }
        vn->vn_mntdir = getmnton(kd, vnode.v_mount);
@@ -1278,7 +1288,8 @@ procstat_get_vnode_info_kvm(kvm_t *kd, s
        return (0);
 
 fail:
-       snprintf(errbuf, _POSIX2_LINE_MAX, "error");
+       if (errbuf != NULL)
+               snprintf(errbuf, _POSIX2_LINE_MAX, "error");
        return (1);
 }
 
@@ -1359,7 +1370,10 @@ procstat_get_vnode_info_sysctl(struct fi
        if (vntype == PS_FST_VTYPE_VNON || vntype == PS_FST_VTYPE_VBAD)
                return (0);
        if ((status & KF_ATTR_VALID) == 0) {
-               snprintf(errbuf, _POSIX2_LINE_MAX, "? (no info available)");
+               if (errbuf != NULL) {
+                       snprintf(errbuf, _POSIX2_LINE_MAX,
+                           "? (no info available)");
+               }
                return (1);
        }
        if (path && *path) {
@@ -1400,7 +1414,8 @@ procstat_get_socket_info(struct procstat
                return (procstat_get_socket_info_sysctl(fst, sock, errbuf));
        } else {
                warnx("unknown access method: %d", procstat->type);
-               snprintf(errbuf, _POSIX2_LINE_MAX, "error");
+               if (errbuf != NULL)
+                       snprintf(errbuf, _POSIX2_LINE_MAX, "error");
                return (1);
        }
 }
@@ -1498,7 +1513,8 @@ procstat_get_socket_info_kvm(kvm_t *kd, 
        return (0);
 
 fail:
-       snprintf(errbuf, _POSIX2_LINE_MAX, "error");
+       if (errbuf != NULL)
+               snprintf(errbuf, _POSIX2_LINE_MAX, "error");
        return (1);
 }
 
_______________________________________________
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

Reply via email to