X-Git-Url: http://git.tuebingen.mpg.de/?a=blobdiff_plain;f=osl.c;h=c5f70ffa1af9bfa5f57c29f5c3ee4a8ce2462b98;hb=24b877cf29cefb2d8b02029e2f10ccbaccf472f5;hp=0a2863dd2a7e3d48463448bca426693b391d6629;hpb=188bc1b06879b7fcd56c599e20db0ac87a0da4e3;p=osl.git

diff --git a/osl.c b/osl.c
index 0a2863d..c5f70ff 100644
--- a/osl.c
+++ b/osl.c
@@ -13,7 +13,14 @@
 #include "util.h"
 #include "osl_core.h"
 
-/* Taken from Drepper: How to write shared libraries, Appendix B. */
+/*
+ * Taken from Drepper: How to write shared libraries, Appendix B.
+ *
+ * The main reason for this rather fancy implementation of strerror() is to
+ * avoid having an array of pointers. This is desirable because initialized
+ * pointer variables increase the startup time of the library due to the
+ * processing of relocations.
+ */
 #include <stddef.h>
 #define MSGSTRFIELD(line) MSGSTRFIELD1(line)
 #define MSGSTRFIELD1(line) str##line
@@ -277,6 +284,9 @@ int init_table_structure(const struct osl_table_description *desc,
 		ret = -E_OSL_BAD_STORAGE_FLAGS;
 		if (st == OSL_DISK_STORAGE && sf & OSL_RBTREE)
 			goto err;
+		if ((sf & OSL_RBTREE) && !(sf & OSL_UNIQUE))
+			WARNING_LOG("invalid storage flags for column %s: "
+				"OSL_RBTREE && !OSL_UNIQUE\n", cd->name);
 		ret = -E_OSL_BAD_STORAGE_SIZE;
 		if (sf & OSL_FIXED_SIZE && !cd->data_size)
 			goto err;
@@ -339,7 +349,7 @@ err:
 int read_table_desc(struct osl_object *map, struct osl_table_description *desc)
 {
 	char *buf = map->data;
-	uint8_t version, compat_version, create_version;
+	uint8_t table_version;
 	uint16_t header_size;
 	int ret, i;
 	unsigned offset;
@@ -349,20 +359,11 @@ int read_table_desc(struct osl_object *map, struct osl_table_description *desc)
 		return -E_OSL_SHORT_TABLE;
 	if (strncmp(buf + IDX_OSL_MAGIC, OSL_MAGIC, strlen(OSL_MAGIC)))
 		return -E_OSL_NO_MAGIC;
-	version = read_u8(buf + IDX_VERSION);
-	/*
-	 * The on-disk version consists of two version numbers: the
-	 * create_version (low 4 bits) is the CURRENT_TABLE_VERSION version
-	 * number of the library that created the table, and compat_version
-	 * (high 4 bits) tells us the lowest version of the library that can
-	 * still read this table.
-	 */
-	create_version = version & 0xf;
-	compat_version = version >> 4;
-	INFO_LOG("create_version: %u, compat_version: %u\n", create_version,
-		compat_version);
-	if (create_version < MIN_TABLE_VERSION /* table too old */
-		|| compat_version > CURRENT_TABLE_VERSION) /* libosl too old */
+	table_version = read_u8(buf + IDX_VERSION);
+	INFO_LOG("osl versions (table/min/current): %u/%u/%u\n",
+		table_version, MIN_TABLE_VERSION, CURRENT_TABLE_VERSION);
+	if (table_version < MIN_TABLE_VERSION /* table too old */
+		|| table_version > CURRENT_TABLE_VERSION) /* libosl too old */
 		return -E_OSL_VERSION_MISMATCH;
 	desc->flags = read_u8(buf + IDX_TABLE_FLAGS);
 	desc->num_columns = read_u16(buf + IDX_NUM_COLUMNS);
@@ -497,8 +498,7 @@ static int create_table_index(struct osl_table *t)
 	sprintf(buf + IDX_OSL_MAGIC, "%s", OSL_MAGIC);
 	write_u8(buf + IDX_TABLE_FLAGS, t->desc->flags);
 	write_u8(buf + IDX_DIRTY_FLAG, 0);
-	write_u8(buf + IDX_VERSION, CURRENT_TABLE_VERSION
-		+ (COMPAT_TABLE_VERSION << 4));
+	write_u8(buf + IDX_VERSION, CURRENT_TABLE_VERSION);
 	write_u16(buf + IDX_NUM_COLUMNS, t->num_mapped_columns + t->num_disk_storage_columns);
 	write_u16(buf + IDX_HEADER_SIZE, t->index_header_size);
 	offset = IDX_COLUMN_DESCRIPTIONS;
@@ -516,7 +516,7 @@ static int create_table_index(struct osl_table *t)
 		strcpy(buf + offset + IDX_CD_NAME, cd->name);
 		offset += index_column_description_size(cd->name);
 	}
-	assert(offset = size);
+	assert(offset == size);
 	filename = index_filename(t->desc);
 	if (filename)
 		ret = write_file(filename, buf, size);
@@ -1569,7 +1569,8 @@ __export int osl_update_object(struct osl_table *t, const struct osl_row *r,
 	}
 	remove_rb_node(t, col_num, r);
 	if (cd->storage_type == OSL_NO_STORAGE) { /* TODO: If fixed size, reuse object? */
-		free(r->volatile_objects[col->volatile_num].data);
+		if (!(cd->storage_flags & OSL_DONT_FREE))
+			free(r->volatile_objects[col->volatile_num].data);
 		r->volatile_objects[col->volatile_num] = *obj;
 	} else if (cd->storage_type == OSL_DISK_STORAGE) {
 		char *ds_name;
@@ -1698,6 +1699,7 @@ __export int osl_get_nth_row(const struct osl_table *t, unsigned col_num,
 	unsigned num_rows;
 	int ret;
 
+	*result = NULL;
 	if (n == 0)
 		return -E_OSL_RB_KEY_NOT_FOUND;
 	ret = osl_get_num_rows(t, &num_rows);