2003-01-25 Anders Carlsson <andersca@codefactory.se>
[platform/upstream/dbus.git] / ChangeLog
index 5c16e6d..3961e97 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,376 @@
+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)
+       
+       * bus/driver.c (create_unique_client_name): make this function
+       never recycle client names. Also, caller should initialize 
+       the DBusString.
+
+       * dbus/dbus-sysdeps.c (_dbus_get_current_time): new function
+
+2003-01-21  Anders Carlsson  <andersca@codefactory.se>
+
+       * dbus/dbus-marshal.c: (_dbus_marshal_double),
+       (_dbus_marshal_int32), (_dbus_marshal_uint32),
+       (_dbus_marshal_int32_array), (_dbus_marshal_uint32_array),
+       (_dbus_marshal_double_array), (_dbus_marshal_string_array),
+       (_dbus_demarshal_int32_array), (_dbus_demarshal_uint32_array),
+       (_dbus_demarshal_double_array), (_dbus_demarshal_string_array),
+       (_dbus_marshal_get_field_end_pos), (_dbus_marshal_test):
+       * dbus/dbus-marshal.h:
+       * dbus/dbus-protocol.h:
+       Add support for marshalling and demarshalling integer, double
+       and string arrays.
+       
+2003-01-21  Anders Carlsson  <andersca@codefactory.se>
+
+       * bus/Makefile.am:
+       Add driver.[ch]
+       
+       * bus/connection.c: (connection_disconnect_handler):
+       Remove the connection from the bus driver's list.
+       
+       (connection_watch_callback): Dispatch messages.
+
+       (free_connection_data): Free connection name.
+       
+       (bus_connection_setup): Add connection to the bus driver's list.
+       (bus_connection_remove_owned_service): 
+       (bus_connection_set_name), (bus_connection_get_name):
+       Add functions for setting and getting the connection's name.
+       
+       * bus/connection.h:
+       Add function headers.
+       
+       * bus/driver.c: (create_unique_client_name),
+       (bus_driver_handle_hello_message),
+       (bus_driver_send_welcome_message), (bus_driver_message_handler),
+       (bus_driver_add_connection), (bus_driver_remove_connection):
+       * bus/driver.h:
+       * bus/main.c:
+       * bus/services.c: (bus_service_free):
+       * bus/services.h:
+       New file that handles communication and registreation with the bus
+       itself. 
+       
+2003-01-21  Anders Carlsson  <andersca@codefactory.se>
+
+       * dbus/dbus-connection.c: (dbus_connection_send_message):
+       Add a new client_serial parameter.
+       
+       (dbus_connection_send_message_with_reply):
+       Remove a @todo since we've implemented the blocking function.
+       
+       (dbus_connection_send_message_with_reply_and_block):
+       New function that sends a message and waits for a reply and
+       then returns the reply.
+       
+       * dbus/dbus-connection.h:
+       Add new functions.
+       
+       * dbus/dbus-errors.c: (dbus_result_to_string):
+       * dbus/dbus-errors.h:
+       Add new DBUS_RESULT.
+       
+       * dbus/dbus-message-internal.h:
+       * dbus/dbus-message.c: (_dbus_message_get_reply_serial),
+       (_dbus_message_set_sender), (dbus_message_write_header),
+       (dbus_message_new_reply), (decode_header_data),
+       (_dbus_message_loader_return_buffer), (_dbus_message_test):
+       * dbus/dbus-message.h:
+       Add new functions that set the reply serial and sender.
+       Also marshal and demarshal them correctly and add test.
+       
+       * dbus/dbus-protocol.h:
+       Add new DBUS_MESSAGE_TYPE_SENDER.
+       
+       * glib/dbus-glib.h:
+       * glib/dbus-gmain.c: (watch_callback), (free_callback_data),
+       (add_watch), (remove_watch), (add_timeout), (remove_timeout),
+       (dbus_connection_hookup_with_g_main):
+       * glib/test-dbus-glib.c: (main):
+       Rewrite to use GIOChannel and remove the GSource crack.
+       
+       * test/echo-client.c: (main):
+       * test/watch.c: (check_messages):
+       Update for changed APIs
+       
+2003-01-19  Anders Carlsson  <andersca@codefactory.se>
+
+       * dbus/Makefile.am: Add dbus-timeout.[cħ]
+       
+       * dbus/dbus-connection.c: (_dbus_connection_new_for_transport):
+       Create a DBusTimeoutList.       
+       (dbus_connection_set_timeout_functions): Add new function to
+       set timeout callbacks
+       
+       * dbus/dbus-connection.h: Add public DBusTimeout API.
+       
+       * dbus/dbus-message.c: (dbus_message_get_service):
+       * dbus/dbus-message.h:  New function.
+
+       * dbus/dbus-server.c: Fix small doc typo.
+       
+       * dbus/dbus-timeout.[ch]: New files for mainloop timeouts.
+
+2003-01-19  Anders Carlsson  <andersca@codefactory.se>
+
+       * dbus/dbus-string.c (_dbus_string_move_len): Don't delete all
+       of the string, just as long as specified.
+
+2003-01-19  Havoc Pennington  <hp@pobox.com>
+
+       * dbus/dbus-connection.c (dbus_connection_get_is_authenticated):
+       new function
+
+       * dbus/dbus-server.c (dbus_server_set_max_connections)
+       (dbus_server_get_max_connections, dbus_server_get_n_connections):
+       keep track of current number of connections, and add API for
+       setting a max (but haven't implemented enforcing the max yet)
+
+2003-01-18  Havoc Pennington  <hp@pobox.com>
+
+       * dbus/dbus-transport-unix.c (unix_do_iteration): only do the
+       reading/writing if read_watch != NULL or write_watch != NULL.
+
+       * dbus/dbus-message.c (_dbus_message_loader_return_buffer): fix
+       the message loader code to actually load message->header and
+       message->body into the newly-created message.
+
+       * dbus/dbus-transport-unix.c (check_write_watch): fix a mem leak
+       in OOM case
+
+       * dbus/dbus-connection.c (dbus_connection_set_max_message_size) 
+       (dbus_connection_get_max_message_size) 
+       (dbus_connection_set_max_live_messages_size) 
+       (dbus_connection_get_max_live_messages_size): implement some
+       resource limitation functions
+
+       * dbus/dbus-resources.c: new file implementing some of the
+       resource limits stuff
+
+       * dbus/dbus-message.c (dbus_message_iter_get_byte_array): add
+       missing docs, add @todo to handle OOM etc.
+
+       * dbus/dbus-marshal.c (_dbus_demarshal_byte_array): add missing
+       docs
+       
+2003-01-18  Havoc Pennington  <hp@pobox.com>
+
+       * dbus/dbus-connection.c (dbus_connection_unref): disconnect the 
+       connection if it hasn't been already.
+
+       * dbus/dbus-connection.h: kill off the idea of an ErrorFunction,
+       replace with DisconnectFunction.
+
+2003-01-18  Havoc Pennington  <hp@pobox.com>
+
+        Building --disable-verbose-mode --disable-asserts --disable-tests
+       cuts the library from 112K to 45K or so
+       
+       * configure.in: check for varargs macro support, 
+       add --enable-verbose-mode, --enable-asserts. 
+
+       * dbus/dbus-internals.h (_dbus_assert): support
+       DBUS_DISABLE_ASSERT
+       (_dbus_verbose): support DBUS_ENABLE_VERBOSE_MODE
+
+2003-01-18  Havoc Pennington  <hp@pobox.com>
+
+       * dbus/dbus-test.c: include config.h so that tests actually run
+
+       * dbus/dbus-string.c: add assertions that stuff is 8-byte aligned,
+       so the failure mode when that assumption fails will be plenty
+       obvious.
+
+2003-01-18  Havoc Pennington  <hp@pobox.com>
+
+       * configure.in: default --enable-tests to $USE_MAINTAINER_MODE
+
+       * dbus/Makefile.am: fix it up so dubs-test-main.c is included in
+       the distribution
+
+       * test/Makefile.am: don't use special variable "TESTS" for echo-*
+       since we don't want to use those in make check
+
+2003-01-15  Havoc Pennington  <hp@redhat.com>
+
+        Release 0.2
+       
+       * NEWS: update
+
+2003-01-15  Havoc Pennington  <hp@redhat.com>
+
+       * test/Makefile.am: fix so that test source code ends up in the
+       distribution on make distcheck
+
+2003-01-15  Havoc Pennington  <hp@redhat.com>
+
+        Release 0.1.
+       
+       * NEWS: update  
+
+2003-01-15  Havoc Pennington  <hp@redhat.com>
+
+       * dbus/dbus-test.c (dbus_internal_symbol_do_not_use_run_tests):
+       fix build when --disable-tests
+
+       * Makefile.am (EXTRA_DIST): put HACKING in here
+
+       * HACKING: document procedure for making a tarball release.
+
+2003-01-14  Anders Carlsson  <andersca@codefactory.se>
+
+       * bus/connection.c: (connection_error_handler),
+       (bus_connection_setup):
+       * bus/main.c: (main):
+       Make sure that the DBusConnectionData struct is NULLed
+       out to prevent a segfault.
+       
+       * dbus/dbus-errors.c: (dbus_result_to_string):
+       * dbus/dbus-errors.h:
+       * dbus/dbus-message.c: (dbus_message_get_fields),
+       (dbus_message_get_fields_valist), (_dbus_message_test):
+       * dbus/dbus-message.h:
+       Make dbus_message_get_fields return a result code so we can
+       track invalid fields as well as oom.
+       
+2003-01-11  Havoc Pennington  <hp@pobox.com>
+
+       * configure.in: change --enable-test/--enable-ansi action-if-given
+       to enable_foo=$enableval instead of enable_foo=yes
+
+2003-01-08  Havoc Pennington  <hp@pobox.com>
+
+       * dbus/dbus-string.c (_dbus_string_align_length): new function
+
+       * dbus/dbus-test-main.c: move main() for test app here
+       * dbus/dbus-test.c
+       (dbus_internal_symbol_do_not_use_run_tests): we have to export a
+       symbol to run tests, because dbus-test isn't in the main 
+       library
+
+        Code review nitpicks.
+       
+       * dbus/dbus-message.c (dbus_message_write_header): add newlines
+       for people with narrow emacs ;-). Assert client_serial was filled
+       in. Assert message->name != NULL.
+       (dbus_message_append_fields): have "first_field_type" arg separate
+       from va list, needed for C++ binding that also uses varargs IIRC
+       and helps with type safety
+       (dbus_message_new): add @todo about using DBusString to store
+       service/name internally
+       (dbus_message_new): don't leak ->service and ->name on OOM later
+       in the function
+       (dbus_message_unref): free the service name
+       (dbus_message_get_fields): same change to varargs
+       i.e. first_field_type
+       (_dbus_message_loader_return_buffer): assert that the message data
+       is aligned (if not it's a bug in our code). Put in verbose griping
+       about why we set corrupted = TRUE.
+       (decode_header_data): add FIXME that char* is evil.  Was going to
+       add FIXME about evil locale-specific string.h strncmp, but just
+       switched to wacky string-as-uint32 optimization. Move check for
+       "no room for field name" above get_const_data_len() to avoid
+       assertion failure in get_const_data_len if we have trailing 2
+       bytes or the like. Check for service and name fields being
+       provided twice. Don't leak service/name on error. Require field
+       names to be aligned to 4 bytes.
+
+       * dbus/dbus-marshal.c: move byte swap stuff to header
+       (_dbus_pack_int32): uscore-prefix
+       (_dbus_unpack_int32): uscore-prefix
+       (_dbus_unpack_uint32): export
+       (_dbus_demarshal_string): add @todo complaining about use of
+       memcpy()
+       (_dbus_marshal_get_field_end_pos): add @todo about bad error
+       handling allowing corrupt data to go unchecked
+
+2003-01-08  Havoc Pennington  <hp@redhat.com>
+
+       * dbus/dbus-transport-unix.c (unix_do_iteration): add read/write 
+       to the select() as needed for authentication. (should be using
+       _dbus_poll() not select, but for another day)
+
+       * dbus/dbus.h: include dbus/dbus-protocol.h
+
 2003-01-08  Anders Carlsson  <andersca@codefactory.se>
 
        * dbus/Makefile.am (dbusinclude_HEADERS): Install