svn commit: samba r24343 - in branches/SAMBA_3_0: . source/lib/tdb/common

jelmer at samba.org jelmer at samba.org
Sun Aug 12 02:28:19 GMT 2007


Author: jelmer
Date: 2007-08-12 02:28:15 +0000 (Sun, 12 Aug 2007)
New Revision: 24343

WebSVN: http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=rev&root=samba&rev=24343

Log:
Use standard data type uint32_t rather than tdb-specific u32.

Modified:
   branches/SAMBA_3_0/
   branches/SAMBA_3_0/source/lib/tdb/common/error.c
   branches/SAMBA_3_0/source/lib/tdb/common/io.c
   branches/SAMBA_3_0/source/lib/tdb/common/lock.c
   branches/SAMBA_3_0/source/lib/tdb/common/open.c
   branches/SAMBA_3_0/source/lib/tdb/common/tdb.c
   branches/SAMBA_3_0/source/lib/tdb/common/tdb_private.h
   branches/SAMBA_3_0/source/lib/tdb/common/transaction.c
   branches/SAMBA_3_0/source/lib/tdb/common/traverse.c


Changeset:

Property changes on: branches/SAMBA_3_0
___________________________________________________________________
Name: bzr:revision-info
...skipped...
Name: bzr:file-ids
...skipped...
Name: bzr:revision-id:v3-trunk0
...skipped...

Modified: branches/SAMBA_3_0/source/lib/tdb/common/error.c
===================================================================
--- branches/SAMBA_3_0/source/lib/tdb/common/error.c	2007-08-12 01:49:38 UTC (rev 24342)
+++ branches/SAMBA_3_0/source/lib/tdb/common/error.c	2007-08-12 02:28:15 UTC (rev 24343)
@@ -49,7 +49,7 @@
 /* Error string for the last tdb error */
 const char *tdb_errorstr(struct tdb_context *tdb)
 {
-	u32 i;
+	uint32_t i;
 	for (i = 0; i < sizeof(emap) / sizeof(struct tdb_errname); i++)
 		if (tdb->ecode == emap[i].ecode)
 			return emap[i].estring;

Modified: branches/SAMBA_3_0/source/lib/tdb/common/io.c
===================================================================
--- branches/SAMBA_3_0/source/lib/tdb/common/io.c	2007-08-12 01:49:38 UTC (rev 24342)
+++ branches/SAMBA_3_0/source/lib/tdb/common/io.c	2007-08-12 02:28:15 UTC (rev 24343)
@@ -100,9 +100,9 @@
 }
 
 /* Endian conversion: we only ever deal with 4 byte quantities */
