Hi tech@, Remove setting null-terminated string operation, since bzero() has done this operation before. Sorry if I miss some points.
Thanks! Best Regards Nan Xiao Index: pctr.c =================================================================== RCS file: /cvs/src/usr.bin/pctr/pctr.c,v retrieving revision 1.22 diff -u -p -r1.22 pctr.c --- pctr.c 8 Feb 2015 23:40:34 -0000 1.22 +++ pctr.c 9 Sep 2017 03:43:36 -0000 @@ -169,7 +169,6 @@ pctr_cpu_creds(void) bzero(arch, sizeof(arch)); if (sysctl(mib, 2, arch, &len, NULL, 0) == -1) err(1, "HW_MACHINE"); - arch[len] = '\0'; if (strcmp(arch, "i386") == 0) atype = ARCH_I386; @@ -198,7 +197,6 @@ pctr_cpu_creds(void) bzero(vendor, sizeof(vendor)); if (sysctl(mib, 2, vendor, &len, NULL, 0) == -1) err(1, "CPU_CPUVENDOR"); - vendor[len] = '\0'; switch (atype) { case ARCH_I386: