Index: head/usr.sbin/fstyp/hammer.c =================================================================== --- head/usr.sbin/fstyp/hammer.c (revision 356707) +++ head/usr.sbin/fstyp/hammer.c (revision 356708) @@ -1,198 +1,198 @@ /*- * Copyright (c) 2016 The DragonFly Project * All rights reserved. * * This software was developed by Edward Tomasz Napierala under sponsorship * from the FreeBSD Foundation. * * Redistribution and use in source and binary forms, with or without * modification, are permitted provided that the following conditions * are met: * 1. Redistributions of source code must retain the above copyright * notice, this list of conditions and the following disclaimer. * 2. Redistributions in binary form must reproduce the above copyright * notice, this list of conditions and the following disclaimer in the * documentation and/or other materials provided with the distribution. * * THIS SOFTWARE IS PROVIDED BY THE AUTHORS AND CONTRIBUTORS ``AS IS'' AND * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE * ARE DISCLAIMED. IN NO EVENT SHALL THE AUTHORS OR CONTRIBUTORS BE LIABLE * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF * SUCH DAMAGE. */ #include __FBSDID("$FreeBSD$"); #include #include #include #include #include #include #include "hammer_disk.h" #include "fstyp.h" static hammer_volume_ondisk_t __read_ondisk(FILE *fp) { hammer_volume_ondisk_t ondisk; ondisk = read_buf(fp, 0, sizeof(*ondisk)); if (ondisk == NULL) err(1, "failed to read ondisk"); return (ondisk); } static int __test_ondisk(const hammer_volume_ondisk_t ondisk) { static int count = 0; static hammer_uuid_t fsid, fstype; static char label[64]; if (ondisk->vol_signature != HAMMER_FSBUF_VOLUME && ondisk->vol_signature != HAMMER_FSBUF_VOLUME_REV) return (1); if (ondisk->vol_rootvol != HAMMER_ROOT_VOLNO) return (2); if (ondisk->vol_no < 0 || ondisk->vol_no > HAMMER_MAX_VOLUMES - 1) return (3); if (ondisk->vol_count < 1 || ondisk->vol_count > HAMMER_MAX_VOLUMES) return (4); if (count == 0) { count = ondisk->vol_count; assert(count != 0); memcpy(&fsid, &ondisk->vol_fsid, sizeof(fsid)); memcpy(&fstype, &ondisk->vol_fstype, sizeof(fstype)); - strncpy(label, ondisk->vol_label, sizeof(label)); + strlcpy(label, ondisk->vol_label, sizeof(label)); } else { if (ondisk->vol_count != count) return (5); if (memcmp(&ondisk->vol_fsid, &fsid, sizeof(fsid))) return (6); if (memcmp(&ondisk->vol_fstype, &fstype, sizeof(fstype))) return (7); - if (strncmp(ondisk->vol_label, label, sizeof(label))) + if (strcmp(ondisk->vol_label, label)) return (8); } return (0); } int fstyp_hammer(FILE *fp, char *label, size_t size) { hammer_volume_ondisk_t ondisk; int error = 1; ondisk = __read_ondisk(fp); if (ondisk->vol_no != HAMMER_ROOT_VOLNO) goto done; if (ondisk->vol_count != 1) goto done; if (__test_ondisk(ondisk)) goto done; strlcpy(label, ondisk->vol_label, size); error = 0; done: free(ondisk); return (error); } static int __test_volume(const char *volpath) { hammer_volume_ondisk_t ondisk; FILE *fp; int volno = -1; if ((fp = fopen(volpath, "r")) == NULL) err(1, "failed to open %s", volpath); ondisk = __read_ondisk(fp); fclose(fp); if (__test_ondisk(ondisk)) goto done; volno = ondisk->vol_no; done: free(ondisk); return (volno); } static int __fsvtyp_hammer(const char *blkdevs, char *label, size_t size, int partial) { hammer_volume_ondisk_t ondisk; FILE *fp; char *dup, *p, *volpath, x[HAMMER_MAX_VOLUMES]; int i, volno, error = 1; memset(x, 0, sizeof(x)); dup = strdup(blkdevs); p = dup; while (p) { volpath = p; if ((p = strchr(p, ':')) != NULL) *p++ = '\0'; if ((volno = __test_volume(volpath)) == -1) break; x[volno]++; } if ((fp = fopen(volpath, "r")) == NULL) err(1, "failed to open %s", volpath); ondisk = __read_ondisk(fp); fclose(fp); free(dup); if (volno == -1) goto done; if (partial) goto success; for (i = 0; i < HAMMER_MAX_VOLUMES; i++) if (x[i] > 1) goto done; for (i = 0; i < HAMMER_MAX_VOLUMES; i++) if (x[i] == 0) break; if (ondisk->vol_count != i) goto done; for (; i < HAMMER_MAX_VOLUMES; i++) if (x[i] != 0) goto done; success: strlcpy(label, ondisk->vol_label, size); error = 0; done: free(ondisk); return (error); } int fsvtyp_hammer(const char *blkdevs, char *label, size_t size) { return (__fsvtyp_hammer(blkdevs, label, size, 0)); } int fsvtyp_hammer_partial(const char *blkdevs, char *label, size_t size) { return (__fsvtyp_hammer(blkdevs, label, size, 1)); }