2003-01-29 Havoc Pennington <hp@pobox.com>
[platform/upstream/dbus.git] / ChangeLog
index adbd7bb..7e0f9cd 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,371 @@
+2003-01-29  Havoc Pennington  <hp@pobox.com>
+
+       * dbus/dbus-message.c (check_message_handling): fix assertion
+       failure on set_client_serial
+
+2003-01-28  Havoc Pennington  <hp@pobox.com>
+
+       * dbus/dbus-server-debug.c: Add doc section comments
+       
+       * dbus/dbus-transport-debug.c: add doc section comments
+       
+2003-01-28  Havoc Pennington  <hp@redhat.com>
+
+       * dbus/dbus-string.c (_dbus_string_base64_decode): append bytes in
+       the reverse order from how I had it
+       (_dbus_string_base64_encode): reverse encoding order. I was
+       basically byteswapping everything during encoding.
+
+2003-01-28  Anders Carlsson  <andersca@codefactory.se>
+
+       * dbus/dbus-connection-internal.h:
+       * dbus/dbus-connection.c: (_dbus_connection_add_timeout),
+       (_dbus_connection_remove_timeout):
+       Add functions for adding and removing timeouts.
+       
+       * dbus/dbus-message.c: (dbus_message_new_from_message):
+       Add new function that takes a message and creates an exact
+       copy of it, but with the refcount set to 1.
+       (check_message_handling):
+       Fix build error.
+       
+       * dbus/dbus-server-protected.h:
+       * dbus/dbus-server.c: (_dbus_server_init_base),
+       (_dbus_server_finalize_base), (_dbus_server_add_timeout),
+       (dbus_server_set_timeout_functions):
+       (_dbus_server_remove_timeout):
+       New functions so that a server can add and remove timeouts.
+
+       (dbus_server_listen):
+       Add commented out call to dbus_server_debug_new.
+
+       * dbus/dbus-timeout.c: (_dbus_timeout_new):
+       Actually set the handler, doh.
+       
+       * dbus/dbus-transport.c: (_dbus_transport_open):
+       Add commented out call to dbus_transport_debug_client_new.
+       
+       * dbus/Makefile.am:
+       Add dbus-transport-debug.[ch] and dbus-server-debug.[ch]
+       
+2003-01-28  Havoc Pennington  <hp@pobox.com>
+
+       * dbus/dbus-message.c (check_message_handling): function to check 
+       on the loaded message, iterates over it etc.
+
+2003-01-28  Havoc Pennington  <hp@pobox.com>
+
+       * test/Makefile.am (dist-hook): fix make distdir
+
+       * dbus/Makefile.am (TESTS_ENVIRONMENT): fix make check
+
+2003-01-27  Havoc Pennington  <hp@pobox.com>
+
+       * dbus/dbus-mempool.c (time_for_size): replace printf with 
+       _dbus_verbose
+
+       * dbus/dbus-message-builder.c (_dbus_message_data_load): allow
+       empty lines; fix the SAVE_LENGTH stuff to be
+       START_LENGTH/END_LENGTH so it actually works; couple other 
+       bugfixes
+       
+       * test/Makefile.am (dist-hook): add dist-hook for .message files
+
+       * dbus/dbus-string.c (DBUS_STRING_COPY_PREAMBLE): source of a copy
+       can be constant or locked.
+       (_dbus_string_free): allow freeing a const string as
+       documented/intended
+
+       * dbus/dbus-sysdeps.c (_dbus_concat_dir_and_file): utility
+
+       * dbus/dbus-test-main.c (main): take an argument which is the 
+       directory containing test data
+
+       * dbus/dbus-message.c (_dbus_message_test): pass a test_data_dir
+       argument to this and load all the messages in test/data/
+       checking that they can be loaded or not loaded as appropriate.
+
+2003-01-27  Anders Carlsson  <andersca@codefactory.se>
+
+       * bus/dispatch.c: (bus_dispatch_message_handler):
+       Dispatch messages sent to services.
+       
+       * bus/driver.c: (bus_driver_send_service_deleted),
+       (bus_driver_send_service_created), (bus_driver_send_service_lost),
+       (bus_driver_send_service_acquired):
+       Add helper functions for sending service related messages.
+
+       (bus_driver_send_welcome_message):
+       Send HELLO_REPLY instead of WELCOME.
+       
+       (bus_driver_handle_list_services):
+       Send LIST_SERVICES_REPLY instead of SERVICES.
+       
+       (bus_driver_handle_own_service),
+       (bus_driver_handle_service_exists):
+       New message handlers.
+       
+       (bus_driver_handle_message):
+       Invoke new message handlers.
+       
+       (bus_driver_remove_connection):
+       Don't remove any services here since that's done automatically
+       by bus_service_remove_owner now.
+       
+       * bus/driver.h:
+       New function signatures.
+       
+       * bus/services.c: (bus_service_add_owner):
+       Send ServiceAcquired message if we're the only primary owner.
+       
+       (bus_service_remove_owner):
+       Send ServiceAcquired/ServiceLost messages.
+       
+       (bus_service_set_prohibit_replacement),
+       (bus_service_get_prohibit_replacement):
+       Functions for setting prohibit replacement.
+       
+       (bus_service_has_owner):
+       New function that checks if a connection is in the owner queue of 
+       a certain service.
+       
+       * bus/services.h:
+       Add new function signatures.
+       
+       * dbus/dbus-list.c: (_dbus_list_test):
+       Add tests for _dbus_list_remove_last and traversing the list backwards.
+       
+       * dbus/dbus-list.h:
+       Fix a typo in _dbus_list_get_prev_link, if we're at the first element we can't
+       go any further, so return NULL then.
+       
+       * dbus/dbus-protocol.h:
+       Add new messages, service flags and service replies.
+       
+2003-01-26  Havoc Pennington  <hp@pobox.com>
+
+       * dbus/dbus-message-builder.c: implement, completely untested.
+
+       * test/data/*: add data to be used in testing. 
+       ".message" files are our simple loadable text format.
+       ".message-raw" will be binary dumps of messages.
+
+       * dbus/dbus-string.c (_dbus_string_starts_with_c_str): new
+
+2003-01-26  Havoc Pennington  <hp@pobox.com>
+
+       * dbus/dbus-sysdeps.c (_dbus_file_get_contents): new function
+
+       * dbus/dbus-errors.c (dbus_result_to_string): add
+       file errors
+
+       * dbus/dbus-message-builder.c: new file, will contain code to load
+       up messages from files. Not implemented yet.
+
+2003-01-26  Havoc Pennington  <hp@pobox.com>
+
+       * dbus/dbus-message.c (dbus_message_set_sender): support deleting
+       the sender by setting to NULL
+
+2003-01-26  Havoc Pennington  <hp@pobox.com>
+
+        The unit tests pass, but otherwise untested.  If it breaks, the
+       tests should have been better. ;-)
+       
+       * bus/driver.c (bus_driver_handle_hello): return if we disconnect
+       the connection.
+
+       * dbus/dbus-message.c: redo everything so we maintain 
+       message->header as the only copy of the various fields.
+       This avoids the possibility of out-of-memory in some cases,
+       for example dbus_message_lock() can't run out of memory anymore,
+       and avoids extra copying. Figured I may as well go ahead and do 
+       this since it was busted for dbus_message_lock to not return 
+       failure on OOM, and dbus_message_write_header was totally
+       unchecked for OOM. Also fixed some random other bugs.
+
+       * dbus/dbus-marshal.c (_dbus_marshal_get_field_end_pos): verify
+       that strings are nul-terminated. Also, end_pos can be equal 
+       to string length just not greater than, I think.
+       (_dbus_marshal_set_int32): new function
+       (_dbus_marshal_set_uint32): new function
+       (_dbus_marshal_set_string): new function
+
+       * dbus/dbus-connection.c (_dbus_connection_new_for_transport): fix
+       a warning, init timeout_list to NULL
+       (dbus_connection_send_message): don't use uninitialized variable
+       "serial"
+
+       * dbus/dbus-string.c (_dbus_string_replace_len): new function
+
+2003-01-26  Anders Carlsson  <andersca@codefactory.se>
+
+       * bus/driver.c: (bus_driver_handle_hello),
+       (bus_driver_send_welcome_message):
+       Plug leaks
+       
+2003-01-26  Anders Carlsson  <andersca@codefactory.se>
+
+       * dbus/dbus-auth.c: (process_auth), (_dbus_auth_unref):
+       * dbus/dbus-connection.c: (_dbus_connection_new_for_transport),
+       (dbus_connection_unref):
+       * dbus/dbus-marshal.c: (_dbus_marshal_test):
+       * dbus/dbus-message.c: (dbus_message_unref),
+       Plug memory leaks.
+       
+       (dbus_message_get_fields): 
+       Remove debugging printout.
+
+       (_dbus_message_loader_return_buffer):
+       Don't store the header string.
+       
+       (_dbus_message_test):
+       Plug leaks.
+
+2003-01-26  Richard Hult  <rhult@codefactory.se>
+
+       * glib/dbus-gmain.c (dbus_connection_dispatch): Traverse a copy of
+       the file descriptor list, since it can change under us.
+
+2003-01-25  Anders Carlsson  <andersca@codefactory.se>
+
+       * glib/dbus-gmain.c: (dbus_connection_prepare),
+       (dbus_connection_check), (dbus_connection_dispatch), (add_watch),
+       (remove_watch), (dbus_connection_hookup_with_g_main):
+       Rewrite the glib handling to use its own GSource instead of a 
+       GIOChannel so we can catch messages put in the queue while waiting
+       for a reply.
+       
+2003-01-25  Anders Carlsson  <andersca@codefactory.se>
+
+       * bus/Makefile.am:
+       * bus/connection.c: (connection_disconnect_handler),
+       (connection_watch_callback), (bus_connection_setup):
+       * bus/dispatch.c: (send_one_message),
+       (bus_dispatch_broadcast_message), (bus_dispatch_message_handler),
+       (bus_dispatch_add_connection), (bus_dispatch_remove_connection):
+       * bus/dispatch.h:
+       * bus/driver.c: (bus_driver_send_service_deleted),
+       (bus_driver_send_service_created), (bus_driver_handle_hello),
+       (bus_driver_send_welcome_message),
+       (bus_driver_handle_list_services), (bus_driver_remove_connection),
+       (bus_driver_handle_message):
+       * bus/driver.h:
+       Refactor code, put the message dispatching in its own file. Use 
+       _DBUS_HANDLE_OOM. Also send ServiceDeleted messages when a client
+       is disconnected.
+       
+2003-01-25  Anders Carlsson  <andersca@codefactory.se>
+
+       * dbus/dbus-internals.h:
+       Add _DBUS_HANDLE_OOM macro, it doesn't do anything currently.
+       
+       * dbus/dbus-message.c: (dbus_message_get_sender):
+       * dbus/dbus-message.h:
+       Implement dbus_message_get_sender.
+       
+       * dbus/dbus-protocol.h:
+       Add message and service defines.
+       
+2003-01-25  Anders Carlsson  <andersca@codefactory.se>
+
+       * dbus/dbus-connection.c: (dbus_connection_send_message):
+       * dbus/dbus-message-internal.h:
+       * dbus/dbus-message.c: (_dbus_message_get_client_serial),
+       (dbus_message_write_header):
+       Remove _dbus_messag_unlock and don't set the client serial on a 
+       message if one already exists.
+       
+2003-01-24  Havoc Pennington  <hp@pobox.com>
+
+       * dbus/dbus-list.c (alloc_link): put a thread lock on the global
+       list_pool
+
+       * bus/driver.c (bus_driver_handle_list_services): fix a leak 
+       on OOM
+
+2003-01-25  Anders Carlsson  <andersca@codefactory.se>
+
+       * dbus/dbus-list.c: (alloc_link), (free_link):
+       Use a memory pool for the links.
+       
+2003-01-25  Anders Carlsson  <andersca@codefactory.se>
+
+       * bus/connection.c: (bus_connection_foreach):
+       * bus/connection.h:
+       Add new bus_connection_foreach function.
+       
+       * bus/driver.c: (send_one_message), (bus_driver_broadcast_message):
+       Add function that broadcasts a message to all clients.
+       
+       (bus_driver_send_service_created), (bus_driver_handle_hello),
+       (bus_driver_send_welcome_message),
+       (bus_driver_handle_list_services), (bus_driver_message_handler):
+       Implement functions that take care of listing services, and notifying
+       clients when new services are created.
+       
+       * bus/services.c: (bus_services_list):
+       * bus/services.h:
+       Add new function that returns an array of strings with the currently
+       registered services.
+       
+       * glib/dbus-glib.h:
+       * glib/dbus-gmain.c:
+       Update copyright year.
+       
+2003-01-25  Anders Carlsson  <andersca@codefactory.se>
+
+       * dbus/dbus-connection.c: (dbus_connection_send_message):
+       Unlock the message in case it was sent earlier.
+       
+       (dbus_connection_send_message_with_reply_and_block):
+       Remove the reply message from the list.
+       
+       * dbus/dbus-marshal.c: (_dbus_demarshal_string_array):
+       Set array_len and new_pos correctly.
+       
+       (_dbus_marshal_test):
+       Remove debug output.
+       
+       * dbus/dbus-message-internal.h:
+       * dbus/dbus-message.c: (_dbus_message_get_reply_serial):
+       New function that returns the reply serial.
+       
+       (_dbus_message_unlock):
+       New function that unlocks a message and resets its header.
+
+       (dbus_message_append_string_array), 
+       (dbus_message_get_fields_valist),
+       (dbus_message_iter_get_field_type),
+       (dbus_message_iter_get_string_array),   
+       (dbus_message_get_fields),      
+       (dbus_message_append_fields_valist):
+       Handle string arrays.
+       
+       (dbus_message_set_sender):
+       Make this function public since the bus daemon needs it.
+       
+       (decode_header_data):
+       Set the reply serial to -1 initially.
+
+       * dbus/dbus-message.h:
+       Add dbus_message_set_sender.    
+
+2003-01-24  Havoc Pennington  <hp@pobox.com>
+
+       * doc/dbus-specification.sgml: add some stuff
+
+2003-01-22  Havoc Pennington  <hp@pobox.com>
+
+       * doc/dbus-specification.sgml: Start to document the protocol.
+
+2003-01-22  Havoc Pennington  <hp@pobox.com>
+
+       * dbus/dbus-connection.c
+       (dbus_connection_send_message_with_reply_and_block): add some @todo
+
+       * bus/driver.c (bus_driver_add_connection): add a FIXME about memleak
+
 2003-01-21  Havoc Pennington  <hp@pobox.com>
 
         (patch untested because can't compile)