2005-01-12  Joe Shaw  <joeshaw@novell.com>

* NEWS: Update for 0.23.

* configure.in: Release 0.23.

2005-01-12  Joe Shaw  <joeshaw@novell.com>

* mono/Makefile.am, mono/example/Makefile.am: Always build the
dbus DLL with --debug.  Clean up after the .mdb files this leaves
behind.

2005-01-12  Joe Shaw  <joeshaw@novell.com>

* mono/Makefile.am, mono/example/Makefile.am: Always build the
dbus DLL with --debug.  Clean up after the .mdb files this leaves
behind.

2005-01-12  Joe Shaw  <joeshaw@novell.com>

* mono/doc/Makefile.am: Need to uninstall the docs on "make
uninstall"

2005-01-12  Joe Shaw  <joeshaw@novell.com>

* mono/Makefile.am, mono/example/Makefile.am: Always build the
dbus DLL with --debug.

2005-01-12  Joe Shaw  <joeshaw@novell.com>

* mono/Makefile.am: Always build the dbus DLL with --debug.

* mono/Arguments.cs (GetDBusTypeConstructor): If the type
is an enum, get the enum's underlying type.  Another mono
1.1.3 fix.

fix doh! in test for double parsing

2005-01-11  Joe Shaw  <joeshaw@novell.com>

Patch from Sjoerd Simons <sjoerd@luon.net>

* mono/Makefile.am, mono/example/Makefile.am: Don't redefine
DESTDIR.  It breaks stuff.

2005-01-11  Joe Shaw  <joeshaw@novell.com>

Patch from Tambet Ingo <tambet@ximian.com>

* mono/DBusType/Array.cs (Get): Get the underlying element type by
calling type.GetElementType().  The code previously depended on
broken Mono behavior, which was fixed in Mono 1.1.3.

* mono/DBusType/Dict.cs (constructor): Fix the parameters for
Activator.CreateInstance() so that the class's constructor is
called with the right parameters.

2005-01-11  Joe Shaw  <joeshaw@novell.com>

Patch from Timo Teräs <ext-timo.teras@nokia.com>

* dbus/dbus-connection.c
(_dbus_connection_queue_received_message_link): Call
_dbus_connection_remove_timeout() instead of the _locked()
variant, since it's always called from
_dbus_connection_handle_watch(), which handles the locking.
Removed the _locked() variant since it's no longer used.

2005-01-03  Havoc Pennington  <hp@redhat.com>

* dbus/dbus-internals.h: I'm an idiot, _dbus_assert certainly can
return

2004-12-26  Havoc Pennington  <hp@redhat.com>

* dbus/dbus-internals.h: add _DBUS_GNUC_NORETURN to _dbus_assert

add a file to hold the validation routines

2005-01-03  Havoc Pennington  <hp@redhat.com>

* dbus/dbus-sysdeps.c (_dbus_sysdeps_test): fix using == on
floating point

* dbus/dbus-string.c (_dbus_string_insert_alignment): new function

finish off my TODO list for stuff needed to port dbus-message.c. Next
step is to do so.

just when the file was looking all beautiful, add horrible hacky code to
fixup array lengths after setting a value somewhere within the array.

cleanup args to read func in test suite

checkpoint set_variable_length work

values-only DBusTypeWriter

minor cleanup

clean up the naming of dbus_marshal functions

delete some more noise, put args in consistent order (a big bug trap sadly),
and work on the set value stuff some

cleanups and fixes

implement _dbus_type_writer_write_reader() (to copy a block of values)
which is the first step toward a sane reimplementation of all that delete/modify
and realign nonsense in dbus-message.c

change strings to be marshaled consistently with the other basic types (address of the value passed in, so char**)

make reader const-correct

extend unit test to display progress and to verify that we are inserting not appending in all cases

add DBusTypeMark

2005-01-01  Havoc Pennington  <hp@redhat.com>

* configure.in: add -Wfloat-equal

chop dbus-marshal-basic in half and move it to be insertion rather than append based

voc Pennington  <hp@redhat.com>

* dbus/dbus-sysdeps.h: add _DBUS_DOUBLES_BITWISE_EQUAL macro,
for a variety of reasons '==' doesn't do this.

all types now unit tested and working.

add more "subclassing" to the test node classes

yay, variant values working before the new year.

