--- libaitio/inc/aitio.h	2012/07/31 16:04:52	1.22.2.15
+++ libaitio/inc/aitio.h	2012/07/31 16:42:38	1.22.2.16
@@ -3,7 +3,7 @@
 *  by Michael Pounov <misho@elwix.org>
 *
 * $Author: misho $
-* $Id: aitio.h,v 1.22.2.15 2012/07/31 16:04:52 misho Exp $
+* $Id: aitio.h,v 1.22.2.16 2012/07/31 16:42:38 misho Exp $
 *
 **************************************************************************
 The ELWIX and AITNET software is distributed under the following
@@ -182,10 +182,10 @@ typedef struct {
 
 #define AIT_SET_DATA(_vl, _p, _len)	do { ait_val_t *__val = io_realloc((_vl), (sizeof(ait_val_t) + _len)); \
 						if (__val) { \
+							void *__p = (_p); \
+							if (__p) \
+								memcpy(__val->val_data, __p, _len); \
 							__val->val_type = data; AIT_LEN(__val) = _len; \
-							if ((_p)) { \
-								memcpy(__val->val_data, (_p), _len); \
-							} \
 							(_vl) = __val; \
 						} \
 					} while (0);
@@ -203,19 +203,19 @@ typedef struct {
 							__val->val_type = buffer; AIT_LEN(__val) = _len; \
 							memset(__val->val.buffer, _c, _len); \
 						} } while (0)
-#define AIT_SET_BUF(_vl, _v, _len)	do { ait_val_t *__val = (_vl); assert(__val); \
+#define AIT_SET_BUF(_vl, _v, _len)	do { ait_val_t *__val = (_vl); void *__p = (_v); assert(__val); \
 						__val->val.buffer = io_malloc(_len); \
 						if (__val->val.buffer) { \
 							__val->val_type = buffer; AIT_LEN(__val) = _len; \
-							if ((_v)) \
-								memcpy(__val->val.buffer, (_v), _len); \
+							if (__p) \
+								memcpy(__val->val.buffer, __p, _len); \
 							else \
 								memset(__val->val.buffer, 0, _len); \
 						} } while (0)
-#define AIT_SET_STR(_vl, _v)		do { ait_val_t *__val = (_vl); assert(__val); \
+#define AIT_SET_STR(_vl, _v)		do { ait_val_t *__val = (_vl); const char *__s = (_v); assert(__val); \
 						__val->val_type = string; \
-						if ((_v) && *(_v)) { \
-							__val->val.string = (int8_t*) io_strdup((_v)); \
+						if (__s && *__s) { \
+							__val->val.string = (int8_t*) io_strdup(__s); \
 							AIT_LEN(__val) = strlen((const char*) \
 									__val->val.string) + 1; \
 						} else { \
@@ -230,39 +230,44 @@ typedef struct {
 							memset(__val->val.string, 0, _len); \
 						} \
 					} while (0)
-#define AIT_SET_STRCAT(_vl, _v)		do { ait_val_t *__val = (_vl); assert(__val); \
-						assert(AIT_TYPE(__val) == string); \
+#define AIT_SET_STRCAT(_vl, _v)		do { ait_val_t *__val = (_vl); const char *__s = (_v); int __l; \
+						assert(__val); assert(AIT_TYPE(__val) == string); \
+						if (!__s || !*__s) \
+							break; \
+						else \
+							__l = strlen(__s); \
 						if (!__val->val.string) \
-							AIT_LEN(__val) = 1; \
-						void *__p = io_realloc(__val->val.string, \
-								AIT_LEN(__val) + strlen((_v))); \
+							__l++; \
+						void *__p = io_realloc(__val->val.string, AIT_LEN(__val) + __l); \
 						if (__p) { \
-							AIT_LEN(__val) += strlen((_v)); \
-							if (!__val->val.string) \
-								memset(__p, 0, AIT_LEN(__val)); \
+							AIT_LEN(__val) += __l; \
 							__val->val.string = __p; \
-							strlcat((char*) __val->val.string, (_v), \
+							strlcat((char*) __val->val.string, __s, \
 									AIT_LEN(__val)); \
-						} else \
-							if (!__val->val.string) \
-								AIT_LEN(__val) = 0; \
+						} \
 					} while (0)
