diff options
| author | Michael Brown | 2007-01-15 10:18:06 +0100 |
|---|---|---|
| committer | Michael Brown | 2007-01-15 10:18:06 +0100 |
| commit | afa752f5fbebf49fcd399f85d1db85ef29cefc03 (patch) | |
| tree | ed9054fc05e71e45672066aaf9f1e1b507dd1b85 /src/net | |
| parent | Must free http on the error path; nothing else will do it (diff) | |
| download | ipxe-afa752f5fbebf49fcd399f85d1db85ef29cefc03.tar.gz ipxe-afa752f5fbebf49fcd399f85d1db85ef29cefc03.tar.xz ipxe-afa752f5fbebf49fcd399f85d1db85ef29cefc03.zip | |
Protocol's get() method no longer takes ownership of the URI. HTTP is the
exception rather than the rule; we may as well keep things clean for other
protocols.
Diffstat (limited to 'src/net')
| -rw-r--r-- | src/net/tcp/http.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/src/net/tcp/http.c b/src/net/tcp/http.c index 346f5a83c..3ae7b1e95 100644 --- a/src/net/tcp/http.c +++ b/src/net/tcp/http.c @@ -366,7 +366,6 @@ static void http_reap ( struct async *async ) { struct http_request *http = container_of ( async, struct http_request, async ); - free_uri ( http->uri ); free ( http ); } @@ -385,8 +384,6 @@ static struct async_operations http_async_operations = { * @v buffer Buffer into which to download file * @v parent Parent asynchronous operation * @ret rc Return status code - * - * If it returns success, this function takes ownership of the URI. */ int http_get ( struct uri *uri, struct buffer *buffer, struct async *parent ) { struct http_request *http; |
