Re: smsbox and mutex detailed analysis

Benjamin Lee wrote:


> 2002-02-18 05:15:28 [3] PANIC: gwlib/conn.c:174: unlock_out_real: Mutex unlock failed. (Called from gwlib/conn.c:793:conn_register.)


This is probably the bug I mentioned a couple of weeks ago. As I said 
before, it can be fixed by changing the order of the unlock statements 
at the end of conn_register, or (a cleaner way) by adding a 
lock_out/unlock_out pair at the start of conn_destroy.

Btw, someone who has had problems with CPU use from HTTP requests could 
try whether changing the "if (revents & POLLIN) {" line to
"if (revents & (POLLIN | POLLERR)) {" in conn.c/poll_callback affects 
that. The latter is what it says in my improved version of conn.c and 
I've never seen the CPU use problem, but I don't think I saw it with the 
old version either, so it would be nice to know whether that fixes it.