Module Name: src Committed By: christos Date: Tue Dec 29 16:45:04 UTC 2015
Modified Files: src/sbin/gpt: add.c biosboot.c create.c gpt.c gpt.h main.c migrate.c set.c show.c unset.c Log Message: - add more informational messages - handle case where last argument is an option not a device (set -l) To generate a diff of this commit: cvs rdiff -u -r1.42 -r1.43 src/sbin/gpt/add.c cvs rdiff -u -r1.23 -r1.24 src/sbin/gpt/biosboot.c cvs rdiff -u -r1.19 -r1.20 src/sbin/gpt/create.c cvs rdiff -u -r1.65 -r1.66 src/sbin/gpt/gpt.c cvs rdiff -u -r1.31 -r1.32 src/sbin/gpt/gpt.h cvs rdiff -u -r1.6 -r1.7 src/sbin/gpt/main.c cvs rdiff -u -r1.29 -r1.30 src/sbin/gpt/migrate.c cvs rdiff -u -r1.12 -r1.13 src/sbin/gpt/set.c src/sbin/gpt/unset.c cvs rdiff -u -r1.34 -r1.35 src/sbin/gpt/show.c Please note that diffs are not public domain; they are subject to the copyright notices on the relevant files.
Modified files: Index: src/sbin/gpt/add.c diff -u src/sbin/gpt/add.c:1.42 src/sbin/gpt/add.c:1.43 --- src/sbin/gpt/add.c:1.42 Thu Dec 3 16:46:22 2015 +++ src/sbin/gpt/add.c Tue Dec 29 11:45:04 2015 @@ -33,7 +33,7 @@ __FBSDID("$FreeBSD: src/sbin/gpt/add.c,v 1.14 2006/06/22 22:05:28 marcel Exp $"); #endif #ifdef __RCSID -__RCSID("$NetBSD: add.c,v 1.42 2015/12/03 21:46:22 christos Exp $"); +__RCSID("$NetBSD: add.c,v 1.43 2015/12/29 16:45:04 christos Exp $"); #endif #include <sys/types.h> @@ -167,7 +167,7 @@ cmd_add(gpt_t gpt, int argc, char *argv[ while ((ch = getopt(argc, argv, GPT_AIS "b:l:t:")) != -1) { switch(ch) { case 'b': - if (gpt_human_get(&block) == -1) + if (gpt_human_get(gpt, &block) == -1) goto usage; break; case 'l': Index: src/sbin/gpt/biosboot.c diff -u src/sbin/gpt/biosboot.c:1.23 src/sbin/gpt/biosboot.c:1.24 --- src/sbin/gpt/biosboot.c:1.23 Thu Dec 3 16:49:51 2015 +++ src/sbin/gpt/biosboot.c Tue Dec 29 11:45:04 2015 @@ -1,4 +1,4 @@ -/* $NetBSD: biosboot.c,v 1.23 2015/12/03 21:49:51 christos Exp $ */ +/* $NetBSD: biosboot.c,v 1.24 2015/12/29 16:45:04 christos Exp $ */ /* * Copyright (c) 2009 The NetBSD Foundation, Inc. @@ -37,7 +37,7 @@ #include <sys/cdefs.h> #ifdef __RCSID -__RCSID("$NetBSD: biosboot.c,v 1.23 2015/12/03 21:49:51 christos Exp $"); +__RCSID("$NetBSD: biosboot.c,v 1.24 2015/12/29 16:45:04 christos Exp $"); #endif #include <sys/stat.h> @@ -266,7 +266,7 @@ cmd_biosboot(gpt_t gpt, int argc, char * goto usage; break; case 'i': - if (gpt_uint_get(&entry) == -1) + if (gpt_uint_get(gpt, &entry) == -1) goto usage; break; case 'L': Index: src/sbin/gpt/create.c diff -u src/sbin/gpt/create.c:1.19 src/sbin/gpt/create.c:1.20 --- src/sbin/gpt/create.c:1.19 Thu Dec 3 16:30:54 2015 +++ src/sbin/gpt/create.c Tue Dec 29 11:45:04 2015 @@ -33,7 +33,7 @@ __FBSDID("$FreeBSD: src/sbin/gpt/create.c,v 1.11 2005/08/31 01:47:19 marcel Exp $"); #endif #ifdef __RCSID -__RCSID("$NetBSD: create.c,v 1.19 2015/12/03 21:30:54 christos Exp $"); +__RCSID("$NetBSD: create.c,v 1.20 2015/12/29 16:45:04 christos Exp $"); #endif #include <sys/types.h> @@ -138,7 +138,7 @@ cmd_create(gpt_t gpt, int argc, char *ar primary_only = 1; break; case 'p': - if (gpt_uint_get(&parts) == -1) + if (gpt_uint_get(gpt, &parts) == -1) return -1; break; default: Index: src/sbin/gpt/gpt.c diff -u src/sbin/gpt/gpt.c:1.65 src/sbin/gpt/gpt.c:1.66 --- src/sbin/gpt/gpt.c:1.65 Sat Dec 26 08:12:16 2015 +++ src/sbin/gpt/gpt.c Tue Dec 29 11:45:04 2015 @@ -35,7 +35,7 @@ __FBSDID("$FreeBSD: src/sbin/gpt/gpt.c,v 1.16 2006/07/07 02:44:23 marcel Exp $"); #endif #ifdef __RCSID -__RCSID("$NetBSD: gpt.c,v 1.65 2015/12/26 13:12:16 jnemeth Exp $"); +__RCSID("$NetBSD: gpt.c,v 1.66 2015/12/29 16:45:04 christos Exp $"); #endif #include <sys/param.h> @@ -592,34 +592,39 @@ out: close(gpt->fd); } +static void +gpt_vwarnx(gpt_t gpt, const char *fmt, va_list ap, const char *e) +{ + if (gpt && (gpt->flags & GPT_QUIET)) + return; + fprintf(stderr, "%s: ", getprogname()); + if (gpt) + fprintf(stderr, "%s: ", gpt->device_name); + vfprintf(stderr, fmt, ap); + if (e) + fprintf(stderr, " (%s)\n", e); + else + fputc('\n', stderr); +} + void gpt_warnx(gpt_t gpt, const char *fmt, ...) { va_list ap; - if (gpt->flags & GPT_QUIET) - return; - fprintf(stderr, "%s: %s: ", getprogname(), gpt->device_name); va_start(ap, fmt); - vfprintf(stderr, fmt, ap); + gpt_vwarnx(gpt, fmt, ap, NULL); va_end(ap); - fprintf(stderr, "\n"); } void gpt_warn(gpt_t gpt, const char *fmt, ...) { va_list ap; - int e = errno; - if (gpt->flags & GPT_QUIET) - return; - fprintf(stderr, "%s: %s: ", getprogname(), gpt->device_name); va_start(ap, fmt); - vfprintf(stderr, fmt, ap); + gpt_vwarnx(gpt, fmt, ap, strerror(errno)); va_end(ap); - fprintf(stderr, " (%s)\n", strerror(e)); - errno = e; } void @@ -627,9 +632,10 @@ gpt_msg(gpt_t gpt, const char *fmt, ...) { va_list ap; - if (gpt->flags & GPT_QUIET) + if (gpt && (gpt->flags & GPT_QUIET)) return; - printf("%s: ", gpt->device_name); + if (gpt) + printf("%s: ", gpt->device_name); va_start(ap, fmt); vprintf(fmt, ap); va_end(ap); @@ -922,17 +928,24 @@ gpt_size_get(gpt_t gpt, off_t *size) } int -gpt_human_get(off_t *human) +gpt_human_get(gpt_t gpt, off_t *human) { int64_t human_num; - if (*human > 0) + if (*human > 0) { + gpt_warn(gpt, "Already set to %jd new `%s'", (intmax_t)*human, + optarg); return -1; - if (dehumanize_number(optarg, &human_num) < 0) + } + if (dehumanize_number(optarg, &human_num) < 0) { + gpt_warn(gpt, "Bad number `%s'", optarg); return -1; + } *human = human_num; - if (*human < 1) + if (*human < 1) { + gpt_warn(gpt, "Number `%s' < 1", optarg); return -1; + } return 0; } @@ -941,16 +954,18 @@ gpt_add_find(gpt_t gpt, struct gpt_find { switch (ch) { case 'a': - if (find->all > 0) + if (find->all > 0) { + gpt_warn(gpt, "-a is already set"); return -1; + } find->all = 1; break; case 'b': - if (gpt_human_get(&find->block) == -1) + if (gpt_human_get(gpt, &find->block) == -1) return -1; break; case 'i': - if (gpt_uint_get(&find->entry) == -1) + if (gpt_uint_get(gpt, &find->entry) == -1) return -1; break; case 'L': @@ -968,6 +983,7 @@ gpt_add_find(gpt_t gpt, struct gpt_find return -1; break; default: + gpt_warn(gpt, "Unknown find option `%c'", ch); return -1; } return 0; @@ -1038,11 +1054,11 @@ gpt_add_ais(gpt_t gpt, off_t *alignment, { switch (ch) { case 'a': - if (gpt_human_get(alignment) == -1) + if (gpt_human_get(gpt, alignment) == -1) return -1; return 0; case 'i': - if (gpt_uint_get(entry) == -1) + if (gpt_uint_get(gpt, entry) == -1) return -1; return 0; case 's': @@ -1050,6 +1066,7 @@ gpt_add_ais(gpt_t gpt, off_t *alignment, return -1; return 0; default: + gpt_warn(gpt, "Unknown alignment/index/size option `%c'", ch); return -1; } } @@ -1167,8 +1184,10 @@ gpt_attr_update(gpt_t gpt, u_int entry, struct gpt_ent *ent; unsigned int i; - if (entry == 0 || (set == 0 && clr == 0)) + if (entry == 0 || (set == 0 && clr == 0)) { + gpt_warnx(gpt, "Nothing to set"); return -1; + } if ((hdr = gpt_hdr(gpt)) == NULL) return -1; @@ -1203,14 +1222,16 @@ gpt_attr_update(gpt_t gpt, u_int entry, } int -gpt_uint_get(u_int *entry) +gpt_uint_get(gpt_t gpt, u_int *entry) { char *p; if (*entry > 0) return -1; *entry = (u_int)strtoul(optarg, &p, 10); - if (*p != 0 || *entry < 1) + if (*p != 0 || *entry < 1) { + gpt_warn(gpt, "Bad number `%s'", optarg); return -1; + } return 0; } int Index: src/sbin/gpt/gpt.h diff -u src/sbin/gpt/gpt.h:1.31 src/sbin/gpt/gpt.h:1.32 --- src/sbin/gpt/gpt.h:1.31 Sat Dec 5 19:39:26 2015 +++ src/sbin/gpt/gpt.h Tue Dec 29 11:45:04 2015 @@ -123,8 +123,8 @@ int gpt_attr_get(gpt_t, uint64_t *); const char *gpt_attr_list(char *, size_t, uint64_t); void gpt_attr_help(const char *); int gpt_attr_update(gpt_t, u_int, uint64_t, uint64_t); -int gpt_uint_get(u_int *); -int gpt_human_get(off_t *); +int gpt_uint_get(gpt_t, u_int *); +int gpt_human_get(gpt_t, off_t *); int gpt_uuid_get(gpt_t, gpt_uuid_t *); int gpt_name_get(gpt_t, void *); int gpt_add_hdr(gpt_t, int, off_t); Index: src/sbin/gpt/main.c diff -u src/sbin/gpt/main.c:1.6 src/sbin/gpt/main.c:1.7 --- src/sbin/gpt/main.c:1.6 Wed Dec 2 21:02:43 2015 +++ src/sbin/gpt/main.c Tue Dec 29 11:45:04 2015 @@ -1,4 +1,4 @@ -/* $NetBSD: main.c,v 1.6 2015/12/03 02:02:43 christos Exp $ */ +/* $NetBSD: main.c,v 1.7 2015/12/29 16:45:04 christos Exp $ */ /*- * Copyright (c) 2002 Marcel Moolenaar @@ -34,7 +34,7 @@ #include <sys/cdefs.h> #ifdef __RCSID -__RCSID("$NetBSD: main.c,v 1.6 2015/12/03 02:02:43 christos Exp $"); +__RCSID("$NetBSD: main.c,v 1.7 2015/12/29 16:45:04 christos Exp $"); #endif #include <stdio.h> @@ -165,7 +165,7 @@ main(int argc, char *argv[]) flags |= GPT_QUIET; break; case 's': - if (gpt_uint_get(&secsz) == -1) + if (gpt_uint_get(NULL, &secsz) == -1) usage(); break; case 'v': @@ -194,13 +194,20 @@ main(int argc, char *argv[]) prefix(cmd); - gpt = gpt_open(dev, flags | cmdsw[i]->flags, verbose, mediasz, secsz); - if (gpt == NULL) - return EXIT_FAILURE; + if (*dev != '-') { + gpt = gpt_open(dev, flags | cmdsw[i]->flags, + verbose, mediasz, secsz); + if (gpt == NULL) + return EXIT_FAILURE; + } else { + argc++; + gpt = NULL; + } if ((*cmdsw[i]->fptr)(gpt, argc, argv) == -1) return EXIT_FAILURE; - gpt_close(gpt); + if (gpt) + gpt_close(gpt); return EXIT_SUCCESS; } Index: src/sbin/gpt/migrate.c diff -u src/sbin/gpt/migrate.c:1.29 src/sbin/gpt/migrate.c:1.30 --- src/sbin/gpt/migrate.c:1.29 Sat Dec 5 23:27:05 2015 +++ src/sbin/gpt/migrate.c Tue Dec 29 11:45:04 2015 @@ -33,7 +33,7 @@ __FBSDID("$FreeBSD: src/sbin/gpt/migrate.c,v 1.16 2005/09/01 02:42:52 marcel Exp $"); #endif #ifdef __RCSID -__RCSID("$NetBSD: migrate.c,v 1.29 2015/12/06 04:27:05 christos Exp $"); +__RCSID("$NetBSD: migrate.c,v 1.30 2015/12/29 16:45:04 christos Exp $"); #endif #include <sys/types.h> @@ -337,7 +337,7 @@ cmd_migrate(gpt_t gpt, int argc, char *a force = 1; break; case 'p': - if (gpt_uint_get(&parts) == -1) + if (gpt_uint_get(gpt, &parts) == -1) return usage(); break; case 's': Index: src/sbin/gpt/set.c diff -u src/sbin/gpt/set.c:1.12 src/sbin/gpt/set.c:1.13 --- src/sbin/gpt/set.c:1.12 Sat Dec 5 19:39:26 2015 +++ src/sbin/gpt/set.c Tue Dec 29 11:45:04 2015 @@ -33,7 +33,7 @@ __FBSDID("$FreeBSD: src/sbin/gpt/add.c,v 1.14 2006/06/22 22:05:28 marcel Exp $"); #endif #ifdef __RCSID -__RCSID("$NetBSD: set.c,v 1.12 2015/12/06 00:39:26 christos Exp $"); +__RCSID("$NetBSD: set.c,v 1.13 2015/12/29 16:45:04 christos Exp $"); #endif #include <sys/types.h> @@ -79,7 +79,7 @@ cmd_set(gpt_t gpt, int argc, char *argv[ return usage(); break; case 'i': - if (gpt_uint_get(&entry) == -1) + if (gpt_uint_get(gpt, &entry) == -1) return usage(); break; case 'l': Index: src/sbin/gpt/unset.c diff -u src/sbin/gpt/unset.c:1.12 src/sbin/gpt/unset.c:1.13 --- src/sbin/gpt/unset.c:1.12 Sat Dec 5 19:39:26 2015 +++ src/sbin/gpt/unset.c Tue Dec 29 11:45:04 2015 @@ -33,7 +33,7 @@ __FBSDID("$FreeBSD: src/sbin/gpt/add.c,v 1.14 2006/06/22 22:05:28 marcel Exp $"); #endif #ifdef __RCSID -__RCSID("$NetBSD: unset.c,v 1.12 2015/12/06 00:39:26 christos Exp $"); +__RCSID("$NetBSD: unset.c,v 1.13 2015/12/29 16:45:04 christos Exp $"); #endif #include <sys/types.h> @@ -78,7 +78,7 @@ cmd_unset(gpt_t gpt, int argc, char *arg return usage(); break; case 'i': - if (gpt_uint_get(&entry) == -1) + if (gpt_uint_get(gpt, &entry) == -1) return usage(); break; case 'l': Index: src/sbin/gpt/show.c diff -u src/sbin/gpt/show.c:1.34 src/sbin/gpt/show.c:1.35 --- src/sbin/gpt/show.c:1.34 Sat Dec 26 08:08:45 2015 +++ src/sbin/gpt/show.c Tue Dec 29 11:45:04 2015 @@ -33,7 +33,7 @@ __FBSDID("$FreeBSD: src/sbin/gpt/show.c,v 1.14 2006/06/22 22:22:32 marcel Exp $"); #endif #ifdef __RCSID -__RCSID("$NetBSD: show.c,v 1.34 2015/12/26 13:08:45 martin Exp $"); +__RCSID("$NetBSD: show.c,v 1.35 2015/12/29 16:45:04 christos Exp $"); #endif #include <sys/types.h> @@ -310,7 +310,7 @@ cmd_show(gpt_t gpt, int argc, char *argv xshow |= SHOW_GUID; break; case 'i': - if (gpt_uint_get(&entry) == -1) + if (gpt_uint_get(gpt, &entry) == -1) return usage(); break; case 'l':