aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPeter Seebach <peter.seebach@windriver.com>2016-03-25 15:39:05 -0500
committerPeter Seebach <peter.seebach@windriver.com>2016-03-25 15:39:05 -0500
commit099c6b1a77b0d5a11b7c23ec49785dc255c37930 (patch)
treeb11b28cfab0dfd4993d90a4ee0093788c99787ab
parenta6cb08e927ff915ec06c17c1cea2bdbd54637f84 (diff)
downloadpseudo-099c6b1a77b0d5a11b7c23ec49785dc255c37930.tar.gz
pseudo-099c6b1a77b0d5a11b7c23ec49785dc255c37930.tar.bz2
pseudo-099c6b1a77b0d5a11b7c23ec49785dc255c37930.zip
actually handle remove_xattr correctly
The variable name is required but wasn't being extracted from the client's message, resulting in xattr removal never working. This does not fully address some deeper problems with the xattr implementation, but at least the common removal case works.
-rw-r--r--ChangeLog.txt4
-rw-r--r--pseudo.c1
2 files changed, 5 insertions, 0 deletions
diff --git a/ChangeLog.txt b/ChangeLog.txt
index 1c1419f..66a3a13 100644
--- a/ChangeLog.txt
+++ b/ChangeLog.txt
@@ -1,3 +1,7 @@
+2016-03-22:
+ * (seebs) extract variable name from message for remove_xattr so it
+ actually works.
+
2016-03-11:
* (seebs) set sane umask in server, just in case.
diff --git a/pseudo.c b/pseudo.c
index 2b60fd4..67ebb6a 100644
--- a/pseudo.c
+++ b/pseudo.c
@@ -484,6 +484,7 @@ pseudo_op(pseudo_msg_t *msg, const char *program, const char *tag, char **respon
case OP_GET_XATTR:
case OP_LIST_XATTR:
case OP_REPLACE_XATTR:
+ case OP_REMOVE_XATTR:
case OP_SET_XATTR:
/* In a rename there are two paths, null separated in msg->path */
initial_len = strlen(msg->path);