minor tweaking

add int64 to test suites, fix minor bug where marshaling assumed that DBusOctets8 was aligned when passed in as a function argument

remove old unit test code, and clean up whitespace

test framework extension

further extend tests

new test framework in place

2004-12-31  Havoc Pennington  <hp@redhat.com>

* dbus/dbus-string.c (_dbus_string_equal_substrings): new function
I keep wishing I had

use vtable instead of flags for types only iterators

add "types only" TypeReader, will move to use a vtable instead of a flag in a minute

- add variant reader
- further squish the iterator structs

store writer container_type in one byte to shrink iterator

remove unused element_type_len field

- remove DICT type
- add SIGNATURE type
- fix max name len of 256, should be 255 so it fits in a byte if it's going to be a number
  near 256
- generalize some of the recursive type marshaling code to share between arrays and variants
- just started on implementing variant

2004-12-30  John (J5) Palmieri  <johnp@redhat.com>

* python/dbus.py: s/ACTIVATION_REPLY_ACTIVE/ACTIVATION_REPLY_ACTIVATED

2004-12-30  John (J5) Palmieri  <johnp@redhat.com>

        * python/dbus_bindings.pyx.in: Change DBUS_ACTIVATION_REPLY_ACTIVATED
        and DBUS_ACTIVATION_REPLY_ALREADY_ACTIVE to match the values in
        dbus-protocol.h.  Because they are defines and not enums they are not
        autogenerated.

DBusTypeWriterClass isn't going to work well, so remove vestiges of it.

switch to a vtable approach for the reader

put it in the test suite arrays (ok, I need to rewrite the unit tests here to be more automated...)

struct of array of struct and array of struct of array

checkpoint with array-of-struct working

arrays are working, woot

sync busted marshaling work in progress

2004-12-26  John (J5) Palmieri  <johnp@redhat.com>

* python/dbus_bindings.pyx.in (bus_activate_service): Bind
dbus_bus_activate_service

* python/dbus.py (Bus.activate_service): activate a service on the
bus.

2004-12-24  Havoc Pennington  <hp@redhat.com>

* test/decode-gcov.c: change to use .gcno and .gcda files, but the
file format has also changed and I haven't adapted to that yet

* Makefile.am: load .gcno files from latest gcc

* Forgot to add ChangeLog on last commit

* patch from Rob Taylor <robtaylor@fastmail.fm>
- wrap bus_get_unix_user method in low level bindings
- add get_unix_user method to the Bus class
- fix extract.py so it can handle unsigned long return types

* dbus/make-dbus-glib-error-enum.sh: omit the function keyword for
  better POSIX compliance.

2004-12-19  Havoc Pennington  <hp@redhat.com>

* dbus/dbus-string.c (_dbus_string_insert_4_aligned)
(_dbus_string_insert_8_aligned): new functions

* dbus/dbus-string.c (_dbus_string_alloc_space): new function

back up some files I'm working on that don't do anything yet

2004-12-18  Havoc Pennington  <hp@redhat.com>

* dbus/dbus-string.c (_dbus_string_validate_ascii): use ISASCII
macro

* dbus/dbus-message.c: fix a comment, and add a still-unused
not-implemented function

* dbus/dbus-marshal.h: fix comment

* dbus/dbus-internals.h (_DBUS_ISASCII): new macro

2004-12-17  Joe Shaw  <joeshaw@novell.com>

* mono/DBusType/Byte.cs, mono/DBusType/Int32.cs,
mono/DBusType/Int64.cs, mono/DBusType/UInt32.cs,
mono/DBusType/UInt64.cs: Use Enum.GetUnderlyingType() instead of
Type.UnderlyingSystemType to get the actual system type
underneath.  This code previously depended on the broken Mono
behavior, which was fixed in 1.1.3.

2004-11-27  Havoc Pennington  <hp@redhat.com>

* dbus/dbus-string.h (_dbus_string_get_byte): inline when asserts
are disabled
(_dbus_string_get_const_data): inline when asserts are disabled

* dbus/dbus-message.c: record the _dbus_current_generation of
creation so we can complain if dbus_shutdown() is used improperly.
Do this only if checks are enabled.

* dbus/dbus-connection.c: ditto

2004-11-26  Havoc Pennington  <hp@redhat.com>

