diff options
Diffstat (limited to 'usr/src')
-rw-r--r-- | usr/src/boot/Makefile.version | 2 | ||||
-rw-r--r-- | usr/src/boot/sys/boot/efi/loader/devicename.c | 29 | ||||
-rw-r--r-- | usr/src/boot/sys/boot/i386/libi386/devicename.c | 264 |
3 files changed, 149 insertions, 146 deletions
diff --git a/usr/src/boot/Makefile.version b/usr/src/boot/Makefile.version index ce9a4e112c..782a9fa9c1 100644 --- a/usr/src/boot/Makefile.version +++ b/usr/src/boot/Makefile.version @@ -33,4 +33,4 @@ LOADER_VERSION = 1.1 # Use date like formatting here, YYYY.MM.DD.XX, without leading zeroes. # The version is processed from left to right, the version number can only # be increased. -BOOT_VERSION = $(LOADER_VERSION)-2017.6.10.1 +BOOT_VERSION = $(LOADER_VERSION)-2017.6.11.1 diff --git a/usr/src/boot/sys/boot/efi/loader/devicename.c b/usr/src/boot/sys/boot/efi/loader/devicename.c index 64a417dacc..72588b08f5 100644 --- a/usr/src/boot/sys/boot/efi/loader/devicename.c +++ b/usr/src/boot/sys/boot/efi/loader/devicename.c @@ -1,4 +1,4 @@ -/*- +/* * Copyright (c) 1998 Michael Smith <msmith@freebsd.org> * Copyright (c) 2006 Marcel Moolenaar * All rights reserved. @@ -26,16 +26,13 @@ */ #include <sys/cdefs.h> -__FBSDID("$FreeBSD$"); #include <stand.h> #include <string.h> #include <sys/disklabel.h> #include <sys/param.h> #include <bootstrap.h> -#ifdef EFI_ZFS_BOOT #include <libzfs.h> -#endif #include <efi.h> #include <efilib.h> @@ -99,7 +96,7 @@ efi_parsedev(struct devdesc **dev, const char *devspec, const char **path) /* look for a device that matches */ for (i = 0; devsw[i] != NULL; i++) { dv = devsw[i]; - if (!strncmp(devspec, dv->dv_name, strlen(dv->dv_name))) + if (strncmp(devspec, dv->dv_name, strlen(dv->dv_name)) == 0) break; } if (devsw[i] == NULL) @@ -107,24 +104,21 @@ efi_parsedev(struct devdesc **dev, const char *devspec, const char **path) np = devspec + strlen(dv->dv_name); -#ifdef EFI_ZFS_BOOT if (dv->dv_type == DEVT_ZFS) { int err; - idev = malloc(sizeof(struct zfs_devdesc)); + idev = malloc(sizeof (struct zfs_devdesc)); if (idev == NULL) return (ENOMEM); - err = zfs_parsedev((struct zfs_devdesc*)idev, np, path); + err = zfs_parsedev((struct zfs_devdesc *)idev, np, path); if (err != 0) { free(idev); return (err); } cp = strchr(np + 1, ':'); - } else -#endif - { - idev = malloc(sizeof(struct devdesc)); + } else { + idev = malloc(sizeof (struct devdesc)); if (idev == NULL) return (ENOMEM); @@ -147,7 +141,7 @@ efi_parsedev(struct devdesc **dev, const char *devspec, const char **path) } if (path != NULL) - *path = (*cp == 0) ? cp : cp + 1; + *path = (*cp == '\0') ? cp : cp + 1; if (dev != NULL) *dev = idev; else @@ -161,17 +155,16 @@ efi_fmtdev(void *vdev) struct devdesc *dev = (struct devdesc *)vdev; static char buf[SPECNAMELEN + 1]; - switch(dev->d_type) { -#ifdef EFI_ZFS_BOOT + switch (dev->d_type) { case DEVT_ZFS: return (zfs_fmtdev(dev)); -#endif case DEVT_NONE: - strcpy(buf, "(no device)"); + strlcpy(buf, "(no device)", sizeof (buf)); break; default: - sprintf(buf, "%s%d:", dev->d_dev->dv_name, dev->d_unit); + snprintf(buf, sizeof (buf), "%s%d:", dev->d_dev->dv_name, + dev->d_unit); break; } diff --git a/usr/src/boot/sys/boot/i386/libi386/devicename.c b/usr/src/boot/sys/boot/i386/libi386/devicename.c index c7705d7138..b692214b95 100644 --- a/usr/src/boot/sys/boot/i386/libi386/devicename.c +++ b/usr/src/boot/sys/boot/i386/libi386/devicename.c @@ -1,4 +1,4 @@ -/*- +/* * Copyright (c) 1998 Michael Smith <msmith@freebsd.org> * All rights reserved. * @@ -25,46 +25,45 @@ */ #include <sys/cdefs.h> -__FBSDID("$FreeBSD$"); #include <stand.h> #include <string.h> +#include <sys/param.h> #include "bootstrap.h" #include "disk.h" #include "libi386.h" #include "../zfs/libzfs.h" -static int i386_parsedev(struct i386_devdesc **dev, const char *devspec, const char **path); +static int i386_parsedev(struct i386_devdesc **, const char *, const char **); -/* - * Point (dev) at an allocated device specifier for the device matching the - * path in (devspec). If it contains an explicit device specification, +/* + * Point dev at an allocated device specifier for the device matching the + * path in devspec. If it contains an explicit device specification, * use that. If not, use the default device. */ int i386_getdev(void **vdev, const char *devspec, const char **path) { - struct i386_devdesc **dev = (struct i386_devdesc **)vdev; - int rv; - - /* - * If it looks like this is just a path and no - * device, go with the current device. - */ - if ((devspec == NULL) || - (devspec[0] == '/') || - (strchr(devspec, ':') == NULL)) { - - if (((rv = i386_parsedev(dev, getenv("currdev"), NULL)) == 0) && - (path != NULL)) - *path = devspec; - return(rv); - } - - /* - * Try to parse the device name off the beginning of the devspec - */ - return(i386_parsedev(dev, devspec, path)); + struct i386_devdesc **dev = (struct i386_devdesc **)vdev; + int rv; + + /* + * If it looks like this is just a path and no + * device, go with the current device. + */ + if ((devspec == NULL) || + (devspec[0] == '/') || (strchr(devspec, ':') == NULL)) { + + rv = i386_parsedev(dev, getenv("currdev"), NULL); + if (rv == 0 && path != NULL) + *path = devspec; + return (rv); + } + + /* + * Try to parse the device name off the beginning of the devspec + */ + return (i386_parsedev(dev, devspec, path)); } /* @@ -79,113 +78,123 @@ i386_getdev(void **vdev, const char *devspec, const char **path) * For disk-type devices, the syntax is: * * disk<unit>[s<slice>][<partition>]: - * + * */ static int i386_parsedev(struct i386_devdesc **dev, const char *devspec, const char **path) { - struct i386_devdesc *idev; - struct devsw *dv; - int i, unit, err; - char *cp; - const char *np; - - /* minimum length check */ - if (strlen(devspec) < 2) - return(EINVAL); - - /* look for a device that matches */ - for (i = 0, dv = NULL; devsw[i] != NULL; i++) { - if (!strncmp(devspec, devsw[i]->dv_name, strlen(devsw[i]->dv_name))) { - dv = devsw[i]; - break; + struct i386_devdesc *idev; + struct devsw *dv; + int i, unit, err; + char *cp; + const char *np; + + /* minimum length check */ + if (strlen(devspec) < 2) + return (EINVAL); + + /* look for a device that matches */ + for (i = 0, dv = NULL; devsw[i] != NULL; i++) { + dv = devsw[i]; + if (strncmp(devspec, dv->dv_name, strlen(dv->dv_name)) == 0) + break; } - } - if (dv == NULL) - return(ENOENT); - idev = malloc(sizeof(struct i386_devdesc)); - err = 0; - np = (devspec + strlen(dv->dv_name)); - - switch(dv->dv_type) { - case DEVT_NONE: /* XXX what to do here? Do we care? */ - break; - - case DEVT_DISK: - err = disk_parsedev((struct disk_devdesc *)idev, np, path); - if (err != 0) - goto fail; - break; - - case DEVT_CD: - case DEVT_NET: - unit = 0; - - if (*np && (*np != ':')) { - unit = strtol(np, &cp, 0); /* get unit number if present */ - if (cp == np) { - err = EUNIT; - goto fail; - } - } else { + if (devsw[i] == NULL) + return (ENOENT); + + np = devspec + strlen(dv->dv_name); + idev = NULL; + err = 0; + + switch (dv->dv_type) { + case DEVT_NONE: + break; + + case DEVT_DISK: + idev = malloc(sizeof (struct i386_devdesc)); + if (idev == NULL) + return (ENOMEM); + + err = disk_parsedev((struct disk_devdesc *)idev, np, path); + if (err != 0) + goto fail; + break; + + case DEVT_ZFS: + idev = malloc(sizeof (struct zfs_devdesc)); + if (idev == NULL) + return (ENOMEM); + + err = zfs_parsedev((struct zfs_devdesc *)idev, np, path); + if (err != 0) + goto fail; + break; + + default: + idev = malloc(sizeof (struct devdesc)); + if (idev == NULL) + return (ENOMEM); + + unit = 0; cp = (char *)np; + + if (*np && (*np != ':')) { + /* get unit number if present */ + unit = strtol(np, &cp, 0); + if (cp == np) { + err = EUNIT; + goto fail; + } + } + if (*cp && (*cp != ':')) { + err = EINVAL; + goto fail; + } + + idev->d_unit = unit; + if (path != NULL) + *path = (*cp == '\0') ? cp : cp + 1; + break; } - if (*cp && (*cp != ':')) { - err = EINVAL; - goto fail; - } - idev->d_unit = unit; - if (path != NULL) - *path = (*cp == 0) ? cp : cp + 1; - break; - case DEVT_ZFS: - err = zfs_parsedev((struct zfs_devdesc *)idev, np, path); - if (err != 0) - goto fail; - break; - default: - err = EINVAL; - goto fail; - } - idev->d_dev = dv; - idev->d_type = dv->dv_type; - if (dev == NULL) { + idev->d_dev = dv; + idev->d_type = dv->dv_type; + + if (dev != NULL) + *dev = idev; + else + free(idev); + return (0); + +fail: free(idev); - } else { - *dev = idev; - } - return(0); - - fail: - free(idev); - return(err); + return (err); } char * i386_fmtdev(void *vdev) { - struct i386_devdesc *dev = (struct i386_devdesc *)vdev; - static char buf[128]; /* XXX device length constant? */ - - switch(dev->d_type) { - case DEVT_NONE: - strcpy(buf, "(no device)"); - break; - - case DEVT_CD: - case DEVT_NET: - sprintf(buf, "%s%d:", dev->d_dev->dv_name, dev->d_unit); - break; - - case DEVT_DISK: - return (disk_fmtdev(vdev)); - - case DEVT_ZFS: - return(zfs_fmtdev(vdev)); - } - return(buf); + struct i386_devdesc *dev = (struct i386_devdesc *)vdev; + static char buf[SPECNAMELEN + 1]; + + switch (dev->d_type) { + case DEVT_NONE: + strlcpy(buf, "(no device)", sizeof (buf)); + break; + + case DEVT_DISK: + return (disk_fmtdev(vdev)); + + case DEVT_ZFS: + return (zfs_fmtdev(vdev)); + + default: + snprintf(buf, sizeof (buf), "%s%d:", dev->d_dev->dv_name, + dev->d_unit); + break; + } + return (buf); } @@ -195,12 +204,13 @@ i386_fmtdev(void *vdev) int i386_setcurrdev(struct env_var *ev, int flags, const void *value) { - struct i386_devdesc *ncurr; - int rv; - - if ((rv = i386_parsedev(&ncurr, value, NULL)) != 0) - return(rv); - free(ncurr); - env_setenv(ev->ev_name, flags | EV_NOHOOK, value, NULL, NULL); - return(0); + struct i386_devdesc *ncurr; + int rv; + + if ((rv = i386_parsedev(&ncurr, value, NULL)) != 0) + return (rv); + + free(ncurr); + env_setenv(ev->ev_name, flags | EV_NOHOOK, value, NULL, NULL); + return (0); } |