[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#18736: chroot regression - chroot avoids the chroot() call too eager
From: |
Bernhard Voelker |
Subject: |
bug#18736: chroot regression - chroot avoids the chroot() call too eagerly. |
Date: |
Wed, 15 Oct 2014 23:55:11 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.1.0 |
On 10/15/2014 07:17 PM, Pádraig Brady wrote:
> I agree with your analysis and that we should revert
> to the previous behavior here, which is done in
> the attached patch.
Hi Padraig,
I also agree that chroot(1) should chroot(2) in such a case, but wouldn't
be the obvious fix to STREQ() the canonicalized DIR against "/" rather
than reverting the whole change - something like the following?
Have a nice day,
Berny
diff --git a/src/chroot.c b/src/chroot.c
index 171ced9..7f60106 100644
--- a/src/chroot.c
+++ b/src/chroot.c
@@ -175,7 +175,13 @@ is_root (const char* dir)
error (EXIT_CANCELED, errno, _("failed to get attributes of %s"),
quote (dir));
- return SAME_INODE (root_ino, arg_st);
+ if (! SAME_INODE (root_ino, arg_st))
+ return false;
+
+ char *resolved = canonicalize_file_name (dir);
+ bool is_res_root = resolved && STREQ ("/", resolved);
+ free (resolved);
+ return is_res_root;
}
void
--
1.8.4.5
- bug#18736: chroot regression - chroot avoids the chroot() call too eagerly., Rogier, 2014/10/15
- bug#18736: chroot regression - chroot avoids the chroot() call too eagerly., Pádraig Brady, 2014/10/15
- bug#18736: chroot regression - chroot avoids the chroot() call too eagerly.,
Bernhard Voelker <=
- bug#18736: chroot regression - chroot avoids the chroot() call too eagerly., Pádraig Brady, 2014/10/15
- bug#18736: chroot regression - chroot avoids the chroot() call too eagerly., Pádraig Brady, 2014/10/15
- bug#18736: chroot regression - chroot avoids the chroot() call too eagerly., Bernhard Voelker, 2014/10/16
- bug#18736: chroot regression - chroot avoids the chroot() call too eagerly., Pádraig Brady, 2014/10/16
- bug#18736: chroot regression - chroot avoids the chroot() call too eagerly., Bernhard Voelker, 2014/10/16