[systemd-devel] [PATCH] path: follow symbolic link for parent path

[Umut Tezduyar Lindskog]

Thanks Zbigniew,

I have only fixed the fallback: case and missed to fix the real case
where we use the name_to_handle. Could you please change that too. It
will look something like this:

@@ -489,7 +489,7 @@ int path_is_mount_point(const char *t, bool allow_symlink) {
-        r = name_to_handle_at(AT_FDCWD, parent, &h.handle,
&mount_id_parent, 0);
+        r = name_to_handle_at(AT_FDCWD, parent, &h.handle,
&mount_id_parent, AT_SYMLINK_FOLLOW);

Umut

On Wed, Dec 17, 2014 at 3:20 AM, Zbigniew Jędrzejewski-Szmek
<zbyszek at in.waw.pl> wrote:
> On Tue, Dec 16, 2014 at 09:58:39PM +0100, Umut Tezduyar Lindskog wrote:
>> ---
>>  src/shared/path-util.c | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/src/shared/path-util.c b/src/shared/path-util.c
>> index dcc8321..304281f 100644
>> --- a/src/shared/path-util.c
>> +++ b/src/shared/path-util.c
>> @@ -518,7 +518,7 @@ fallback:
>>          if (r < 0)
>>                  return r;
>>
>> -        r = lstat(parent, &b);
>> +        r = stat(parent, &b);
>>          if (r < 0)
>>                  return -errno;
> Applied.
>
> Zbyszek
> _______________________________________________
> systemd-devel mailing list
> systemd-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/systemd-devel