mirror/borg
1
0
Fork 0
mirror of https://github.com/borgbackup/borg.git synced 2025-02-22 14:11:27 +00:00
Code Issues Releases Wiki Activity

fuse: fix crash if empty (None) xattr is read

Browse source
This commit is contained in:
Marian Beermann 2017-05-20 16:28:56 +02:00
parent 5db667881a
commit c6309b148b
2 changed files with 5 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/borg/fuse.py
View file

@ -288,7 +288,7 @@ def listxattr(self, inode, ctx=None):
def getxattr(self, inode, name, ctx=None): def getxattr(self, inode, name, ctx=None):
item = self.get_item(inode) item = self.get_item(inode)
try: try:
return item.get('xattrs', {})[name] return item.get('xattrs', {})[name] or b''
except KeyError: except KeyError:
raise llfuse.FUSEError(llfuse.ENOATTR) from None raise llfuse.FUSEError(llfuse.ENOATTR) from None

5
src/borg/testsuite/archiver.py
View file

@ -326,6 +326,7 @@ def create_test_files(self):
# into "fakeroot space". Because the xattrs exposed by borgfs are these of an underlying file # into "fakeroot space". Because the xattrs exposed by borgfs are these of an underlying file
# (from fakeroots point of view) they are invisible to the test process inside the fakeroot. # (from fakeroots point of view) they are invisible to the test process inside the fakeroot.
xattr.setxattr(os.path.join(self.input_path, 'fusexattr'), 'user.foo', b'bar') xattr.setxattr(os.path.join(self.input_path, 'fusexattr'), 'user.foo', b'bar')
xattr.setxattr(os.path.join(self.input_path, 'fusexattr'), 'user.empty', b'')
# XXX this always fails for me # XXX this always fails for me
# ubuntu 14.04, on a TMP dir filesystem with user_xattr, using fakeroot # ubuntu 14.04, on a TMP dir filesystem with user_xattr, using fakeroot
# same for newer ubuntu and centos. # same for newer ubuntu and centos.
@ -1874,8 +1875,10 @@ def has_noatime(some_file):
in_fn = 'input/fusexattr' in_fn = 'input/fusexattr'
out_fn = os.path.join(mountpoint, 'input', 'fusexattr') out_fn = os.path.join(mountpoint, 'input', 'fusexattr')
if not xattr.XATTR_FAKEROOT and xattr.is_enabled(self.input_path): if not xattr.XATTR_FAKEROOT and xattr.is_enabled(self.input_path):
assert no_selinux(xattr.listxattr(out_fn)) == ['user.foo', ] assert sorted(no_selinux(xattr.listxattr(out_fn))) == ['user.empty', 'user.foo', ]
assert xattr.getxattr(out_fn, 'user.foo') == b'bar' assert xattr.getxattr(out_fn, 'user.foo') == b'bar'
# Special case: getxattr returns None (not b'') when reading an empty xattr.
assert xattr.getxattr(out_fn, 'user.empty') is None
else: else:
assert xattr.listxattr(out_fn) == [] assert xattr.listxattr(out_fn) == []
try: try: