Author: jfrederic.clere(a)jboss.com Date: 2009-05-13 15:43:48 -0400 (Wed, 13 May 2009) New Revision: 2421 Modified: trunk/mod_cluster/native/include/slotmem.h trunk/mod_cluster/native/mod_slotmem/sharedmem_util.c Log: Small errors detected while reviewing the code. Modified: trunk/mod_cluster/native/include/slotmem.h =================================================================== --- trunk/mod_cluster/native/include/slotmem.h 2009-05-04 11:57:23 UTC (rev 2420) +++ trunk/mod_cluster/native/include/slotmem.h 2009-05-13 19:43:48 UTC (rev 2421) @@ -122,9 +122,10 @@ */ apr_status_t (* ap_slotmem_free)(ap_slotmem_t *s, int item_id, void*mem); /** - * Lock the slotmem. + * Return the used id in the slotmem table. * @param s ap_slotmem_t to use. - * @return APR_SUCCESS if all went well + * @param ids array of int where to store the free idents. + * @return number of slotmem in use. */ int (*ap_slotmem_get_used)(ap_slotmem_t *s, int *ids); /** Modified: trunk/mod_cluster/native/mod_slotmem/sharedmem_util.c =================================================================== --- trunk/mod_cluster/native/mod_slotmem/sharedmem_util.c 2009-05-04 11:57:23 UTC (rev 2420) +++ trunk/mod_cluster/native/mod_slotmem/sharedmem_util.c 2009-05-13 19:43:48 UTC (rev 2421) @@ -214,7 +214,7 @@ } if (!isfree) { rv = func((void *)ptr, data, i, pool); - if (rv == APR_SUCCESS) + if (rv != APR_SUCCESS) return(rv); } ptr = ptr + mem->size;