-void *tdb_convert(void *buf, u32 size)
+void *tdb_convert(void *buf, uint32_t size)
 {
-	u32 i, *p = (u32 *)buf;
+	uint32_t i, *p = (uint32_t *)buf;
 	for (i = 0; i < size / 4; i++)
 		p[i] = TDB_BYTEREV(p[i]);
 	return buf;
@@ -143,17 +143,17 @@
   do an unlocked scan of the hash table heads to find the next non-zero head. The value
   will then be confirmed with the lock held
 */		
-static void tdb_next_hash_chain(struct tdb_context *tdb, u32 *chain)
+static void tdb_next_hash_chain(struct tdb_context *tdb, uint32_t *chain)
 {
-	u32 h = *chain;
+	uint32_t h = *chain;
 	if (tdb->map_ptr) {
 		for (;h < tdb->header.hash_size;h++) {
-			if (0 != *(u32 *)(TDB_HASH_TOP(h) + (unsigned char *)tdb->map_ptr)) {
+			if (0 != *(uint32_t *)(TDB_HASH_TOP(h) + (unsigned char *)tdb->map_ptr)) {
 				break;
 			}
 		}
 	} else {
-		u32 off=0;
+		uint32_t off=0;
 		for (;h < tdb->header.hash_size;h++) {
 			if (tdb_ofs_read(tdb, TDB_HASH_TOP(h), &off) != 0 || off != 0) {
 				break;

Modified: branches/SAMBA_3_0/source/lib/tdb/common/lock.c
===================================================================
--- branches/SAMBA_3_0/source/lib/tdb/common/lock.c	2007-08-12 01:49:38 UTC (rev 24342)
+++ branches/SAMBA_3_0/source/lib/tdb/common/lock.c	2007-08-12 02:28:15 UTC (rev 24343)
@@ -401,7 +401,7 @@
 int tdb_unlock_record(struct tdb_context *tdb, tdb_off_t off)
 {
 	struct tdb_traverse_lock *i;
-	u32 count = 0;
+	uint32_t count = 0;
 
 	if (off == 0)
 		return 0;

Modified: branches/SAMBA_3_0/source/lib/tdb/common/open.c
===================================================================
--- branches/SAMBA_3_0/source/lib/tdb/common/open.c	2007-08-12 01:49:38 UTC (rev 24342)
+++ branches/SAMBA_3_0/source/lib/tdb/common/open.c	2007-08-12 02:28:15 UTC (rev 24343)
@@ -35,8 +35,8 @@
 /* This is based on the hash algorithm from gdbm */
 static unsigned int default_tdb_hash(TDB_DATA *key)
 {
-	u32 value;	/* Used to compute the hash value.  */
-	u32   i;	/* Used to cycle through random values. */
+	uint32_t value;	/* Used to compute the hash value.  */
+	uint32_t i;	/* Used to cycle through random values. */
 
 	/* Set the initial value from the key size. */
 	for (value = 0x238F13AF * key->dsize, i=0; i < key->dsize; i++)
@@ -138,7 +138,7 @@
 	struct stat st;
 	int rev = 0, locked = 0;
 	unsigned char *vp;
-	u32 vertest;
+	uint32_t vertest;
 
 	if (!(tdb = (struct tdb_context *)calloc(1, sizeof *tdb))) {
 		/* Can't log this */
@@ -233,8 +233,8 @@
 		rev = (tdb->flags & TDB_CONVERT);
 	}
 	vp = (unsigned char *)&tdb->header.version;
-	vertest = (((u32)vp[0]) << 24) | (((u32)vp[1]) << 16) |
-		  (((u32)vp[2]) << 8) | (u32)vp[3];
+	vertest = (((uint32_t)vp[0]) << 24) | (((uint32_t)vp[1]) << 16) |
+		  (((uint32_t)vp[2]) << 8) | (uint32_t)vp[3];
 	tdb->flags |= (vertest==TDB_VERSION) ? TDB_BIGENDIAN : 0;
 	if (!rev)
 		tdb->flags &= ~TDB_CONVERT;

Modified: branches/SAMBA_3_0/source/lib/tdb/common/tdb.c
===================================================================
--- branches/SAMBA_3_0/source/lib/tdb/common/tdb.c	2007-08-12 01:49:38 UTC (rev 24342)
+++ branches/SAMBA_3_0/source/lib/tdb/common/tdb.c	2007-08-12 02:28:15 UTC (rev 24343)
@@ -63,7 +63,7 @@
 
 /* Returns 0 on fail.  On success, return offset of record, and fills
    in rec */
-static tdb_off_t tdb_find(struct tdb_context *tdb, TDB_DATA key, u32 hash,
+static tdb_off_t tdb_find(struct tdb_context *tdb, TDB_DATA key, uint32_t hash,
 			struct list_struct *r)
 {
 	tdb_off_t rec_ptr;
@@ -90,10 +90,11 @@
 }
 
 /* As tdb_find, but if you succeed, keep the lock */
-tdb_off_t tdb_find_lock_hash(struct tdb_context *tdb, TDB_DATA key, u32 hash, int locktype,
+tdb_off_t tdb_find_lock_hash(struct tdb_context *tdb, TDB_DATA key, 
+							 uint32_t hash, int locktype,
 			   struct list_struct *rec)
 {
-	u32 rec_ptr;
+	uint32_t rec_ptr;
 
 	if (tdb_lock(tdb, BUCKET(hash), locktype) == -1)
 		return 0;
@@ -107,7 +108,7 @@
    is <= the old data size and the key exists.
    on failure return -1.
 */
-static int tdb_update_hash(struct tdb_context *tdb, TDB_DATA key, u32 hash, TDB_DATA dbuf)
+static int tdb_update_hash(struct tdb_context *tdb, TDB_DATA key, uint32_t hash, TDB_DATA dbuf)
 {
 	struct list_struct rec;
 	tdb_off_t rec_ptr;
@@ -146,7 +147,7 @@
 	tdb_off_t rec_ptr;
 	struct list_struct rec;
 	TDB_DATA ret;
-	u32 hash;
+	uint32_t hash;
 
 	/* find which hash bucket it is in */
 	hash = tdb->hash_fn(&key);
@@ -184,7 +185,7 @@
 	tdb_off_t rec_ptr;
 	struct list_struct rec;
 	int ret;
-	u32 hash;
+	uint32_t hash;
 
 	/* find which hash bucket it is in */
 	hash = tdb->hash_fn(&key);
@@ -207,7 +208,7 @@
    this doesn't match the conventions in the rest of this module, but is
    compatible with gdbm
 */
-static int tdb_exists_hash(struct tdb_context *tdb, TDB_DATA key, u32 hash)
+static int tdb_exists_hash(struct tdb_context *tdb, TDB_DATA key, uint32_t hash)
 {
 	struct list_struct rec;
 	
@@ -219,7 +220,7 @@
 
 int tdb_exists(struct tdb_context *tdb, TDB_DATA key)
 {
-	u32 hash = tdb->hash_fn(&key);
+	uint32_t hash = tdb->hash_fn(&key);
 	return tdb_exists_hash(tdb, key, hash);
 }
 
@@ -258,7 +259,7 @@
 	return 0;
 }
 
-static int tdb_count_dead(struct tdb_context *tdb, u32 hash)
+static int tdb_count_dead(struct tdb_context *tdb, uint32_t hash)
 {
 	int res = 0;
 	tdb_off_t rec_ptr;
@@ -283,7 +284,7 @@
 /*
  * Purge all DEAD records from a hash chain
  */
-static int tdb_purge_dead(struct tdb_context *tdb, u32 hash)
+static int tdb_purge_dead(struct tdb_context *tdb, uint32_t hash)
 {
 	int res = -1;
 	struct list_struct rec;
@@ -319,7 +320,7 @@
 }
 
 /* delete an entry in the database given a key */
-static int tdb_delete_hash(struct tdb_context *tdb, TDB_DATA key, u32 hash)
+static int tdb_delete_hash(struct tdb_context *tdb, TDB_DATA key, uint32_t hash)
 {
 	tdb_off_t rec_ptr;
 	struct list_struct rec;
@@ -373,14 +374,14 @@
 
 int tdb_delete(struct tdb_context *tdb, TDB_DATA key)
 {
-	u32 hash = tdb->hash_fn(&key);
+	uint32_t hash = tdb->hash_fn(&key);
 	return tdb_delete_hash(tdb, key, hash);
 }
 
 /*
  * See if we have a dead record around with enough space
  */
-static tdb_off_t tdb_find_dead(struct tdb_context *tdb, u32 hash,
+static tdb_off_t tdb_find_dead(struct tdb_context *tdb, uint32_t hash,
 			       struct list_struct *r, tdb_len_t length)
 {
 	tdb_off_t rec_ptr;
@@ -414,7 +415,7 @@
 int tdb_store(struct tdb_context *tdb, TDB_DATA key, TDB_DATA dbuf, int flag)
 {
 	struct list_struct rec;
-	u32 hash;
+	uint32_t hash;
 	tdb_off_t rec_ptr;
 	char *p = NULL;
 	int ret = -1;
@@ -552,7 +553,7 @@
 /* Append to an entry. Create if not exist. */
 int tdb_append(struct tdb_context *tdb, TDB_DATA key, TDB_DATA new_dbuf)
 {
-	u32 hash;
+	uint32_t hash;
 	TDB_DATA dbuf;
 	int ret = -1;
 

Modified: branches/SAMBA_3_0/source/lib/tdb/common/tdb_private.h
===================================================================
--- branches/SAMBA_3_0/source/lib/tdb/common/tdb_private.h	2007-08-12 01:49:38 UTC (rev 24342)
+++ branches/SAMBA_3_0/source/lib/tdb/common/tdb_private.h	2007-08-12 02:28:15 UTC (rev 24343)
@@ -31,16 +31,12 @@
 #include "system/select.h"
 #include "tdb.h"
 
-#ifndef u32
-#define u32 unsigned
-#endif
-
 #ifndef HAVE_GETPAGESIZE
 #define getpagesize() 0x2000
 #endif
 
-typedef u32 tdb_len_t;
-typedef u32 tdb_off_t;
+typedef uint32_t tdb_len_t;
+typedef uint32_t tdb_off_t;
 
 #ifndef offsetof
 #define offsetof(t,f) ((unsigned int)&((t *)0)->f)
@@ -96,8 +92,8 @@
 	tdb_len_t rec_len; /* total byte length of record */
 	tdb_len_t key_len; /* byte length of key */
 	tdb_len_t data_len; /* byte length of data */
-	u32 full_hash; /* the full 32 bit hash of the key */
-	u32 magic;   /* try to catch errors */
+	uint32_t full_hash; /* the full 32 bit hash of the key */
+	uint32_t magic;   /* try to catch errors */
 	/* the following union is implied:
 		union {
 			char record[rec_len];
@@ -105,7 +101,7 @@
 				char key[key_len];
 				char data[data_len];
 			}
-			u32 totalsize; (tailer)
+			uint32_t totalsize; (tailer)
 		}
 	*/
 };
@@ -114,8 +110,8 @@
 /* this is stored at the front of every database */
 struct tdb_header {
 	char magic_food[32]; /* for /etc/magic */
-	u32 version; /* version of the code */
-	u32 hash_size; /* number of hash entries */
+	uint32_t version; /* version of the code */
+	uint32_t hash_size; /* number of hash entries */
 	tdb_off_t rwlocks; /* obsolete - kept to detect old formats */
 	tdb_off_t recovery_start; /* offset of transaction recovery region */
 	tdb_off_t sequence_number; /* used when TDB_SEQNUM is set */
@@ -124,14 +120,14 @@
 
 struct tdb_lock_type {
 	int list;
-	u32 count;
-	u32 ltype;
+	uint32_t count;
+	uint32_t ltype;
 };
 
 struct tdb_traverse_lock {
 	struct tdb_traverse_lock *next;
-	u32 off;
-	u32 hash;
+	uint32_t off;
+	uint32_t hash;
 	int lock_rw;
 };
 
@@ -139,7 +135,7 @@
 struct tdb_methods {
 	int (*tdb_read)(struct tdb_context *, tdb_off_t , void *, tdb_len_t , int );
 	int (*tdb_write)(struct tdb_context *, tdb_off_t, const void *, tdb_len_t);
-	void (*next_hash_chain)(struct tdb_context *, u32 *);
+	void (*next_hash_chain)(struct tdb_context *, uint32_t *);
 	int (*tdb_oob)(struct tdb_context *, tdb_off_t , int );
 	int (*tdb_expand_file)(struct tdb_context *, tdb_off_t , tdb_off_t );
 	int (*tdb_brlock)(struct tdb_context *, tdb_off_t , int, int, int, size_t);
@@ -157,7 +153,7 @@
 	struct tdb_lock_type *lockrecs; /* only real locks, all with count>0 */
 	enum TDB_ERROR ecode; /* error code for last tdb error */
 	struct tdb_header header; /* a cached copy of the header */
-	u32 flags; /* the flags passed to tdb_open */
+	uint32_t flags; /* the flags passed to tdb_open */
 	struct tdb_traverse_lock travlocks; /* current traversal locks */
 	struct tdb_context *next; /* all tdbs to avoid multiple opens */
 	dev_t device;	/* uniquely identifies this tdb */
@@ -186,7 +182,7 @@
 int tdb_write_unlock_record(struct tdb_context *tdb, tdb_off_t off);
 int tdb_ofs_read(struct tdb_context *tdb, tdb_off_t offset, tdb_off_t *d);
 int tdb_ofs_write(struct tdb_context *tdb, tdb_off_t offset, tdb_off_t *d);
-void *tdb_convert(void *buf, u32 size);
+void *tdb_convert(void *buf, uint32_t size);
 int tdb_free(struct tdb_context *tdb, tdb_off_t offset, struct list_struct *rec);
 tdb_off_t tdb_allocate(struct tdb_context *tdb, tdb_len_t length, struct list_struct *rec);
 int tdb_ofs_read(struct tdb_context *tdb, tdb_off_t offset, tdb_off_t *d);
@@ -202,7 +198,7 @@
 		   int (*parser)(TDB_DATA key, TDB_DATA data,
 				 void *private_data),
 		   void *private_data);
-tdb_off_t tdb_find_lock_hash(struct tdb_context *tdb, TDB_DATA key, u32 hash, int locktype,
+tdb_off_t tdb_find_lock_hash(struct tdb_context *tdb, TDB_DATA key, uint32_t hash, int locktype,
 			   struct list_struct *rec);
 void tdb_io_init(struct tdb_context *tdb);
 int tdb_expand(struct tdb_context *tdb, tdb_off_t size);

Modified: branches/SAMBA_3_0/source/lib/tdb/common/transaction.c
===================================================================
--- branches/SAMBA_3_0/source/lib/tdb/common/transaction.c	2007-08-12 01:49:38 UTC (rev 24342)
+++ branches/SAMBA_3_0/source/lib/tdb/common/transaction.c	2007-08-12 02:28:15 UTC (rev 24343)
@@ -101,7 +101,7 @@
 struct tdb_transaction {
 	/* we keep a mirrored copy of the tdb hash heads here so
 	   tdb_next_hash_chain() can operate efficiently */
-	u32 *hash_heads;
+	uint32_t *hash_heads;
 
 	/* the original io methods - used to do IOs to the real db */
 	const struct tdb_methods *io_methods;
@@ -206,7 +206,7 @@
 	   hash heads */
 	if (len == sizeof(tdb_off_t) && off >= FREELIST_TOP &&
 	    off < FREELIST_TOP+TDB_HASHTABLE_SIZE(tdb)) {
-		u32 chain = (off-FREELIST_TOP) / sizeof(tdb_off_t);
+		uint32_t chain = (off-FREELIST_TOP) / sizeof(tdb_off_t);
 		memcpy(&tdb->transaction->hash_heads[chain], buf, len);
 	}
 
@@ -317,9 +317,9 @@
 /*
   accelerated hash chain head search, using the cached hash heads
 */
-static void transaction_next_hash_chain(struct tdb_context *tdb, u32 *chain)
+static void transaction_next_hash_chain(struct tdb_context *tdb, uint32_t *chain)
 {
-	u32 h = *chain;
+	uint32_t h = *chain;
 	for (;h < tdb->header.hash_size;h++) {
 		/* the +1 takes account of the freelist */
 		if (0 != tdb->transaction->hash_heads[h+1]) {
@@ -440,8 +440,8 @@
 
 	/* setup a copy of the hash table heads so the hash scan in
 	   traverse can be fast */
-	tdb->transaction->hash_heads = (u32 *)
-		calloc(tdb->header.hash_size+1, sizeof(u32));
+	tdb->transaction->hash_heads = (uint32_t *)
+		calloc(tdb->header.hash_size+1, sizeof(uint32_t));
 	if (tdb->transaction->hash_heads == NULL) {
 		tdb->ecode = TDB_ERR_OOM;
 		goto fail;
@@ -572,7 +572,7 @@
 	struct tdb_transaction_el *el;
 	tdb_len_t recovery_size = 0;
 
-	recovery_size = sizeof(u32);
+	recovery_size = sizeof(uint32_t);
 	for (el=tdb->transaction->elements;el;el=el->next) {
 		if (el->offset >= tdb->transaction->old_map_size) {
 			continue;
@@ -678,7 +678,7 @@
 	struct list_struct *rec;
 	tdb_off_t recovery_offset, recovery_max_size;
 	tdb_off_t old_map_size = tdb->transaction->old_map_size;
-	u32 magic, tailer;
+	uint32_t magic, tailer;
 
 	/*
 	  check that the recovery area has enough space
@@ -781,7 +781,7 @@
 {	
 	const struct tdb_methods *methods;
 	tdb_off_t magic_offset = 0;
-	u32 zero = 0;
+	uint32_t zero = 0;
 
 	if (tdb->transaction == NULL) {
 		TDB_LOG((tdb, TDB_DEBUG_ERROR, "tdb_transaction_commit: no transaction\n"));
@@ -934,7 +934,7 @@
 {
 	tdb_off_t recovery_head, recovery_eof;
 	unsigned char *data, *p;
-	u32 zero = 0;
+	uint32_t zero = 0;
 	struct list_struct rec;
 
 	/* find the recovery area */
@@ -988,7 +988,7 @@
 	/* recover the file data */
 	p = data;
 	while (p+8 < data + rec.data_len) {
-		u32 ofs, len;
+		uint32_t ofs, len;
 		if (DOCONV()) {
 			tdb_convert(p, 8);
 		}

Modified: branches/SAMBA_3_0/source/lib/tdb/common/traverse.c
===================================================================
--- branches/SAMBA_3_0/source/lib/tdb/common/traverse.c	2007-08-12 01:49:38 UTC (rev 24342)
+++ branches/SAMBA_3_0/source/lib/tdb/common/traverse.c	2007-08-12 02:28:15 UTC (rev 24343)
@@ -279,7 +279,7 @@
 /* find the next entry in the database, returning its key */
 TDB_DATA tdb_nextkey(struct tdb_context *tdb, TDB_DATA oldkey)
 {
-	u32 oldhash;
+	uint32_t oldhash;
 	TDB_DATA key = tdb_null;
 	struct list_struct rec;
 	unsigned char *k = NULL;



More information about the samba-cvs mailing list