diff options
| author |   Joël Esponde <joel.esponde@easymile.com> | 2019-07-18 15:47:39 +0200 |
|---|---|---|
| committer |   Armin Kuster <akuster808@gmail.com> | 2019-09-04 08:21:53 -0700 |
| commit | fe6546aab208e5fa2e238aa266db0ea66ad520a1 (patch) | |
| tree | 546e099f25e1d35609c127694fdb8ef385953fca | |
| parent | 423115b70a4a2cdef4b3882ad4491446b84a1f1e (diff) | |
| download | openembedded-core-contrib-fe6546aab208e5fa2e238aa266db0ea66ad520a1.tar.gz | |
package.bbclass: fix directories setuid and setgid bits
populate_packages relies on ``mkdir`` to both create a directory and set
its permissions. However, ``mkdir`` honors the ``umask`` value.
Therefore, some bits may be lost in the operation. In our case, the
setgid bit on the directories were lost.
This commit fixes this by having a distinct call to create the directory
and to set the permissions.
Signed-off-by: Jean-Tiare Le Bigot <jean-tiare.le-bigot@easymile.com>
Signed-off-by: Armin Kuster <akuster808@gmail.com>
| -rw-r--r-- | meta/classes/package.bbclass | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/meta/classes/package.bbclass b/meta/classes/package.bbclass index 2c44fc1f0d..472d542b11 100644 --- a/meta/classes/package.bbclass +++ b/meta/classes/package.bbclass @@ -1210,7 +1210,8 @@ python populate_packages () { src = os.path.join(src, p) dest = os.path.join(dest, p) fstat = cpath.stat(src) - os.mkdir(dest, fstat.st_mode) + os.mkdir(dest) + os.chmod(dest, fstat.st_mode) os.chown(dest, fstat.st_uid, fstat.st_gid) if p not in seen: seen.append(p) |