--- libaitsess/src/sess.c 2011/09/07 16:08:41 1.3.2.2 +++ libaitsess/src/sess.c 2011/09/07 23:38:39 1.3.2.5 @@ -3,7 +3,7 @@ * by Michael Pounov * * $Author: misho $ -* $Id: sess.c,v 1.3.2.2 2011/09/07 16:08:41 misho Exp $ +* $Id: sess.c,v 1.3.2.5 2011/09/07 23:38:39 misho Exp $ * ************************************************************************** The ELWIX and AITNET software is distributed under the following @@ -340,6 +340,10 @@ sess_prepareSession(tagSess * __restrict s, u_char use sess_SetErr(EINVAL, "Error:: invalid argument\n"); return NULL; } + if (s->addr) { + sess_SetErr(EINVAL, "Error:: already attached memory\n"); + return NULL; + } ATTACH_MEMORY(s); if (!s->addr) @@ -359,8 +363,7 @@ sess_prepareSession(tagSess * __restrict s, u_char use hdr->hdr_argc, useDirect); ADD_SEMAPHORE(s, ret); - if (!s->zcopy) - DETACH_MEMORY(s); + DETACH_MEMORY(s); return arr; } @@ -381,8 +384,6 @@ sess_doneSession(tagSess * __restrict s, array_t ** __ if (!s->zcopy) io_arrayFree(*arr); - else - DETACH_MEMORY(s); io_arrayDestroy(arr); } @@ -404,31 +405,21 @@ sess_commitSession(tagSess * __restrict s, array_t * _ return -1; } - if (!s->zcopy) - ATTACH_MEMORY(s); + ATTACH_MEMORY(s); if (!s->addr) { - if (!s->zcopy) - DETACH_MEMORY(s); - + DETACH_MEMORY(s); return -1; } else hdr = (sess_hdr_t*) s->addr; - if (hdr->hdr_magic != SESS_AIT_MAGIC) { - if (!s->zcopy) - DETACH_MEMORY(s); - sess_SetErr(EINVAL, "Error:: shared memory not contains values with proper format\n"); - return -1; - } - DEC_SEMAPHORE(s, rs); if ((ret = io_vals2map(s->addr + sizeof(sess_hdr_t), s->eom - sizeof(sess_hdr_t), arr)) != -1) { + hdr->hdr_magic = SESS_AIT_MAGIC; hdr->hdr_argc = io_arraySize(arr); ret += sizeof(sess_hdr_t); } ADD_SEMAPHORE(s, rs); - if (!s->zcopy) - DETACH_MEMORY(s); + DETACH_MEMORY(s); return ret; }