+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: