[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Re: grub-install --root-directory=/mnt /dev/sda1 fails
From: |
Felix Zielcke |
Subject: |
Re: [PATCH] Re: grub-install --root-directory=/mnt /dev/sda1 fails |
Date: |
Thu, 11 Jun 2009 01:00:21 +0200 |
Am Dienstag, den 09.06.2009, 23:51 +0200 schrieb Vladimir 'phcoder'
Serbinenko:
> +
> +char *grub_make_system_path_relative_to_its_root (char *path)
> +{
> +
> + struct stat st;
> + char buf[500], buf2[500];
> Use malloc instead of static allocation
Changed.
> + p = strrchr (buf, '/');
> + if (p != buf)
> + *p = 0;
> + else *++p = 0;
> You assume path starts with /. You have to check this. Otherwise you
> may get sigsegv
Changed.
> + strcpy(buf2,buf);
> Just save (p - buf) instead of copying buf to buf2
I did this now if realpath () isn't avaible.
Now realpath is used in case it's avaible. POSIX specifies it and
readlink is actually using it if it's avaible. I didn't read the source
correctly.
The problem is just Cygwin. It has it but it returns the cygwin path.
So C:\\Windows would get /cygdrive/c/windows, which is easy to handle.
But realpath ("/boot/grub") would return /boot/grub which isn't true
from Windows/GRUB point of view.
Maybe Christian and Bean can say something about it.
MingW doestn't have it and I don't know what Windows would have, so the
MingW users would still use my own way.
--
Felix Zielcke
make_sys_path_relative.patch.3
Description: Text document
- Re: grub-install --root-directory=/mnt /dev/sda1 fails, Felix Zielcke, 2009/06/01
- [PATCH] Re: grub-install --root-directory=/mnt /dev/sda1 fails, Felix Zielcke, 2009/06/08
- Re: [PATCH] Re: grub-install --root-directory=/mnt /dev/sda1 fails, Vladimir 'phcoder' Serbinenko, 2009/06/09
- Re: [PATCH] Re: grub-install --root-directory=/mnt /dev/sda1 fails,
Felix Zielcke <=
- Re: [PATCH] Re: grub-install --root-directory=/mnt /dev/sda1 fails, Felix Zielcke, 2009/06/11
- Re: [PATCH] Re: grub-install --root-directory=/mnt /dev/sda1 fails, Felix Zielcke, 2009/06/11
- Re: [PATCH] Re: grub-install --root-directory=/mnt /dev/sda1 fails, Felix Zielcke, 2009/06/12
- Re: [PATCH] Re: grub-install --root-directory=/mnt /dev/sda1 fails, Pavel Roskin, 2009/06/12
- Re: [PATCH] Re: grub-install --root-directory=/mnt /dev/sda1 fails, Felix Zielcke, 2009/06/12