On 03/18/2014 03:10 PM, Goswin von Brederlow wrote:
Package: unionfs-fuse
Version: 0.24-2.1
Severity: normal
Tags: patch upstream

Hi,

I've compiled unionfs-fuse with extended attribute support and now I'm
getting errors on ls:

ls -lh union/
ls: union/stats: No such file or directory
total 4.0K
-rw-rw-r-- 1 brederlo users    4 Mar 18 14:43 foo
-r--r--r-- 1 root     root  2.0K Jan  1  1970 stats

The reason is that the lgetxattr() callback doesn't handle the stats
file. This is simple to fix with the patch below.

MfG
        Goswin

--

Index: unionfs-fuse-0.24/src/unionfs.c
===================================================================
--- unionfs-fuse-0.24.orig/src/unionfs.c        2014-03-18 15:08:54.526796991 
+0100
+++ unionfs-fuse-0.24/src/unionfs.c     2014-03-18 15:09:01.703071796 +0100
@@ -639,6 +639,10 @@
  static int unionfs_getxattr(const char *path, const char *name, char *value, 
size_t size) {
        DBG_IN();

+       if (uopt.stats_enabled && strcmp(path, STATS_FILENAME) == 0) {
+               return -ENODATA;
+       }
+
        int i = find_rorw_branch(path);
        if (i == -1) return -errno;




I think this is fixed upstream already, but upstream and this patch don't make it the way I want to have it. I hope I find some time over the weekend to test the ioctl patches I started to work on some time ago. And with ioctl support and the new unionfs-fuse-ctl there is no need anymore for the ugly stats hack at all.


--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to