Browse Source

Fixed two mem leaks from httpd :/

dynamic-accesslists
erdgeist 18 years ago
parent
commit
b7526f25e8
  1. 4
      Makefile
  2. 47
      opentracker.c

4
Makefile

@ -1,6 +1,6 @@
CC?=gcc CC?=gcc
CFLAGS+=-I../libowfat -Wall -pipe -O2 CFLAGS+=-I../libowfat -Wall -pipe -Os
LDFLAGS+=-L../libowfat/ -lowfat -s -lm LDFLAGS+=-L../libowfat/ -lowfat -s
HEADERS=trackerlogic.h scan_urlencoded_query.h HEADERS=trackerlogic.h scan_urlencoded_query.h
SOURCES=opentracker.c trackerlogic.c scan_urlencoded_query.c SOURCES=opentracker.c trackerlogic.c scan_urlencoded_query.c

47
opentracker.c

@ -38,8 +38,6 @@ static void panic(const char* routine) {
struct http_data { struct http_data {
array r; array r;
io_batch iob; io_batch iob;
char* hdrbuf;
int hlen;
unsigned long ip; unsigned long ip;
}; };
@ -61,28 +59,13 @@ int header_complete(struct http_data* r)
return 0; return 0;
} }
void httperror(struct http_data* r,const char* title,const char* message) void httperror(struct http_data* h,const char* title,const char* message)
{ {
char* c; char* c = (char*)malloc(strlen(message)+strlen(title)+200);
c=r->hdrbuf=(char*)malloc(strlen(message)+strlen(title)+200); if( !c) iob_addbuf(&h->iob, "HTTP/1.0 500 internal error\r\nContent-Type: text/plain\r\nConnection: close\r\n\r\nout of memory\n", 90);
else iob_addbuf_free( &h->iob, c,
if (!c) sprintf( c, "HTTP/1.0 %s\r\nContent-Type: text/html\r\nConnection: close\r\nContent-Length: %ld\r\n\r\n<title>%s</title>\n",
{ title, strlen(message)+strlen(title)+16-4,title+4) );
r->hdrbuf="HTTP/1.0 500 internal error\r\nContent-Type: text/plain\r\nConnection: close\r\n\r\nout of memory\n";
r->hlen=strlen(r->hdrbuf);
}
else
{
c+=fmt_str(c,"HTTP/1.0 ");
c+=fmt_str(c,title);
c+=fmt_str(c,"\r\nContent-Type: text/html\r\nConnection: close\r\nContent-Length: ");
c+=fmt_ulong(c,strlen(message)+strlen(title)+16-4);
c+=fmt_str(c,"\r\n\r\n<title>");
c+=fmt_str(c,title+4);
c+=fmt_str(c,"</title>\n");
r->hlen=c - r->hdrbuf;
}
iob_addbuf(&r->iob,r->hdrbuf,r->hlen);
} }
// bestimmten http parameter auslesen und adresse zurueckgeben // bestimmten http parameter auslesen und adresse zurueckgeben
@ -279,7 +262,7 @@ e500:
reply = malloc( numwant*6+128 ); // peerlist + seeder, peers and lametta n*6+81 a.t.m. reply = malloc( numwant*6+128 ); // peerlist + seeder, peers and lametta n*6+81 a.t.m.
if( reply ) if( reply )
reply_size = return_peers_for_torrent( torrent, numwant, reply ); reply_size = return_peers_for_torrent( torrent, numwant, reply );
if( !reply || ( reply_size < 0 ) ) { if( !reply || ( reply_size <= 0 ) ) {
if( reply ) free( reply ); if( reply ) free( reply );
goto e500; goto e500;
} }
@ -301,11 +284,8 @@ e404:
goto bailout; goto bailout;
} }
c=h->hdrbuf=(char*)malloc(80); c=(char*)malloc(80);
c+=fmt_str(c,"HTTP/1.0 200 OK\r\nContent-Type: text/plain\r\nContent-Length: "); iob_addbuf_free( &h->iob, c, sprintf( c, "HTTP/1.0 200 OK\r\nContent-Type: text/plain\r\nContent-Length: %ld\r\n\r\n", (long)reply_size ));
c+=fmt_ulonglong(c, reply_size );
c+=fmt_str(c,"\r\n\r\n");
iob_addbuf(&h->iob,h->hdrbuf,c - h->hdrbuf);
if( reply && reply_size ) iob_addbuf_free(&h->iob, reply, reply_size ); if( reply && reply_size ) iob_addbuf_free(&h->iob, reply, reply_size );
bailout: bailout:
@ -388,7 +368,7 @@ int main()
{ {
array_reset(&h->r); array_reset(&h->r);
iob_reset(&h->iob); iob_reset(&h->iob);
free(h->hdrbuf); h->hdrbuf=0; free(h);
} }
io_close(i); io_close(i);
} }
@ -421,15 +401,14 @@ emerge:
struct http_data* h=io_getcookie(i); struct http_data* h=io_getcookie(i);
int64 r=iob_send(i,&h->iob); int64 r=iob_send(i,&h->iob);
if (r==-1) if (r==-1)
io_eagain(i); io_eagain(i);
else else
if (r<=0) if ((r<=0)||(h->iob.bytesleft==0))
{ {
array_trunc(&h->r); array_reset(&h->r);
iob_reset(&h->iob); iob_reset(&h->iob);
free(h->hdrbuf); h->hdrbuf=0; free(h); free(h);
io_close(i); io_close(i);
} }
} }

Loading…
Cancel
Save