From fd62df64188c0f992876c72fdda8a6da5dba3090 Mon Sep 17 00:00:00 2001 From: Andrew Tridgell Date: Sun, 23 Jan 2005 11:49:15 +0000 Subject: r4943: Smplified the events handling code a lot. The first source of complexity was that events didn't automatically cleanup themselves. This was because the events code was written before we had talloc destructors, so you needed to call event_remove_XX() to clean the event out of the event lists from every piece of code that used events. I have now added automatic event destructors, which in turn allowed me to simplify a lot of the calling code. The 2nd source of complexity was caused by the ref_count, which was needed to cope with event handlers destroying events while handling them, which meant the linked lists became invalid, so the ref_count ws used to mark events for later destruction. The new system is much simpler. I now have a ev->destruction_count, which is incremented in all event destructors. The event dispatch code checks for changes to this and handles it. (This used to be commit a3c7417cfeab429ffb22d5546b205818f531a7b4) --- source4/smbd/service.c | 13 ++++--------- 1 file changed, 4 insertions(+), 9 deletions(-) (limited to 'source4/smbd/service.c') diff --git a/source4/smbd/service.c b/source4/smbd/service.c index 40a2a4b506..7067a26e7a 100644 --- a/source4/smbd/service.c +++ b/source4/smbd/service.c @@ -171,6 +171,7 @@ struct server_stream_socket *service_setup_stream_socket(struct server_service * } talloc_steal(stream_socket, sock); + talloc_steal(stream_socket, stream_socket->event.fde); if (stream_socket->stream.ops->socket_init) { stream_socket->stream.ops->socket_init(stream_socket); @@ -194,15 +195,6 @@ static int server_connection_destructor(void *ptr) conn->stream_socket->stream.ops->close_connection(conn, "shutdown"); } - if (conn->event.fde) { - event_remove_fd(conn->event.ctx, conn->event.fde); - conn->event.fde = NULL; - } - if (conn->event.idle) { - event_remove_timed(conn->event.ctx, conn->event.idle); - conn->event.idle = NULL; - } - return 0; } @@ -250,6 +242,9 @@ struct server_connection *server_setup_connection(struct event_context *ev, srv_conn->event.fde = event_add_fd(ev,&fde); srv_conn->event.idle = event_add_timed(ev,&idle); + talloc_steal(srv_conn, srv_conn->event.fde); + talloc_steal(srv_conn, srv_conn->event.idle); + talloc_set_destructor(srv_conn, server_connection_destructor); if (!socket_check_access(sock, "smbd", lp_hostsallow(-1), lp_hostsdeny(-1))) { -- cgit