summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRussell King <rmk@armlinux.org.uk>2021-09-14 22:36:01 +0100
committerRussell King <rmk@armlinux.org.uk>2021-09-14 22:36:01 +0100
commit9b52ca854449b2fb3b845f9d35a59f7089338e1a (patch)
tree7728b4fa23c013f074fc323d954849e2cabc45ae
parent2f45bcf78b77193ad6107fbb92ad0f93eb1ad4fc (diff)
Fix double-close of client
respond_error() already closes the client and frees the structures, so we should not call close_client() after calling respond_error(). Signed-off-by: Russell King <rmk@armlinux.org.uk>
-rw-r--r--event-httpd.c3
1 files changed, 0 insertions, 3 deletions
diff --git a/event-httpd.c b/event-httpd.c
index 865f82f..63fb2f3 100644
--- a/event-httpd.c
+++ b/event-httpd.c
@@ -238,7 +238,6 @@ static void receive(GObject *source, GAsyncResult *res, gpointer user_data)
c->resource = resource;
if (!resource->ops) {
respond_error(c, 204, "No Content");
- close_client(c);
return;
}
@@ -247,7 +246,6 @@ static void receive(GObject *source, GAsyncResult *res, gpointer user_data)
case GET:
if (!resource->ops->get) {
respond_error(c, 204, "No Content");
- close_client(c);
return;
}
@@ -262,7 +260,6 @@ static void receive(GObject *source, GAsyncResult *res, gpointer user_data)
case UPDATE:
if (!resource->ops->update) {
respond_error(c, 204, "No Content");
- close_client(c);
return;
}