* test/glib/test-profile.c: add with_bus mode to profile echoes
that go through the bus.

* test/glib/run-test.sh: add ability to run test-profile

* bus/dbus-daemon-1.1.in: fix to say that SIGHUP causes partial
config file reload.

2004-11-26  Havoc Pennington  <hp@redhat.com>

* test/glib/test-profile.c: clean up how the fake_malloc_overhead
thing was implemented

2004-11-26  Havoc Pennington  <hp@redhat.com>

* test/glib/test-profile.c: tweak a bit, add support for some
made-up minimal malloc overhead with plain sockets, since in
real life some sort of buffers are unavoidable thus we could
count them in the theoretical best case

2004-11-26  Havoc Pennington  <hp@redhat.com>

* dbus/dbus-message.c (dbus_message_cache_or_finalize): fix bug
where I was trying to cache one too many messages

2004-11-26  Havoc Pennington  <hp@redhat.com>

* dbus/dbus-message.c: reimplement message cache as an array which
makes the cache about twice as fast and saves maybe 1.5% overall

2004-11-26  Havoc Pennington  <hp@redhat.com>

* dbus/dbus-threads.c (init_global_locks): forgot to put the
message cache lock here

2004-11-26  Havoc Pennington  <hp@redhat.com>

* dbus/dbus-message.c (struct DBusMessage): put the locked bit and
the "char byte_order" next to each other to save 4 bytes
(dbus_message_new_empty_header): reduce preallocation, since the
message cache should achieve a similar effect
(dbus_message_cache_or_finalize, dbus_message_get_cached): add a
message cache that keeps a few DBusMessage around in a pool,
another 8% speedup or so.

* dbus/dbus-dataslot.c (_dbus_data_slot_list_clear): new function

2004-11-25  Havoc Pennington  <hp@redhat.com>

* dbus/dbus-transport-unix.c (unix_do_iteration): if we're going
to write, without reading or blocking, try it before the poll()
and skip the poll() if nothing remains to write. This is about a
3% speedup in the echo client/server

2004-11-25  Havoc Pennington  <hp@redhat.com>

        The primary change here is to always write() once before adding
the write watch, which gives us about a 10% performance increase.

* dbus/dbus-transport-unix.c: a number of modifications to cope
with removing messages_pending
(check_write_watch): properly handle
DBUS_AUTH_STATE_WAITING_FOR_MEMORY; adapt to removal of
messages_pending stuff
(check_read_watch): properly handle WAITING_FOR_MEMORY and
AUTHENTICATED cases
(unix_handle_watch): after writing, see if the write watch can be
removed
(unix_do_iteration): assert that write_watch/read_watch are
non-NULL rather than testing that they aren't, since they
aren't allowed to be NULL. check_write_watch() at the end so
we add the watch if we did not finish writing (e.g. got EAGAIN)

* dbus/dbus-transport-protected.h: remove messages_pending call,
since it resulted in too much inefficient watch adding/removing;
instead we now require that the transport user does an iteration
after queueing outgoing messages, and after trying the first
write() we add a write watch if we got EAGAIN or exceeded our
max bytes to write per iteration setting

* dbus/dbus-string.c (_dbus_string_validate_signature): add this
function

* dbus/dbus-server-unix.c (unix_finalize): the socket name was
freed and then accessed, valgrind flagged this bug, fix it

* dbus/dbus-message.c: fix several bugs where HEADER_FIELD_LAST was taken
as the last valid field plus 1, where really it is equal to the
last valid field. Corrects some message corruption issues.

* dbus/dbus-mainloop.c: verbosity changes

* dbus/dbus-keyring.c (_dbus_keyring_new_homedir): handle OOM
instead of aborting in one of the test codepaths

* dbus/dbus-internals.c (_dbus_verbose_real): fix a bug that
caused not printing the pid ever again if a verbose was missing
the newline at the end
(_dbus_header_field_to_string): add HEADER_FIELD_SIGNATURE

* dbus/dbus-connection.c: verbosity changes;
(dbus_connection_has_messages_to_send): new function
(_dbus_connection_message_sent): no longer call transport->messages_pending
(_dbus_connection_send_preallocated_unlocked): do one iteration to
try to write() immediately, so we can avoid the write watch. This
is the core purpose of this patchset
(_dbus_connection_get_dispatch_status_unlocked): if disconnected,
dump the outgoing message queue, so nobody will get confused
trying to send them or thinking stuff is pending to be sent

