Ruediger Pluem wrote:
On 07/28/2006 06:34 PM, [EMAIL PROTECTED] wrote:
Added:
httpd/httpd/branches/httpd-proxy-scoreboard/modules/proxy/health_checker_util.c
URL:
http://svn.apache.org/viewvc/httpd/httpd/branches/httpd-proxy-scoreboard/modules/proxy/health_checker_util.c?rev=426604&view=auto
==============================================================================
---
httpd/httpd/branches/httpd-proxy-scoreboard/modules/proxy/health_checker_util.c
(added)
+++
httpd/httpd/branches/httpd-proxy-scoreboard/modules/proxy/health_checker_util.c
Fri Jul 28 09:33:58 2006
+/* read the entry stored in the shared area and build the corresponding worker
structure */
+static apr_status_t get_entry(int id, proxy_worker **worker, char
**balancer_name, apr_pool_t *pool)
+{
+ struct proxy_worker_conf *workerconf = NULL;
+ apr_status_t rv;
+
+ if (myscore == NULL)
+ return APR_ENOSHMAVAIL;
+ rv = checkstorage->ap_slotmem_mem(myscore, id, (void *) &workerconf);
What should be done if we never have added an worker to this id and thus all
fields
in **workerconf are zero? Wouldn't it make sense to set *worker to NULL and
return
an error (something like NOT_INIT)?
I like the idea but the only error code I have found for it is APR_EINIT.
Any comments?
Cheers
Jean-Frederic
Regards
RĂ¼diger