[OE-core] [PATCH] Fix mode +st on TMPDIR when creating it
Benjamin Esquivel
benjamin.esquivel at linux.intel.com
Mon Aug 31 16:02:51 UTC 2015
Looks good, would you check this via an autobuilder on-demand build?
Benjamin
On Fri, 2015-08-28 at 17:34 -0500, Alex Franco wrote:
> A sanity check fails when TMPDIR has setuid, setgid set. It was
> proposed to fix this on TMPDIR creation instead of failing with
> a sanity test only. This patch handles removal of those special
> bits (and additonally, the sticky bit) from BUILDDIR and TMPDIR
> when these directories are created.
>
> [YOCTO #7669]
>
> Signed-off-by: Alex Franco <alejandro.franco at linux.intel.com>
> ---
> meta/classes/sanity.bbclass | 5 +++++
> scripts/oe-setup-builddir | 1 +
> 2 files changed, 6 insertions(+)
>
> diff --git a/meta/classes/sanity.bbclass
> b/meta/classes/sanity.bbclass
> index ef90fc8..2864318 100644
> --- a/meta/classes/sanity.bbclass
> +++ b/meta/classes/sanity.bbclass
> @@ -686,6 +686,7 @@ def check_sanity_version_change(status, d):
> status.addresult(check_not_nfs(tmpdir, "TMPDIR"))
>
> def check_sanity_everybuild(status, d):
> + import os, stat
> # Sanity tests which test the users environment so need to run
> at each build (or are so cheap
> # it makes sense to always run them.
>
> @@ -839,6 +840,10 @@ def check_sanity_everybuild(status, d):
> status.addresult("Error, TMPDIR has changed
> location. You need to either move it back to %s or rebuild\n" %
> saved_tmpdir)
> else:
> bb.utils.mkdirhier(tmpdir)
> + # Remove setuid, setgid and sticky bits from TMPDIR
> + os.chmod(tmpdir, os.stat(tmpdir).st_mode & ~ stat.S_ISUID)
> + os.chmod(tmpdir, os.stat(tmpdir).st_mode & ~ stat.S_ISGID)
> + os.chmod(tmpdir, os.stat(tmpdir).st_mode & ~ stat.S_ISVTX)
> with open(checkfile, "w") as f:
> f.write(tmpdir)
>
> diff --git a/scripts/oe-setup-builddir b/scripts/oe-setup-builddir
> index add0b50..f5b7e4e 100755
> --- a/scripts/oe-setup-builddir
> +++ b/scripts/oe-setup-builddir
> @@ -24,6 +24,7 @@ if [ -z "$BUILDDIR" ]; then
> fi
>
> mkdir -p "$BUILDDIR/conf"
> +chmod -R -st "$BUILDDIR"
>
> if [ ! -d "$BUILDDIR" ]; then
> echo >&2 "Error: The builddir ($BUILDDIR) does not exist!"
More information about the Openembedded-core
mailing list