diff options
| author | Kotresh HR <khiremat@redhat.com> | 2019-06-06 12:54:04 +0530 | 
|---|---|---|
| committer | Xavi Hernandez <xhernandez@redhat.com> | 2019-06-27 08:40:27 +0200 | 
| commit | 5de46c74b281e6d0cf7168f8869bd179e7fff489 (patch) | |
| tree | 8022fcd76a8ead0e6edd13e400bbe1ab597e3b4a /tests/utils/libcxattr.py | |
| parent | 9acd89087baea32b06142d51568df78e0ce57d54 (diff) | |
tests/utils: Fix py2/py3 util python scripts
Following files are fixed.
tests/bugs/distribute/overlap.py
tests/utils/changelogparser.py
tests/utils/create-files.py
tests/utils/gfid-access.py
tests/utils/libcxattr.py
Backport of:
> Change-Id: I3db857cc19e19163d368d913eaec1269fbc37140
> BUG: 1193929
> Signed-off-by: Kotresh HR <khiremat@redhat.com>
Change-Id: I3db857cc19e19163d368d913eaec1269fbc37140
updates: bz#1679998
Signed-off-by: Kotresh HR <khiremat@redhat.com>
Diffstat (limited to 'tests/utils/libcxattr.py')
| -rw-r--r-- | tests/utils/libcxattr.py | 22 | 
1 files changed, 14 insertions, 8 deletions
diff --git a/tests/utils/libcxattr.py b/tests/utils/libcxattr.py index fd0b08378fc..3f3ed1fffbb 100644 --- a/tests/utils/libcxattr.py +++ b/tests/utils/libcxattr.py @@ -10,7 +10,9 @@  import os  import sys -from ctypes import CDLL, c_int, create_string_buffer +from ctypes import CDLL, c_int +from py2py3 import bytearray_to_str, gr_create_string_buffer +from py2py3 import gr_query_xattr, gr_lsetxattr, gr_lremovexattr  class Xattr(object): @@ -47,20 +49,23 @@ class Xattr(object):      @classmethod      def _query_xattr(cls, path, siz, syscall, *a):          if siz: -            buf = create_string_buffer('\0' * siz) +            buf = gr_create_string_buffer(siz)          else:              buf = None          ret = getattr(cls.libc, syscall)(*((path,) + a + (buf, siz)))          if ret == -1:              cls.raise_oserr()          if siz: -            return buf.raw[:ret] +            # py2 and py3 compatibility. Convert bytes array +            # to string +            result = bytearray_to_str(buf.raw) +            return result[:ret]          else:              return ret      @classmethod      def lgetxattr(cls, path, attr, siz=0): -        return cls._query_xattr(path, siz, 'lgetxattr', attr) +        return gr_query_xattr(cls, path, siz, 'lgetxattr', attr)      @classmethod      def lgetxattr_buf(cls, path, attr): @@ -74,20 +79,21 @@ class Xattr(object):      @classmethod      def llistxattr(cls, path, siz=0): -        ret = cls._query_xattr(path, siz, 'llistxattr') +        ret = gr_query_xattr(cls, path, siz, 'llistxattr')          if isinstance(ret, str): -            ret = ret.split('\0') +            ret = ret.strip('\0') +            ret = ret.split('\0') if ret else []          return ret      @classmethod      def lsetxattr(cls, path, attr, val): -        ret = cls.libc.lsetxattr(path, attr, val, len(val), 0) +        ret = gr_lsetxattr(cls, path, attr, val)          if ret == -1:              cls.raise_oserr()      @classmethod      def lremovexattr(cls, path, attr): -        ret = cls.libc.lremovexattr(path, attr) +        ret = gr_lremovexattr(cls, path, attr)          if ret == -1:              cls.raise_oserr()  | 