* bus/test.c: verbosity changes

* bus/driver.c: verbosity/assertion changes

* bus/dispatch.c: a bunch of little tweaks to get it working again
because this patchset changes when/where you need to block.

2004-11-23  Havoc Pennington  <hp@redhat.com>

* test/glib/test-profile.c: modify to accept a plain_sockets
argument in which case it will bench plain sockets instead of
libdbus, for comparison purposes.

2004-11-22  Havoc Pennington  <hp@redhat.com>

* test/glib/test-profile.c (N_CLIENT_THREADS): run multiple
threads for more time, so sysprof can get a grip on it.

* dbus/dbus-string.c (_dbus_string_validate_utf8): remove
pointless variable

2004-11-13  Havoc Pennington  <hp@redhat.com>

* test/glib/test-profile.c: fix this thing up a bit

* dbus/dbus-message.c (dbus_message_new_empty_header): increase
preallocation sizes by a fair bit; not sure if this will be an
overall performance win or not, but it does reduce reallocs.

* dbus/dbus-string.c (set_length, reallocate_for_length): ignore
the test hack that forced constant realloc if asserts are
disabled, so we can profile sanely. Sprinkle in some
_DBUS_UNLIKELY() which are probably pointless, but before I
noticed the real performance problem I put them in.
(_dbus_string_validate_utf8): micro-optimize this thing a little
bit, though callgrind says it didn't help; then special-case
ascii, which did help a lot; then be sure we detect nul bytes as
invalid, which is a bugfix.
(align_length_then_lengthen): add some more _DBUS_UNLIKELY
superstition; use memset to nul the padding instead of a manual
loop.
(_dbus_string_get_length): inline this as a
macro; it showed up in the profile because it's used for loop
tests and so forth

add a little comment about WNOHANG/EINTR

2004-11-10  Colin Walters  <walters@verbum.org>

* dbus/dbus-spawn.c (check_babysit_events): Handle EINTR,
for extra paranoia.

2004-11-09  Colin Walters  <walters@verbum.org>

* dbus/dbus-string.c (_dbus_string_get_length): New
function, writes DBusString to C buffer.

* dbus/dbus-string.h: Prototype it.

* dbus/dbus-message.c (dbus_message_type_to_string): New
function, converts message type into C string.

* dbus/dbus-message.h: Prototype it.

* bus/selinux.c (bus_selinux_check): Take source pid,
target pid, and audit data.  Pass audit data to
avc_has_perm.
(log_audit_callback): New function, appends extra
audit information.
(bus_selinux_allows_acquire_service): Also take
service name, add it to audit data.
(bus_selinux_allows_send): Also take message
type, interface, method member, error name,
and destination, and add them to audit data.
(log_cb): Initialize func_audit.

* bus/selinux.h (bus_selinux_allows_acquire_service)
(bus_selinux_allows_send): Update prototypes

* bus/services.c (bus_registry_acquire_service): Pass
service name to bus_selinux_allows_acquire_service.

* bus/bus.c (bus_context_check_security_policy): Pass
additional audit data.  Move assignment of dest
to its own line.

2004-11-07  Colin Walters  <walters@verbum.org>

* dbus/dbus-transport-unix.c (do_authentication): Always
initialize auth_completed.

2004-11-07  Colin Walters  <walters@verbum.org>

* bus/bus.c (load_config): Break into three
separate functions: process_config_first_time_only,
process_config_every_time, and process_config_postinit.
(process_config_every_time): Move call of
bus_registry_set_service_context_table into
process_config_postinit.
(process_config_postinit): New function, does
any processing that needs to happen late
in initialization (and also on reload).
(bus_context_new): Instead of calling load_config,
open config parser here and call process_config_first_time_only
and process_config_every_time directly.  Later, after
we have forked but before changing UID,
invoke bus_selinux_full_init, and then call
process_config_postinit.
(bus_context_reload_config): As in bus_context_new,
load parse file inside here, and call process_config_every_time
and process_config_postinit.

* bus/services.h, bus/services.c
(bus_registry_set_service_context_table): Rename
from bus_registry_set_sid_table.  Take string hash from config
parser, and convert them here into SIDs.

* bus/config-parser.c (struct BusConfigParser): Have
config parser only store a mapping of service->context
string.
(merge_service_context_hash): New function.
(merge_included): Merge context string hashes instead
of using bus_selinux_id_table_union.
(bus_config_parser_new): Don't use bus_selinux_id_table_new;
simply create a new string hash.
(bus_config_parser_unref): Unref it.
(start_selinux_child): Simply insert strings into hash,
don't call bus_selinux_id_table_copy_over.

* bus/selinux.h, bus/selinux.c (bus_selinux_id_table_union)
(bus_selinux_id_table_copy_over): Delete.

*** empty log message ***

2004-11-03  Colin Walters  <walters@verbum.org>

* bus/selinux.c (bus_selinux_pre_init): Kill some unused
variables.

2004-11-03  Colin Walters  <walters@verbum.org>

* bus/test-main.c (test_pre_hook): Fix test logic,
thanks Joerg Barfurth <Joerg.Barfurth@Sun.COM>.

*** empty log message ***

2004-11-02  Colin Walters  <walters@verbum.org>

* bus/selinux.c (bus_selinux_init): Split into two functions,
bus_selinux_pre_init and bus_selinux_post_init.
(bus_selinux_pre_init): Just determine whether SELinux is
enabled.
(bus_selinux_post_init): Do everything else.

* bus/main.c (main): Call bus_selinux_pre_init before parsing
config file, and bus_selinux_post_init after.  This ensures that
we don't lose the policyreload notification thread that
bus_selinux_init created before forking previously.

* bus/test-main.c (test_pre_hook): Update for split.

Patch from Johan Fischer to make mono doc install create the doc dir.

2004-10-29  Colin Walters  <walters@redhat.com>

* dbus/dbus-sysdeps.h (_dbus_become_daemon): Also take
parameter for fd to write pid to.

* dbus/dbus-sysdeps.c (_dbus_become_daemon): Implement it.

* bus/bus.c (bus_context_new): Pass print_pid_fd
to _dbus_become_daemon (bug #1720)

2004-10-29  Colin Walters  <walters@redhat.com>

Patch from Ed Catmur <ed@catmur.co.uk>

* mono/doc/Makefile.am (install-data-local): Handle
DESTDIR.

2004-10-29  Colin Walters  <walters@redhat.com>

* bus/.cvsignore, qt/.cvsignore: Update.

2004-10-29  Colin Walters  <walters@redhat.com>

Patch from Kristof Vansant <de_lupus@pandora.be>

* configure.in: Detect Slackware.
* bus/Makefile.am (SCRIPT_IN_FILES): Add rc.messagebus.in.
* bus/rc.messagebus.in: New file.

Add comment about workaround.

2004-10-29  Colin Walters  <walters@redhat.com>

* tools/dbus-monitor.c (filter_func): Return
DBUS_HANDLER_RESULT_HANDLED in filter function
for now.  See:
http://freedesktop.org/pipermail/dbus/2004-August/001433.html

2004-10-29  Colin Walters  <walters@redhat.com>

Patch from Matthew Rickard <mjricka@epoch.ncsc.mil>

* bus/services.c (bus_registry_acquire_service):
Correctly retrieve service name from DBusString
for printing.

2004-10-29 Colin Walters  <walters@redhat.com>

* dbus/dbus-glib.h: Update documentation to not
refer to internal APIs.

2004-10-27  Joe Shaw  <joeshaw@novell.com>

* mono/Arguments.cs (GetDBusTypeConstructor):
type.UnderlyingSystemType will return "System.Byte" if you do it
on "byte[]", which is not what we want.  So check the type.IsArray
property and use System.Array instead.

John (J5) Palmieri  <johnp@redhat.com>

        * dbus/dbus-sysdeps.c (fill_user_info): On errors do not free
        the DBusUserInfo structure since this is passed into the function.
        This would cause a double free when the function that allocated
        the structure would try to free it when an error occured.

        * (bus/session.conf.in, bus/Makefile.am, dbus/configure.in):
        use /usr/share/dbus-1/services instead of /usr/lib/dbus-1.0/services
        for service activation to avoid 32bit/64bit parallel install issues

2004-10-21  Colin Walters  <walters@verbum.org>

* AUTHORS: Fix my email address, the @gnu.org one
has been bouncing for some time.  Also add J5.