X-Git-Url: http://git.tuebingen.mpg.de/?p=paraslash.git;a=blobdiff_plain;f=attribute.c;h=4cb2982860de78dd26b9fc55ad552c73a7ca1b9f;hp=4c664996bea7dc869d30ed12e41b4f538fdd5d58;hb=becfb4be64a123630775b6517279ed20e4b498f8;hpb=e4db7671a91a7552c642acc979f0eb278f8d467f

diff --git a/attribute.c b/attribute.c
index 4c664996..4cb29828 100644
--- a/attribute.c
+++ b/attribute.c
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 1997-2011 Andre Noll <maan@systemlinux.org>
+ * Copyright (C) 1997 Andre Noll <maan@tuebingen.mpg.de>
  *
  * Licensed under the GPL v2. For licencing details see COPYING.
  */
@@ -15,8 +15,9 @@
 #include "string.h"
 #include "afh.h"
 #include "afs.h"
-#include "net.h"
 #include "ipc.h"
+#include "sideband.h"
+#include "command.h"
 
 static struct osl_table *attribute_table;
 static int greatest_att_bitnum;
@@ -120,70 +121,63 @@ enum lsatt_flags {
 };
 
 /** Data passed to the action function of lsatt */
-struct lsatt_action_data {
-	/** The result buffer. */
-	struct para_buffer pb;
-	/** The given flags for the lsatt command. */
-	unsigned flags;
-};
-
 static int print_attribute(struct osl_table *table, struct osl_row *row,
 		const char *name, void *data)
 {
-	struct lsatt_action_data *laad = data;
+	struct afs_callback_arg *aca = data;
+	unsigned flags = *(unsigned *)aca->query.data;
 	struct osl_object bitnum_obj;
 	int ret;
 
-	if (!(laad->flags & LSATT_FLAG_LONG))
-		return para_printf(&laad->pb, "%s\n", name);
+	if (!(flags & LSATT_FLAG_LONG)) {
+		para_printf(&aca->pbout, "%s\n", name);
+		return 1;
+	}
 	ret = osl(osl_get_object(table, row, ATTCOL_BITNUM, &bitnum_obj));
 	if (ret < 0) {
-		para_printf(&laad->pb, "%s: %s\n", name, para_strerror(-ret));
+		para_printf(&aca->pbout, "%s: %s\n", name, para_strerror(-ret));
 		return ret;
 	}
-	return para_printf(&laad->pb, "%u\t%s\n", *(unsigned char*)bitnum_obj.data,
+	para_printf(&aca->pbout, "%u\t%s\n", *(unsigned char*)bitnum_obj.data,
 		name);
+	return 1;
 }
 
-static void com_lsatt_callback(int fd, const struct osl_object *query)
+static int com_lsatt_callback(struct afs_callback_arg *aca)
 {
-	struct lsatt_action_data laad = {
-		.flags = *(unsigned *) query->data,
-		.pb = {
-			.max_size = SHMMAX,
-			.private_data = &fd,
-			.max_size_handler = pass_buffer_as_shm
-		}
-
-	};
+	unsigned flags = *(unsigned *)aca->query.data;
+	int ret;
 	struct pattern_match_data pmd = {
 		.table = attribute_table,
-		.loop_col_num = ATTCOL_BITNUM,
+		.loop_col_num = ATTCOL_NAME,
 		.match_col_num = ATTCOL_NAME,
-		.patterns = {.data = (char *)query->data + sizeof(laad.flags),
-			.size = query->size - sizeof(laad.flags)},
+		.patterns = {.data = (char *)aca->query.data + sizeof(flags),
+			.size = aca->query.size - sizeof(flags)},
 		.pm_flags = PM_NO_PATTERN_MATCHES_EVERYTHING,
-		.data = &laad,
+		.data = aca,
 		.action = print_attribute
 	};
-	if (laad.flags & LSATT_FLAG_SORT_BY_ID)
-		pmd.loop_col_num = ATTCOL_NAME;
-	if (laad.flags & LSATT_FLAG_REVERSE)
+	if (flags & LSATT_FLAG_SORT_BY_ID)
+		pmd.loop_col_num = ATTCOL_BITNUM;
+	if (flags & LSATT_FLAG_REVERSE)
 		pmd.pm_flags |= PM_REVERSE_LOOP;
-	for_each_matching_row(&pmd);
-	if (laad.pb.offset)
-		pass_buffer_as_shm(laad.pb.buf, laad.pb.offset, &fd);
-	free(laad.pb.buf);
+	ret = for_each_matching_row(&pmd);
+	if (ret < 0)
+		goto out;
+	if (pmd.num_matches == 0)
+		ret = -E_NO_MATCH;
+out:
+	return ret;
 }
 
-int com_lsatt(struct stream_cipher_context *scc, int argc, char * const * const argv)
+int com_lsatt(struct command_context *cc)
 {
 	unsigned flags = 0;
 	struct osl_object options = {.data = &flags, .size = sizeof(flags)};
-	int ret, i;
+	int i;
 
-	for (i = 1; i < argc; i++) {
-		const char *arg = argv[i];
+	for (i = 1; i < cc->argc; i++) {
+		const char *arg = cc->argv[i];
 		if (arg[0] != '-')
 			break;
 		if (!strcmp(arg, "--")) {
@@ -203,86 +197,8 @@ int com_lsatt(struct stream_cipher_context *scc, int argc, char * const * const
 			continue;
 		}
 	}
-	ret = send_option_arg_callback_request(&options, argc - i, argv + i,
-		com_lsatt_callback, sc_send_result, scc);
-	if (!ret) {
-		if (argc > 1)
-			ret = sc_send_va_buffer(scc, "no matches\n");
-	} else if (ret < 0)
-		sc_send_va_buffer(scc, "%s\n", para_strerror(-ret));
-	return ret;
-}
-
-static void com_setatt_callback(__a_unused int fd, const struct osl_object *query)
-{
-	char *p;
-	uint64_t add_mask = 0, del_mask = 0;
-	int ret;
-	size_t len;
-	struct osl_object obj;
-	struct osl_row *row;
-
-	for (p = query->data; p < (char *)query->data + query->size; p += len + 1) {
-		char c;
-
-		len = strlen(p);
-		ret = -E_ATTR_SYNTAX;
-		if (!*p)
-			goto out;
-		c = p[len - 1];
-		if (c != '+' && c != '-')
-			break;
-		p[len - 1] = '\0';
-		obj.data = p;
-		obj.size = len + 1;
-		ret = osl(osl_get_row(attribute_table, ATTCOL_NAME, &obj, &row));
-		if (ret < 0)
-			goto out;
-		ret = osl(osl_get_object(attribute_table, row, ATTCOL_BITNUM,
-			&obj));
-		if (ret < 0)
-			goto out;
-		if (c == '+')
-			add_mask |= (1UL << *(unsigned char *)obj.data);
-		else
-			del_mask |= (1UL << *(unsigned char *)obj.data);
-	}
-	ret = -E_ATTR_SYNTAX;
-	if (!add_mask && !del_mask)
-		goto out;
-	PARA_DEBUG_LOG("masks: %llx:%llx\n",(long long unsigned)add_mask,
-		(long long unsigned)del_mask);
-	for (; p < (char *)query->data + query->size; p += len + 1) { /* TODO: fnmatch */
-		struct afs_info old_afsi, new_afsi;
-		struct afsi_change_event_data aced = {.old_afsi = &old_afsi};
-
-		len = strlen(p);
-		ret = aft_get_row_of_path(p, &aced.aft_row);
-		if (ret < 0)
-			goto out;
-		ret = get_afsi_object_of_row(aced.aft_row, &obj);
-		if (ret < 0)
-			goto out;
-		ret = load_afsi(&old_afsi, &obj);
-		if (ret < 0)
-			goto out;
-		new_afsi = old_afsi;
-		new_afsi.attributes |= add_mask;
-		new_afsi.attributes &= ~del_mask;
-		save_afsi(&new_afsi, &obj); /* in-place update */
-		afs_event(AFSI_CHANGE, NULL, &aced);
-	}
-out:
-	if (ret < 0)
-		PARA_NOTICE_LOG("%s\n", para_strerror(-ret));
-}
-
-int com_setatt(__a_unused struct stream_cipher_context *scc, int argc, char * const * const argv)
-{
-	if (argc < 3)
-		return -E_ATTR_SYNTAX;
-	return send_standard_callback_request(argc - 1, argv + 1, com_setatt_callback,
-		NULL, NULL);
+	return send_option_arg_callback_request(&options, cc->argc - i, cc->argv + i,
+		com_lsatt_callback, afs_cb_result_handler, cc);
 }
 
 struct addatt_event_data {
@@ -291,18 +207,17 @@ struct addatt_event_data {
 };
 
 
-static void com_addatt_callback(int fd, const struct osl_object *query)
+static int com_addatt_callback(struct afs_callback_arg *aca)
 {
 	char *p;
-	int ret = 1, ret2 = 0;
-	struct para_buffer pb = {
-		.max_size = SHMMAX,
-		.private_data = &fd,
-		.max_size_handler = pass_buffer_as_shm
-	};
+	int ret = 1;
 	size_t len;
 
-	for (p = query->data; p < (char *)query->data + query->size; p += len + 1) {
+	for (
+		p = aca->query.data;
+		p < (char *)aca->query.data + aca->query.size;
+		p += len + 1
+	) {
 		struct osl_object objs[NUM_ATT_COLUMNS];
 		struct osl_row *row;
 		unsigned char bitnum;
@@ -310,16 +225,12 @@ static void com_addatt_callback(int fd, const struct osl_object *query)
 
 		len = strlen(p);
 		if (!len || p[len - 1] == '-' || p[len - 1] == '+') {
-			ret2 = para_printf(&pb, "invalid attribute name: %s\n", p);
-			if (ret2 < 0)
-				goto out;
+			para_printf(&aca->pbout, "invalid attribute name: %s\n", p);
 			continue;
 		}
 		ret = get_attribute_bitnum_by_name(p, &bitnum);
 		if (ret >= 0) {
-			ret2 = para_printf(&pb, "attribute \"%s\" already exists\n", p);
-			if (ret2 < 0)
-				goto out;
+			para_printf(&aca->pbout, "attribute \"%s\" already exists\n", p);
 			continue;
 		}
 		if (ret != -OSL_ERRNO_TO_PARA_ERROR(E_OSL_RB_KEY_NOT_FOUND)) /* error */
@@ -347,42 +258,37 @@ static void com_addatt_callback(int fd, const struct osl_object *query)
 			goto out;
 		aed.name = p;
 		aed.bitnum = bitnum;
-		afs_event(ATTRIBUTE_ADD, &pb, &aed);
+		ret = afs_event(ATTRIBUTE_ADD, &aca->pbout, &aed);
+		if (ret < 0)
+			goto out;
 		greatest_att_bitnum = PARA_MAX(greatest_att_bitnum, (int)bitnum);
 	}
 out:
-	if (ret < 0 && ret2 >= 0)
-		para_printf(&pb, "%s: %s\n", p, para_strerror(-ret));
-	if (pb.offset)
-		pass_buffer_as_shm(pb.buf, pb.offset, &fd);
-	free(pb.buf);
+	if (ret < 0)
+		para_printf(&aca->pbout, "%s: %s\n", p, para_strerror(-ret));
+	return ret;
 }
 
-int com_addatt(struct stream_cipher_context *scc, int argc, char * const * const argv)
+int com_addatt(struct command_context *cc)
 {
 	int ret;
 
-	if (argc < 2)
+	if (cc->argc < 2)
 		return -E_ATTR_SYNTAX;
-	ret = send_standard_callback_request(argc - 1, argv + 1, com_addatt_callback,
-		sc_send_result, scc);
+	ret = send_standard_callback_request(cc->argc - 1, cc->argv + 1,
+		com_addatt_callback, afs_cb_result_handler, cc);
 	if (ret < 0)
-		sc_send_va_buffer(scc, "%s\n", para_strerror(-ret));
+		send_strerror(cc, -ret);
 	return ret;
 }
 
-static void com_mvatt_callback(int fd, const struct osl_object *query)
+static int com_mvatt_callback(struct afs_callback_arg *aca)
 {
-	char *old = query->data;
+	char *old = aca->query.data;
 	size_t size = strlen(old) + 1;
 	char *new = old + size;
 	struct osl_object obj = {.data = old, .size = size};
 	struct osl_row *row;
-	struct para_buffer pb = {
-		.max_size = SHMMAX,
-		.private_data = &fd,
-		.max_size_handler = pass_buffer_as_shm
-	};
 	int ret;
 
 	ret = osl(osl_get_row(attribute_table, ATTCOL_NAME, &obj, &row));
@@ -393,97 +299,67 @@ static void com_mvatt_callback(int fd, const struct osl_object *query)
 	ret = osl(osl_update_object(attribute_table, row, ATTCOL_NAME, &obj));
 out:
 	if (ret < 0)
-		para_printf(&pb, "%s\n", para_strerror(-ret));
+		para_printf(&aca->pbout, "cannot rename %s to %s\n", old, new);
 	else
-		afs_event(ATTRIBUTE_RENAME, &pb, NULL);
-	if (pb.offset)
-		pass_buffer_as_shm(pb.buf, pb.offset, &fd);
-	free(pb.buf);
+		ret = afs_event(ATTRIBUTE_RENAME, &aca->pbout, NULL);
+	return ret;
 }
 
-int com_mvatt(struct stream_cipher_context *scc, int argc, char * const * const argv)
+int com_mvatt(struct command_context *cc)
 {
-	int ret;
-
-	if (argc != 3)
+	if (cc->argc != 3)
 		return -E_ATTR_SYNTAX;
-	ret = send_standard_callback_request(argc - 1, argv + 1, com_mvatt_callback,
-		sc_send_result, scc);
-	if (ret < 0)
-		sc_send_va_buffer(scc, "%s\n", para_strerror(-ret));
-	return ret;
+	return send_standard_callback_request(cc->argc - 1, cc->argv + 1,
+		com_mvatt_callback, afs_cb_result_handler, cc);
 }
 
-/** Data passed to the action handler of com_rmatt(). */
-struct remove_attribute_action_data {
-	/** Message buffer. */
-	struct para_buffer pb;
-	/** Numver of attributes removed. */
-	int num_removed;
-	/** Bitwise "or" of the removed attributes. */
-	uint64_t mask_of_removed_atts;
-};
-
 static int remove_attribute(struct osl_table *table, struct osl_row *row,
 		const char *name, void *data)
 {
-	struct remove_attribute_action_data *raad = data;
+	struct afs_callback_arg *aca = data;
 	int ret;
 	struct rmatt_event_data red = {.name = name};
 
 	ret = get_attribute_bitnum_by_name(name, &red.bitnum);
-	if (ret < 0)
-		return para_printf(&raad->pb, "%s: %s\n", name, para_strerror(-ret));
+	if (ret < 0) {
+		para_printf(&aca->pbout, "cannot remove %s\n", name);
+		return ret;
+	}
+	para_printf(&aca->pbout, "removing attribute %s\n", name);
 	ret = osl(osl_del_row(table, row));
-	if (ret < 0)
-		return para_printf(&raad->pb, "%s: %s\n", name, para_strerror(-ret));
-	ret = para_printf(&raad->pb, "removed attribute %s\n", name);
-	raad->num_removed++;
-	raad->mask_of_removed_atts |= (1 << red.bitnum);
-	afs_event(ATTRIBUTE_REMOVE, &raad->pb, &red);
-	return ret;
+	if (ret < 0) {
+		para_printf(&aca->pbout, "cannot remove %s\n", name);
+		return ret;
+	}
+	return afs_event(ATTRIBUTE_REMOVE, &aca->pbout, &red);
 }
 
-static void com_rmatt_callback(int fd, const struct osl_object *query)
+static int com_rmatt_callback(struct afs_callback_arg *aca)
 {
-	struct remove_attribute_action_data raad = {
-		.num_removed = 0,
-		.pb = {
-			.max_size = SHMMAX,
-			.private_data = &fd,
-			.max_size_handler = pass_buffer_as_shm
-		}
-	};
-	int ret, ret2 = 0;
+	int ret;
 	struct pattern_match_data pmd = {
 		.table = attribute_table,
-		.patterns = *query,
+		.patterns = aca->query,
 		.loop_col_num = ATTCOL_BITNUM,
 		.match_col_num = ATTCOL_NAME,
-		.data = &raad,
+		.data = aca,
 		.action = remove_attribute
 	};
 	ret = for_each_matching_row(&pmd);
 	if (ret < 0)
-		ret2 = para_printf(&raad.pb, "%s\n", para_strerror(-ret));
-	else if (!raad.num_removed)
-		ret2 = para_printf(&raad.pb, "no match -- nothing removed\n");
-	if (ret2 >= 0 && raad.pb.offset)
-		pass_buffer_as_shm(raad.pb.buf, raad.pb.offset, &fd);
-	free(raad.pb.buf);
+		goto out;
+	if (pmd.num_matches == 0)
+		ret = -E_NO_MATCH;
+out:
+	return ret;
 }
 
-int com_rmatt(struct stream_cipher_context *scc, int argc, char * const * const argv)
+int com_rmatt(struct command_context *cc)
 {
-	int ret;
-
-	if (argc < 2)
+	if (cc->argc < 2)
 		return -E_ATTR_SYNTAX;
-	ret = send_standard_callback_request(argc - 1, argv + 1, com_rmatt_callback,
-		sc_send_result, scc);
-	if (ret < 0)
-		sc_send_va_buffer(scc, "%s\n", para_strerror(-ret));
-	return ret;
+	return send_standard_callback_request(cc->argc - 1, cc->argv + 1,
+		com_rmatt_callback, afs_cb_result_handler, cc);
 }
 
 /**
@@ -498,7 +374,7 @@ int com_rmatt(struct stream_cipher_context *scc, int argc, char * const * const
  * used for unset attributes.
  *
  * In practice, not all 64 attributes are defined. In this case, the function
- * only prints \a N + 1 charaters where \a N is the greatest id of a defined
+ * only prints \a N + 1 characters where \a N is the greatest id of a defined
  * attribute.
  */
 void get_attribute_bitmap(const uint64_t *atts, char *buf)
@@ -559,6 +435,47 @@ err:
 	return ret;
 }
 
+static int att_logical_or(struct osl_row *row, void *data)
+{
+	uint64_t *att_mask = data, one = 1;
+	struct osl_object bitnum_obj;
+	int ret = osl_get_object(attribute_table, row, ATTCOL_BITNUM, &bitnum_obj);
+
+	if (ret < 0)
+		return ret;
+	*att_mask |= one << *(unsigned char *)bitnum_obj.data;
+	return 0;
+}
+
+/**
+ * Compute the attribute bit mask and check each afs info bitmap.
+ *
+ * \param aca The query field of \a aca is ignored.
+ *
+ * This iterates over all attributes in the attribute table and computes the
+ * logical or of 1 << b where b is the bit number of the attribute. The
+ * resulting bit mask is passed to aft_check_attributes() which performs the
+ * actual check.
+ *
+ * \return Standard.
+ *
+ * \sa \ref aft_check_attributes().
+ */
+int attribute_check_callback(struct afs_callback_arg *aca)
+{
+	int ret;
+	uint64_t att_mask = 0; /* bits corresponding to a attributes */
+
+	ret = osl_rbtree_loop(attribute_table, ATTCOL_BITNUM, &att_mask,
+		att_logical_or);
+	if (ret < 0) {
+		PARA_ERROR_LOG("attribute table loop failed: %s\n",
+			para_strerror(-ret));
+		return ret;
+	}
+	return aft_check_attributes(att_mask, &aca->pbout);
+}
+
 /**
  * Close the attribute table.
  *
@@ -591,7 +508,7 @@ static int attribute_open(const char *dir)
 		return ret;
 	}
 	attribute_table = NULL;
-	if (ret >= 0 || ret == -OSL_ERRNO_TO_PARA_ERROR(E_OSL_NOENT))
+	if (ret == -OSL_ERRNO_TO_PARA_ERROR(E_OSL_NOENT))
 		return 1;
 	return ret;
 }