2003-01-15 Havoc Pennington <hp@redhat.com>
[platform/upstream/dbus.git] / ChangeLog
index f9e6132..2f8f766 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,488 @@
+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
+       dbus-connection.h
+
+2003-01-08  Anders Carlsson  <andersca@codefactory.se>
+
+       * dbus/dbus-internals.c: (_dbus_type_to_string):
+       New function that returns a string describing a type.
+       
+       * dbus/dbus-marshal.c: (_dbus_demarshal_byte_array):
+       * dbus/dbus-marshal.h:
+       * dbus/dbus-message.c: (dbus_message_get_fields_valist),
+       (dbus_message_iter_get_field_type), (dbus_message_iter_get_double),
+       (dbus_message_iter_get_byte_array):
+       * dbus/dbus-message.h:
+       Add new convenience functions for appending and getting message fields.
+       Also add demarshalling routines for byte arrays.
+
+2003-01-07  Anders Carlsson  <andersca@codefactory.se>
+
+       * dbus/dbus-connection-internal.h:
+       * dbus/dbus-connection.c: (_dbus_connection_new_for_transport),
+       (_dbus_connection_get_next_client_serial),
+       (dbus_connection_send_message):
+       * dbus/dbus-internals.h:
+       * dbus/dbus-marshal.c: (unpack_uint32), (dbus_unpack_int32),
+       (dbus_pack_int32), (_dbus_marshal_double), (_dbus_marshal_int32),
+       (_dbus_marshal_uint32), (_dbus_demarshal_double),
+       (_dbus_demarshal_int32), (_dbus_demarshal_uint32),
+       (_dbus_demarshal_string), (_dbus_marshal_get_field_end_pos),
+       (_dbus_verbose_bytes), (_dbus_marshal_test):
+       * dbus/dbus-marshal.h:
+       * dbus/dbus-message-internal.h:
+       * dbus/dbus-message.c: (_dbus_message_set_client_serial),
+       (dbus_message_write_header), (_dbus_message_lock),
+       (dbus_message_new), (dbus_message_ref), (dbus_message_unref),
+       (dbus_message_get_name), (dbus_message_append_int32),
+       (dbus_message_append_uint32), (dbus_message_append_double),
+       (dbus_message_append_string), (dbus_message_append_byte_array),
+       (dbus_message_get_fields_iter), (dbus_message_iter_ref),
+       (dbus_message_iter_unref), (dbus_message_iter_has_next),
+       (dbus_message_iter_next), (dbus_message_iter_get_field_type),
+       (dbus_message_iter_get_string), (dbus_message_iter_get_int32),
+       (dbus_message_iter_get_uint32), (dbus_message_iter_get_double),
+       (decode_header_data), (_dbus_message_loader_return_buffer),
+       (message_iter_test), (_dbus_message_test):
+       * dbus/dbus-message.h:
+       * dbus/dbus-protocol.h:
+       * dbus/dbus-test.c: (main):
+       * dbus/dbus-test.h:
+       * glib/test-dbus-glib.c: (message_handler), (main):
+       * test/echo-client.c: (main):
+       * test/watch.c: (check_messages):
+       Make messages sendable and receivable for real.
+       
+2003-01-07  Anders Carlsson  <andersca@codefactory.se>
+
+       * dbus/dbus-marshal.c: (_dbus_marshal_double),
+       (_dbus_marshal_string), (_dbus_marshal_byte_array):
+       * dbus/dbus-message.c: (dbus_message_append_int32),
+       (dbus_message_append_uint32), (dbus_message_append_double),
+       (dbus_message_append_string), (dbus_message_append_byte_array):
+       Handle OOM restoration.
+       
+2003-01-07  Anders Carlsson  <andersca@codefactory.se>
+
+       * dbus/dbus-marshal.c: (_dbus_marshal_string),
+       (_dbus_demarshal_string), (_dbus_marshal_test):
+       * dbus/dbus-marshal.h:
+       * dbus/dbus-message.c: (dbus_message_get_name),
+       Document these functions.
+       
+       (dbus_message_append_int32), (dbus_message_append_uint32),
+       (dbus_message_append_double), (dbus_message_append_string),
+       (dbus_message_append_byte_array):
+       * dbus/dbus-message.h:
+       Add functions for adding message fields of different types.
+       
+       * dbus/dbus-protocol.h:
+       Add the different types.
+
+2003-01-05  Havoc Pennington  <hp@pobox.com>
+
+       * bus/connection.c: implement routines for handling connections,
+       first thing is keeping a list of owned services on each connection
+       and setting up watches etc.
+
+       * bus/services.c: implement a mapping from service names to lists
+       of connections
+
+       * dbus/dbus-hash.c: add DBUS_HASH_POINTER
+
+       * dbus/dbus-threads.c (dbus_static_mutex_lock): add functions
+       to use static mutexes for global data
+
+       * dbus/dbus-connection.c (dbus_connection_set_data): add new
+       collection of functions to set/get application-specific data
+       on the DBusConnection.
+
+2003-01-04  Havoc Pennington  <hp@pobox.com>
+
+       * dbus/dbus-sysdeps.c (_dbus_sleep_milliseconds): new function
+       (_dbus_poll): new function
+
+       * dbus/dbus-internals.h (_DBUS_STRUCT_OFFSET): new macro
+       copied from GLib
+
+       * bus/loop.c: initial code for the daemon main loop
+
+2003-01-04  Havoc Pennington  <hp@pobox.com>
+
+       * test/watch.c (error_handler): make it safe if the error handler 
+       is called multiple times (if we s/error handler/disconnect
+       handler/ we should just guarantee it's called only once)
+
+       * dbus/dbus-transport.c (_dbus_transport_disconnect): call the
+       error handler on disconnect (it's quite possible we should
+       just change the error handler to a "disconnect handler," I'm 
+       not sure we have any other meaningful errors)
+
+       * configure.in: check for getpwnam_r
+
+       * dbus/dbus-transport.c, dbus/dbus-transport-unix.c,
+       dbus/dbus-auth.c: add credentials support, add EXTERNAL auth
+       mechanism as in SASL spec, using socket credentials
+
+       * dbus/dbus-sysdeps.c (_dbus_read_credentials_unix_socket): new function
+       (_dbus_send_credentials_unix_socket): new function
+
+       * dbus/dbus-sysdeps.c (_dbus_accept_unix_socket): rename just
+       dbus_accept()
+       (_dbus_write): only check errno if <0 returned
+       (_dbus_write_two): ditto
+
+2003-01-02  Anders Carlsson  <andersca@codefactory.se>
+
+       * dbus/dbus-marshal.c: (_dbus_marshal_utf8_string),
+       (_dbus_marshal_byte_array), (_dbus_demarshal_utf8_string),
+       (_dbus_marshal_test):
+       * dbus/dbus-marshal.h:
+       Add _dbus_marshal_byte_array and rename _dbus_marshal_string
+       to _dbus_marshal_utf8_string. Also fix some tests.
+       
+2002-12-28  Harri Porten  <porten@kde.org>
+
+       * configure.in: added check for C++ compiler and a very cheesy
+       check for the Qt integration
+
+       * Makefile.am (SUBDIRS): compile qt subdir if support is enabled
+
+       * qt/Makefile.am: added 
+
+       * qt/.cvsignore: added  
+
+       * qt/dbus-qthread.cc, qt/dbus-qthread.cpp: renamed former to
+       latter, added #ifdef QT_THREAD_SUPPORT guard.
+
+       * dbus/Makefile.am: added missing headers for make dist
+
+2002-12-28  Kristian Rietveld  <kris@gtk.org>
+
+       * dbus/Makefile.am: fixup export-symbols-regex.
+
+2002-12-27  Anders Carlsson  <andersca@codefactory.se>
+
+       * acinclude.m4: Add this file and put the 
+       PKG_CHECK_MODULE macro in it.
+
+2002-12-27  Anders Carlsson  <andersca@codefactory.se>
+
+       * dbus/dbus-marshal.c: (_dbus_marshal_string),
+       (_dbus_demarshal_double), (_dbus_demarshal_int32),
+       (_dbus_demarshal_uint32), (_dbus_demarshal_string),
+       (_dbus_marshal_test):
+       Make the demarshalling routines align the pos argument.
+       Add string marshalling tests and fix the obvious bugs 
+       discovered.
+       
+2002-12-26  Havoc Pennington  <hp@pobox.com>
+
+       * dbus/dbus-auth.c: fixes fixes fixes
+
+       * dbus/dbus-transport-unix.c: wire up support for
+       encoding/decoding data on the wire
+
+       * dbus/dbus-auth.c (_dbus_auth_encode_data) 
+       (_dbus_auth_decode_data): append to target string 
+       instead of nuking it.
+
+2002-12-26  Havoc Pennington  <hp@pobox.com>
+
+       * dbus/dbus-marshal.h (DBUS_COMPILER_BYTE_ORDER): #ifdef
+       WORDS_BIGENDIAN then compiler byte order is DBUS_BIG_ENDIAN,
+       doh
+
+       * dbus/dbus-marshal.c: Add macros to do int swapping in-place and
+       avoid swap_bytes() overhead (ignoring possible assembly stuff for 
+       now). Main point is because I wanted unpack_uint32 to implement
+       _dbus_verbose_bytes
+       (_dbus_verbose_bytes): new function
+
+       * dbus/dbus-string.c (_dbus_string_validate_ascii): new function
+
+       * dbus/dbus-message.c (_dbus_message_loader_get_is_corrupted): add
+       mechanism to handle a corrupt message stream
+       (_dbus_message_loader_new): fix preallocation to only prealloc, 
+       not prelengthen
+       
+       * dbus/dbus-string.c (_dbus_string_skip_blank): fix this function
+       (_dbus_string_test): enhance tests for copy/move and fix the
+       functions
+
+       * dbus/dbus-transport-unix.c: Hold references in more places to 
+       avoid reentrancy problems
+
+       * dbus/dbus-transport.c: ditto
+
+       * dbus/dbus-connection.c (dbus_connection_dispatch_message): don't
+       leak reference count in no-message case
+
+       * test/watch.c (do_mainloop): handle adding/removing watches
+       during iteration over the watches. Also, ref the connection/server
+       stored on a watch, so we don't try to mangle a destroyed one.
+
+       * dbus/dbus-transport-unix.c (do_authentication): perform
+       authentication
+
+       * dbus/dbus-auth.c (get_state): add a state
+       AUTHENTICATED_WITH_UNUSED_BYTES and return it if required
+       (_dbus_auth_get_unused_bytes): append the unused bytes
+       to the passed in string, rather than prepend
+
+       * dbus/dbus-transport.c (_dbus_transport_init_base): create 
+       the auth conversation DBusAuth
+
+       * dbus/dbus-transport-unix.c (_dbus_transport_new_for_fd)
+       (_dbus_transport_new_for_domain_socket): when creating a
+       transport, pass in whether it's a client-side or server-side
+       transport so we know which DBusAuth to create
+
+2002-12-03  Havoc Pennington  <hp@pobox.com>
+
+       * dbus/dbus-transport-unix.c (unix_finalize): finalize base
+       _after_ finalizing the derived members
+       (unix_connection_set): unref watch if we fail to add it
+
+       * dbus/dbus-connection.c (dbus_connection_unref): delete the
+       transport first, so that the connection owned by the 
+       transport will be valid as the transport finalizes.
+
+       * dbus/dbus-transport-unix.c (unix_finalize): free the write_watch
+       if necessary, and remove watches from the connection.
+       
+       * dbus/dbus-watch.c (_dbus_watch_list_free): improve a comment
+       
+2002-12-26  Anders Carlsson  <andersca@codefactory.se>
+
+       * dbus/dbus-marshal.c: (_dbus_marshal_string),
+       (_dbus_demarshal_double), (_dbus_demarshal_int32),
+       (_dbus_demarshal_uint32), (_dbus_demarshal_string),
+       (_dbus_marshal_test):
+       * dbus/dbus-marshal.h:
+       Add string marshal functions and have the demarshal functions
+       return the new position.
+       
+2002-12-25  Havoc Pennington  <hp@pobox.com>
+
+       * doc/dbus-sasl-profile.txt: docs on the authentication protocol, 
+       it is a simple protocol that just maps directly to SASL.
+
+       * dbus/dbus-auth.h, dbus/dbus-auth.c: authentication protocol
+       initial implementation, not actually used yet.
+       
+       * dbus/dbus-string.c (_dbus_string_find): new function
+       (_dbus_string_equal): new function
+       (_dbus_string_base64_encode): new function
+       (_dbus_string_base64_decode): new function
+
+2002-12-25  Anders Carlsson  <andersca@codefactory.se>
+
+       * dbus/Makefile.am:
+       * dbus/dbus-marshal.c: (swap_bytes), (_dbus_marshal_double),
+       (_dbus_marshal_int32), (_dbus_marshal_uint32),
+       (_dbus_demarshal_double), (_dbus_demarshal_int32),
+       (_dbus_demarshal_uint32), (_dbus_marshal_test):
+       * dbus/dbus-marshal.h:
+       * dbus/dbus-protocol.h:
+       * dbus/dbus-test.c: (main):
+       * dbus/dbus-test.h:
+       Add un-optimized marshalling/demarshalling routines.
+       
+2002-12-25  Harri Porten  <porten@kde.org>
+
+       * qt/dbus-qt.h: adjusted ctor and getter to KDE/Qt conventions
+
+2002-12-24  Zack Rusin  <zack@kde.org>
+
+       * qt/dbus-qthread.cc: adding - integrates QMutex into Dbus
+       * qt/dbus-qt.h: skeleton with two sample implemenatation of the
+       main loop stuff
+
+2002-12-24  Havoc Pennington  <hp@pobox.com>
+
+       * glib/dbus-gthread.c: fix include
+
+       * glib/dbus-glib.h: rename DBusMessageHandler for now. 
+       I think glib API needs to change, though, as you don't 
+       want to use DBusMessageFunction, you want to use the 
+       DBusMessageHandler object. Probably 
+       dbus_connection_open_with_g_main_loop()
+       and dbus_connection_setup_g_main_loop() or something like that
+       (but think of better names...) that just create a connection 
+       that has watch/timeout functions etc. already set up.
+
+       * dbus/dbus-connection.c
+       (dbus_connection_send_message_with_reply): new function just to 
+       show how the message handler helps us deal with replies.
+
+       * dbus/dbus-list.c (_dbus_list_remove_last): new function
+
+       * dbus/dbus-string.c (_dbus_string_test): free a string that
+       wasn't
+
+       * dbus/dbus-hash.c: use memory pools for the hash entries
+       (rebuild_table): be more paranoid about overflow, and 
+       shrink table when we can
+       (_dbus_hash_test): reduce number of sprintfs and write
+       valid C89. Add tests for case where we grow and then 
+       shrink the hash table.
+
+       * dbus/dbus-mempool.h, dbus/dbus-mempool.c: memory pools
+
+       * dbus/dbus-connection.c (dbus_connection_register_handler) 
+       (dbus_connection_unregister_handler): new functions
+
+       * dbus/dbus-message.c (dbus_message_get_name): new
+
+       * dbus/dbus-list.c: fix docs typo
+
+       * dbus/dbus-message-handler.h, dbus/dbus-message-handler.c:
+       an object representing a handler for messages.
+
+2002-12-16  Anders Carlsson  <andersca@codefactory.se>
+
+       * glib/dbus-glib.h:
+       * glib/dbus-gthread.c: (dbus_gthread_init):
+       Don't use the gdbus prefix for public functions.
+       
+2002-12-16  Anders Carlsson  <andersca@codefactory.se>
+
+       * Makefile.am:
+       * configure.in:
+       Add GLib checks and fixup .pc files
+       
+       * glib/Makefile.am:
+       * glib/dbus-glib.h:
+       * glib/dbus-gmain.c: (gdbus_connection_prepare),
+       (gdbus_connection_check), (gdbus_connection_dispatch),
+       (gdbus_add_connection_watch), (gdbus_remove_connection_watch),
+       (dbus_connection_gsource_new):
+       * glib/dbus-gthread.c: (dbus_gmutex_new), (dbus_gmutex_free),
+       (dbus_gmutex_lock), (dbus_gmutex_unlock), (dbus_gthread_init):
+       * glib/test-dbus-glib.c: (message_handler), (main):
+       Add GLib support.
+       
+2002-12-15  Harri Porten  <porten@kde.org>
+
+       * autogen.sh: check for libtoolize before attempting to use it 
+       
+       * dbus/dbus-transport-unix.c: include <sys/time.h> for timeval
+       struct.
+       
+       * .cvsignore: ignore more stamp files
+
+       * dbus/dbus-watch.c (_dbus_watch_list_new): fixed doc error
+
+       * test/Makefile.am: added -I$(top_srcdir) to be able to compile
+       without make install.
+
+2002-12-15  Havoc Pennington  <hp@pobox.com>
+
+       * dbus/dbus-threads.c: add thread stubs that a higher library
+       layer can fill in. e.g. the GLib wrapper might fill them in with
+       GThread stuff. We still need to use this thread API to
+       thread-safe-ize the library.
+
+2002-12-12  Havoc Pennington  <hp@pobox.com>
+
+       * dbus/dbus-transport-unix.c, dbus/dbus-server-unix.c: use the
+       below new interfaces and include fewer system headers.
+
+       * dbus/dbus-sysdeps.c (_dbus_read): new function
+       (_dbus_write): new function
+       (_dbus_write_two): new function
+       (_dbus_connect_unix_socket): new function
+       (_dbus_listen_unix_socket): new function
+
+       * dbus/dbus-message-internal.h: change interfaces to use
+       DBusString
+
 2002-12-11  Havoc Pennington  <hp@pobox.com>
 
        * dbus/dbus-types.h: add dbus_unichar