X-Git-Url: http://git.tuebingen.mpg.de/?p=paraslash.git;a=blobdiff_plain;f=blob.c;h=4ecbc45bb15f42c15342e7900f4de94522ef45bb;hp=56986131c73080e00277a99362915319335aebc4;hb=HEAD;hpb=af228cfb66651b0ad846a9b2777a356dd0816871

diff --git a/blob.c b/blob.c
index 56986131..1802de5d 100644
--- a/blob.c
+++ b/blob.c
@@ -100,10 +100,10 @@ static int print_blob(struct osl_table *table, struct osl_row *row,
 	}
 	ret = osl(osl_get_object(table, row, BLOBCOL_ID, &obj));
 	if (ret < 0) {
-		para_printf(&aca->pbout, "cannot list %s\n", name);
+		afs_error(aca, "cannot list %s\n", name);
 		return ret;
 	}
-	id = *(uint32_t *)obj.data;
+	id = read_u32(obj.data);
 	para_printf(&aca->pbout, "%u\t%s\n", id, name);
 	return 1;
 }
@@ -210,7 +210,7 @@ static int remove_blob(struct osl_table *table, struct osl_row *row,
 	int ret = osl(osl_del_row(table, row));
 
 	if (ret < 0) {
-		para_printf(&aca->pbout, "cannot remove %s\n", name);
+		afs_error(aca, "cannot remove %s\n", name);
 		return ret;
 	}
 	return 1;
@@ -266,7 +266,8 @@ static int com_addblob_callback(__a_unused const struct lls_command * const cmd,
 	struct osl_object objs[NUM_BLOB_COLUMNS];
 	char *name = aca->query.data;
 	size_t name_len = strlen(name) + 1;
-	uint32_t id;
+	uint32_t id = (uint32_t)-1; /* STFU, gcc */
+	char id_buf[sizeof(id)];
 	unsigned num_rows;
 	int ret;
 
@@ -274,10 +275,15 @@ static int com_addblob_callback(__a_unused const struct lls_command * const cmd,
 	if (ret < 0)
 		goto out;
 	if (!num_rows) { /* this is the first entry ever added */
-		/* insert dummy row containing the id */
-		id = 2; /* this entry will be entry #1, so 2 is the next */
-		objs[BLOBCOL_ID].data = &id;
-		objs[BLOBCOL_ID].size = sizeof(id);
+		/*
+		 * Insert dummy row containing the next free ID. Since we are
+		 * about to insert the first blob with ID 1, the next free ID
+		 * will be 2.
+		 */
+		id = 2U;
+		write_u32(id_buf, id);
+		objs[BLOBCOL_ID].data = id_buf;
+		objs[BLOBCOL_ID].size = sizeof(id_buf);
 		objs[BLOBCOL_NAME].data = "";
 		objs[BLOBCOL_NAME].size = 1;
 		objs[BLOBCOL_DEF].data = "";
@@ -296,7 +302,7 @@ static int com_addblob_callback(__a_unused const struct lls_command * const cmd,
 			ret = osl(osl_get_object(table, row, BLOBCOL_ID, &obj));
 			if (ret < 0)
 				goto out;
-			id = *(uint32_t *)obj.data;
+			id = read_u32(obj.data);
 			obj.data = name + name_len;
 			obj.size = aca->query.size - name_len;
 			ret = osl(osl_update_object(table, row, BLOBCOL_DEF, &obj));
@@ -311,15 +317,17 @@ static int com_addblob_callback(__a_unused const struct lls_command * const cmd,
 		ret = osl(osl_get_object(table, row, BLOBCOL_ID, &obj));
 		if (ret < 0)
 			goto out;
-		id = *(uint32_t *)obj.data + 1;
-		obj.data = &id;
+		id = read_u32(obj.data) + 1;
+		write_u32(id_buf, id);
+		obj.data = &id_buf;
 		ret = osl(osl_update_object(table, row, BLOBCOL_ID, &obj));
 		if (ret < 0)
 			goto out;
 	}
 	id--;
-	objs[BLOBCOL_ID].data = &id;
-	objs[BLOBCOL_ID].size = sizeof(id);
+	write_u32(id_buf, id);
+	objs[BLOBCOL_ID].data = &id_buf;
+	objs[BLOBCOL_ID].size = sizeof(id_buf);
 	objs[BLOBCOL_NAME].data = name;
 	objs[BLOBCOL_NAME].size = name_len;
 	objs[BLOBCOL_DEF].data = name + name_len;
@@ -330,7 +338,7 @@ static int com_addblob_callback(__a_unused const struct lls_command * const cmd,
 	ret = afs_event(BLOB_ADD, NULL, table);
 out:
 	if (ret < 0)
-		para_printf(&aca->pbout, "cannot add %s\n", name);
+		afs_error(aca, "cannot add %s\n", name);
 	else
 		para_printf(&aca->pbout, "added %s as id %u\n", name, id);
 	return ret;
@@ -377,7 +385,7 @@ again:
  *
  * This function is called from the addblob command handlers to instruct the
  * afs process to store the input in a blob table. Input is read and decrypted
- * from the file descriptor given by cc and appended to arg_obj, which contains
+ * from the file descriptor given by cc and appended to a buffer which also contains
  * the name of the blob to create. The combined buffer is made available to the
  * afs process via the callback method.
  */
@@ -395,7 +403,7 @@ static int stdin_command(struct command_context *cc,
 	if (ret < 0)
 		return ret;
 	query.size = len + 1 + stdin_obj.size;
-	query.data = para_malloc(query.size);
+	query.data = alloc(query.size);
 	memcpy(query.data, lls_input(0, lpr), len + 1);
 	if (stdin_obj.size > 0)
 		memcpy((char *)query.data + len + 1, stdin_obj.data,
@@ -438,15 +446,14 @@ static int com_mvblob_callback(const struct lls_command * const cmd,
 	ret = osl(osl_get_row(table, BLOBCOL_NAME, &obj, &row));
 
 	if (ret < 0) {
-		para_printf(&aca->pbout, "cannot find source blob %s\n", src);
+		afs_error(aca, "cannot find source blob %s\n", src);
 		goto out;
 	}
 	obj.data = (char *)dest;
 	obj.size = strlen(dest) + 1;
 	ret = osl(osl_update_object(table, row, BLOBCOL_NAME, &obj));
 	if (ret < 0) {
-		para_printf(&aca->pbout, "cannot rename blob %s to %s\n",
-			src, dest);
+		afs_error(aca, "cannot rename blob %s to %s\n", src, dest);
 		goto out;
 	}
 	ret = afs_event(BLOB_RENAME, NULL, table);
@@ -512,11 +519,11 @@ static int blob_get_name_by_id(struct osl_table *table, uint32_t id,
 		return blob_get_name_by_id(table_name ## _table, id, name); \
 	}
 
-static int blob_get_def_by_name(struct osl_table *table, char *name,
+static int blob_get_def_by_name(struct osl_table *table, const char *name,
 		struct osl_object *def)
 {
 	struct osl_row *row;
-	struct osl_object obj = {.data = name, .size = strlen(name) + 1};
+	struct osl_object obj = {.data = (void *)name, .size = strlen(name) + 1};
 	int ret;
 
 	def->data = NULL;
@@ -530,7 +537,7 @@ static int blob_get_def_by_name(struct osl_table *table, char *name,
 
 /** Define the \p get_def_by_id function for this blob type. */
 #define DEFINE_GET_DEF_BY_NAME(table_name, cmd_prefix) \
-	int cmd_prefix ## _get_def_by_name(char *name, struct osl_object *def) \
+	int cmd_prefix ## _get_def_by_name(const char *name, struct osl_object *def) \
 	{ \
 		return blob_get_def_by_name(table_name ## _table, name, def); \
 	}
@@ -617,25 +624,21 @@ static int blob_open(struct osl_table **table,
 			&table_name ## _table_desc, dir); \
 	}
 
-
-/** Define the \p init function for this blob type. */
-#define DEFINE_BLOB_INIT(table_name) \
-	void table_name ## _init(struct afs_table *t) \
-	{ \
-		t->open = table_name ## _open; \
-		t->close = table_name ## _close; \
-		t->create = table_name ## _create;\
-		t->event_handler = table_name ##_event_handler; \
-		table_name ## _table = NULL; \
-	}
-
+/** Blob tables map integers to blobs. */
+#define DEFINE_BLOB_AFS_TABLE_OPS(table_name) \
+	const struct afs_table_operations table_name ## _ops = { \
+		.open = table_name ## _open, \
+		.close = table_name ## _close, \
+		.create = table_name ## _create, \
+		.event_handler = table_name ##_event_handler, \
+	};
 
 /** Define all functions for this blob type. */
 #define DEFINE_BLOB_FUNCTIONS(table_name, short_name, c_short_name) \
 	DEFINE_BLOB_OPEN(table_name) \
 	DEFINE_BLOB_CLOSE(table_name) \
 	DEFINE_BLOB_CREATE(table_name) \
-	DEFINE_BLOB_INIT(table_name) \
+	DEFINE_BLOB_AFS_TABLE_OPS(table_name) \
 	DEFINE_BLOB_COMMAND(ls, LS, table_name, short_name, c_short_name) \
 	DEFINE_BLOB_COMMAND(cat, CAT, table_name, short_name, c_short_name) \
 	DEFINE_BLOB_COMMAND(add, ADD, table_name, short_name, c_short_name) \