Bug#742033: Support for extended attributes causes errors on ls

2014-03-20 Thread Bernd Schubert

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 users4 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.c2014-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



Bug#742033: Support for extended attributes causes errors on ls

2014-03-20 Thread Radek Podgorny
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

hi goswin!

thank you very much for your patch! unfortunately, at least on my
system, the problem was also in the listxattr. so, i've patched all
*xattr functions to return 'not supported' for the stats file.

please let me know if this does not fix the error for you.

also, i looks like you're using old version of unionfs-fuse. see my hg
repo at http://hg.podgorny.cz/unionfs-fuse for the latest code.

thanks,
R.


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 users4 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;
 
 
-BEGIN PGP SIGNATURE-
Version: GnuPG v2.0.22 (GNU/Linux)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/

iEYEARECAAYFAlMrNa4ACgkQ7mej6pjlbYQ7vgCfT9MYNKVWThhw/VOW3UPPYoCd
qE4An2fMXU0XM6afMaAQkHeMFmexCdlh
=dq6m
-END PGP SIGNATURE-


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



Bug#742033: Support for extended attributes causes errors on ls

2014-03-18 Thread Goswin von Brederlow
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 users4 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.c2014-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;
 


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