-#define AIT_SET_STRCPY(_vl, _v)		do { ait_val_t *__val = (_vl); assert(__val); \
-						assert(AIT_TYPE(__val) == string); \
-						void *__p = io_realloc(__val->val.string, \
-								strlen((_v)) + 1); \
+#define AIT_SET_STRCPY(_vl, _v)		do { ait_val_t *__val = (_vl); const char *__s = (_v); int __l; \
+						assert(__val); assert(AIT_TYPE(__val) == string); \
+						if (!__s || !*__s) \
+							break; \
+						else \
+							__l = strlen(__s) + 1; \
+						void *__p = io_realloc(__val->val.string, __l); \
 						if (__p) { \
+							AIT_LEN(__val) = __l; \
 							__val->val.string = __p; \
-							AIT_LEN(__val) = strlen((_v)) + 1; \
-							strlcpy((char*) __val->val.string, (_v), \
+							strlcpy((char*) __val->val.string, __s, \
 									AIT_LEN(__val)); \
 						} } while (0)
-#define AIT_SET_STRLCPY(_vl, _v, _len)	do { ait_val_t *__val = (_vl); assert(__val); \
-						assert(AIT_TYPE(__val) == string); \
+#define AIT_SET_STRLCPY(_vl, _v, _len)	do { ait_val_t *__val = (_vl); const char *__s = (_v); \
+						assert(__val); assert(AIT_TYPE(__val) == string); \
+						if (!__s || !*__s) \
+							break; \
 						void *__p = io_realloc(__val->val.string, _len); \
 						if (__p) { \
-							__val->val.string = __p; AIT_LEN(__val) = _len; \
-							strlcpy((char*) __val->val.string, (_v), \
+							AIT_LEN(__val) = _len; \
+							__val->val.string = __p; \
+							strlcpy((char*) __val->val.string, __s, \
 									AIT_LEN(__val)); \
 						} } while (0)
 #define AIT_SET_BLOB(_vl, _n, _len)	do { ait_val_t *__val = (_vl); assert(__val); \
@@ -305,26 +310,27 @@ typedef struct {
 						__val->val_type = f64; __val->val.f64 = _n; \
 						AIT_LEN(__val) = sizeof(double); } while (0)
 
-#define AIT_COPY_VAL(_vl, _v)		do { memcpy((_vl), (_v), sizeof(ait_val_t)); \
-						switch (AIT_TYPE((_vl))) { \
+#define AIT_COPY_VAL(_vl, _v)		do { ait_val_t *__val = (_vl); assert(__val); \
+						memcpy(__val, (_v), sizeof(ait_val_t)); \
+						switch (AIT_TYPE(__val)) { \
 							case buffer: \
-								AIT_SET_BUF((_vl), \
+								AIT_SET_BUF(__val, \
 										AIT_GET_BUF((_v)), \
 										AIT_LEN((_v))); \
 								break; \
 							case string: \
-								AIT_SET_STR((_vl), \
+								AIT_SET_STR(__val, \
 										AIT_GET_STR((_v))); \
 								break; \
+							case data: \
+								AIT_SET_DATA(__val, \
+										AIT_GET_DATA((_v)), \
+										AIT_LEN((_v))); \
+								break; \
 							default: \
 								break; \
 						} \
 					} while (0)
-#define AIT_COPY_VAL2(_vl, _v)		do { AIT_COPY_VAL((_vl), (_v)); \
-						if (AIT_TYPE((_vl)) == data) \
-							AIT_SET_DATA((_vl), AIT_GET_DATA((_v)), \
-									AIT_LEN((_v))); \
-					} while (0)
 
 #define AIT_INIT_VAL(_vl)		(memset((_vl), 0, sizeof(ait_val_t)))
 #define AIT_INIT_VAL2(_vl, _t)		do { \
@@ -362,15 +368,14 @@ typedef struct {
 									memset(__val->val.buffer, 0, \
 										AIT_LEN(__val)); \
 								break; \
+							case data: \
+								memset(__val->val_data, 0, AIT_LEN(__val)); \
+								break; \
 							default: \
 								__val->val.net = 0LL; \
 								break; \
 						} \
 						AIT_KEY(__val) = 0; \
-					} while (0)
-#define AIT_ZERO_VAL2(_vl)		do { assert((_vl)); \
-						if (AIT_TYPE((_vl)) == data) \
-							memset((_vl)->val_data, 0, AIT_LEN((_vl))); \
 					} while (0)
 
 struct io_ether_addr {