> Am 23.02.2019 um 02:20 schrieb Heinrich Schuchardt <[email protected]>: > > In the `efidebug boot add` command we do not want a unsollicitated leading > backslash added to the file name. > > There is no good reason to mark a loaded file with a backslash as absolute. > Anyway when reading files the file name will be interpreted as relative to > root directory of the device. > > So let's get rid of this backslash. > > Signed-off-by: Heinrich Schuchardt <[email protected]> > --- > lib/efi_loader/efi_device_path.c | 7 +------ > 1 file changed, 1 insertion(+), 6 deletions(-) > > diff --git a/lib/efi_loader/efi_device_path.c > b/lib/efi_loader/efi_device_path.c > index 98c36e798f5..53b40c8c3c2 100644 > --- a/lib/efi_loader/efi_device_path.c > +++ b/lib/efi_loader/efi_device_path.c > @@ -985,12 +985,7 @@ efi_status_t efi_dp_from_name(const char *dev, const > char *devnr, > if (!path) > return EFI_SUCCESS; > > - if (!is_net) { > - /* Add leading / to fs paths, because they're absolute */ > - snprintf(filename, sizeof(filename), "/%s", path); > - } else { > - snprintf(filename, sizeof(filename), "%s", path); > - } > + snprintf(filename, sizeof(filename), "%s", path);
I'm pretty sure that code was there for a reason. Without, grub would not find its config file iirc. Alex > /* DOS style file path: */ > s = filename; > while ((s = strchr(s, '/'))) > -- > 2.20.1 > _______________________________________________ U-Boot mailing list [email protected] https://lists.denx.de/listinfo/u-boot
