X-Git-Url: http://git.indexdata.com/?a=blobdiff_plain;f=src%2Fhttp.c;h=8321bb3b818f4966738cfa19af23431191534906;hb=f9fd1939f30660831f6390a91839357133e005ea;hp=a12fb4e884794dc3eacccfe47892dcd667c0494d;hpb=c453b003094541adc6fc155bc9a7aecf788245bc;p=pazpar2-moved-to-github.git diff --git a/src/http.c b/src/http.c index a12fb4e..8321bb3 100644 --- a/src/http.c +++ b/src/http.c @@ -63,7 +63,8 @@ typedef int socklen_t; #include #include -#include "pazpar2.h" +#include "ppmutex.h" +#include "session.h" #include "http.h" #define MAX_HTTP_HEADER 4096 @@ -87,7 +88,7 @@ static void proxy_io(IOCHAN i, int event); static struct http_channel *http_channel_create(http_server_t http_server, const char *addr, struct conf_server *server); -static void http_destroy(IOCHAN i); +static void http_channel_destroy(IOCHAN i); static http_server_t http_server_create(void); static void http_server_incref(http_server_t hs); @@ -797,9 +798,10 @@ static int http_proxy(struct http_request *rq) p->first_response = 1; c->proxy = p; // We will add EVENT_OUTPUT below - p->iochan = iochan_create(sock, proxy_io, EVENT_INPUT); + p->iochan = iochan_create(sock, proxy_io, EVENT_INPUT, "http_proxy"); iochan_setdata(p->iochan, p); - pazpar2_add_channel(p->iochan); + + iochan_add(ser->iochan_man, p->iochan); } // Do _not_ modify Host: header, just checking it's existence @@ -847,7 +849,7 @@ void http_send_response(struct http_channel *ch) if (!hb) { yaz_log(YLOG_WARN, "Failed to serialize HTTP response"); - http_destroy(ch->iochan); + http_channel_destroy(ch->iochan); } else { @@ -893,7 +895,7 @@ static void http_io(IOCHAN i, int event) if (res <= 0) { http_buf_destroy(hc->http_server, htbuf); - http_destroy(i); + http_channel_destroy(i); return; } htbuf->buf[res] = '\0'; @@ -940,7 +942,7 @@ static void http_io(IOCHAN i, int event) if (res <= 0) { yaz_log(YLOG_WARN|YLOG_ERRNO, "write"); - http_destroy(i); + http_channel_destroy(i); return; } if (res == wb->len) @@ -956,7 +958,7 @@ static void http_io(IOCHAN i, int event) if (!hc->oqueue) { if (!hc->keep_alive) { - http_destroy(i); + http_channel_destroy(i); return; } else @@ -969,11 +971,11 @@ static void http_io(IOCHAN i, int event) } if (!hc->oqueue && hc->proxy && !hc->proxy->iochan) - http_destroy(i); // Server closed; we're done + http_channel_destroy(i); // Server closed; we're done break; default: yaz_log(YLOG_WARN, "Unexpected event on connection"); - http_destroy(i); + http_channel_destroy(i); } } @@ -1008,7 +1010,7 @@ static void proxy_io(IOCHAN pi, int event) } else { - http_destroy(hc->iochan); + http_channel_destroy(hc->iochan); return; } } @@ -1033,7 +1035,7 @@ static void proxy_io(IOCHAN pi, int event) if (res <= 0) { yaz_log(YLOG_WARN|YLOG_ERRNO, "write"); - http_destroy(hc->iochan); + http_channel_destroy(hc->iochan); return; } if (res == htbuf->len) @@ -1054,7 +1056,7 @@ static void proxy_io(IOCHAN pi, int event) break; default: yaz_log(YLOG_WARN, "Unexpected event on connection"); - http_destroy(hc->iochan); + http_channel_destroy(hc->iochan); } } @@ -1062,7 +1064,7 @@ static void http_fire_observers(struct http_channel *c); static void http_destroy_observers(struct http_channel *c); // Cleanup channel -static void http_destroy(IOCHAN i) +static void http_channel_destroy(IOCHAN i) { struct http_channel *s = iochan_getdata(i); http_server_t http_server; @@ -1129,6 +1131,7 @@ static struct http_channel *http_channel_create(http_server_t hs, http_server_incref(hs); r->http_server = hs; r->http_sessions = hs->http_sessions; + assert(r->http_sessions); r->server = server; r->proxy = 0; r->iochan = 0; @@ -1168,14 +1171,13 @@ static void http_accept(IOCHAN i, int event) enable_nonblock(s); yaz_log(YLOG_DEBUG, "New command connection"); - c = iochan_create(s, http_io, EVENT_INPUT | EVENT_EXCEPT); + c = iochan_create(s, http_io, EVENT_INPUT | EVENT_EXCEPT, "http_session_socket"); ch = http_channel_create(server->http_server, inet_ntoa(addr.sin_addr), server); ch->iochan = c; iochan_setdata(c, ch); - - pazpar2_add_channel(c); + iochan_add(server->iochan_man, c); } /* Create a http-channel listener, syntax [host:]port */ @@ -1188,7 +1190,6 @@ int http_init(const char *addr, struct conf_server *server) int one = 1; const char *pp; short port; - http_server_t http_server; yaz_log(YLOG_LOG, "HTTP listener %s", addr); @@ -1241,14 +1242,14 @@ int http_init(const char *addr, struct conf_server *server) return 1; } - http_server = http_server_create(); - server->http_server = http_server; + server->http_server = http_server_create(); - http_server->listener_socket = l; + server->http_server->listener_socket = l; - c = iochan_create(l, http_accept, EVENT_INPUT | EVENT_EXCEPT); + c = iochan_create(l, http_accept, EVENT_INPUT | EVENT_EXCEPT, "http_server"); iochan_setdata(c, server); - pazpar2_add_channel(c); + + iochan_add(server->iochan_man, c); return 0; } @@ -1373,15 +1374,10 @@ void http_server_destroy(http_server_t hs) { int r; - if (hs->mutex) - { - yaz_mutex_enter(hs->mutex); - r = --(hs->ref_count); - yaz_mutex_leave(hs->mutex); - } - else - r = --(hs->ref_count); - + yaz_mutex_enter(hs->mutex); /* OK: hs->mutex may be NULL */ + r = --(hs->ref_count); + yaz_mutex_leave(hs->mutex); + if (r == 0) { struct http_buf *b = hs->http_buf_freelist; @@ -1395,6 +1391,8 @@ void http_server_destroy(http_server_t hs) while (c) { struct http_channel *c_next = c->next; + nmem_destroy(c->nmem); + wrbuf_destroy(c->wrbuf); xfree(c); c = c_next; } @@ -1419,7 +1417,7 @@ void http_mutex_init(struct conf_server *server) assert(server); assert(server->http_server->mutex == 0); - yaz_mutex_create(&server->http_server->mutex); + pazpar2_mutex_create(&server->http_server->mutex, "http_server"); server->http_server->http_sessions = http_sessions_create(); }