[buildstream] 16/19: plugins/sources/tar.py: Ensure read permissions for groups as well

[ro...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

root pushed a commit to branch tlater/casd-socket-permissions
in repository https://gitbox.apache.org/repos/asf/buildstream.git

commit 397ddcfe7a728aec06307ee24f7c8be05130f490
Author: Tristan Maat <tr...@codethink.co.uk>
AuthorDate: Fri Nov 8 12:24:43 2019 +0000

    plugins/sources/tar.py: Ensure read permissions for groups as well
---
 src/buildstream/plugins/sources/tar.py | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/buildstream/plugins/sources/tar.py b/src/buildstream/plugins/sources/tar.py
index 702b7ba..ac80177 100644
--- a/src/buildstream/plugins/sources/tar.py
+++ b/src/buildstream/plugins/sources/tar.py
@@ -69,14 +69,14 @@ from ._downloadablefilesource import DownloadableFileSource
 class ReadableTarInfo(tarfile.TarInfo):
     """
            The goal is to override `TarFile`'s `extractall` semantics by ensuring that on extraction, the
-           files are readable by the owner of the file. This is done by overriding the accessor for the
+           files are readable by the owners of the file. This is done by overriding the accessor for the
            `mode` attribute in `TarInfo`, the class that encapsulates the internal meta-data of the tarball,
            so that the owner-read bit is always set.
     """
     @property
     def mode(self):
-        # ensure file is readable by owner
-        return self.__permission | 0o400
+        # ensure file is readable by owner and group
+        return self.__permission | 0o440
 
     @mode.setter
     def mode(self, permission):