--- libaitio/inc/aitio.h 2012/07/31 16:04:52 1.22.2.15 +++ libaitio/inc/aitio.h 2012/07/31 20:13:33 1.22.2.17 @@ -3,7 +3,7 @@ * by Michael Pounov * * $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.17 2012/07/31 20:13:33 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,22 +310,23 @@ 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); assert((_v)); \ + 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; \ default: \ break; \ } \ } while (0) -#define AIT_COPY_VAL2(_vl, _v) do { AIT_COPY_VAL((_vl), (_v)); \ +#define AIT_COPY_DATA(_vl, _v) do { AIT_COPY_VAL((_vl), (_v)); \ if (AIT_TYPE((_vl)) == data) \ AIT_SET_DATA((_vl), AIT_GET_DATA((_v)), \ AIT_LEN((_v))); \ @@ -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 {