summaryrefslogtreecommitdiffstats
path: root/dbus
Commit message (Collapse)AuthorAgeFilesLines
...
* 2006-09-16 Havoc Pennington <hp@redhat.com>Havoc Pennington2006-09-162-27/+46
| | | | | | | | | | | | * dbus/dbus-sysdeps-unix.c (_dbus_open_tcp_socket) (_dbus_open_unix_socket, _dbus_open_socket): change API for _dbus_open_socket so the domain/type/protocol from system headers are not required. This is kind of pointless though unless we move part of _dbus_connect_tcp_socket into sysdeps.c instead of sysdeps-unix.c, which would mean adding a wrapper around bind/listen/etc. Also, add DBusError to the socket-opening functions so they don't require use of errno.
* 2006-09-16 Havoc Pennington <hp@redhat.com>Havoc Pennington2006-09-168-79/+150
| | | | | | | | | | * dbus/dbus-sysdeps-unix.h: small change to Peter's patch to make dbus-sysdeps-unix-util.c build, add unix-specific sysdeps header. * dbus/dbus-sysdeps.h, dbus-sysdeps-unix.c: patch from Peter Kümmel bug #8249 to make the sysdeps.h read/write/open/close functions specifically for sockets only, and move generic read/write/open/close into unix-specific code.
* 2006-09-14 Havoc Pennington <hp@redhat.com>Havoc Pennington2006-09-141-0/+6
| | | | | * dbus/dbus-transport-unix.c (unix_do_iteration): fix a valgrind complaint about accessing revents when poll returns 0, from Julian Seward.
* * dbus/dbus-sysdeps-util-unix.c: Forgot to pass NULL as the secondJohn (J5) Palmieri2006-09-141-1/+1
| | | | parameter to _dbus_close
* * dbus/dbus-sysdeps-util-unix.c (_dbus_write_pid_file):John (J5) Palmieri2006-09-141-1/+1
| | | | use _dbus_close instead of close
* * dbus/dbus-sysdeps.c: Add support for LOCAL_CREDS socketJohn (J5) Palmieri2006-09-141-13/+58
| | | | | | credentials. Fixes "external" authentication under e.g. NetBSD which does not support any other socket credentials mechanism. (Patch from Julio M. Merino Vidal <jmmv at NetBSD dot org>)
* * dbus/dbus-server.c: remove a todo itemJohn (J5) Palmieri2006-09-141-1/+1
|
* * dbus/dbus-threads.c: Allow recursive mutex's to be passed intoJohn (J5) Palmieri2006-09-145-43/+101
| | | | | | | dbus_threads_init and be used by the dbus mutex functions to avoid deadlocks. * doc/TODO: Remove recursive mutex dbus_connection_dispatch TODO item
* * dbus/dbus-sysdeps-util-unix.c (_dbus_directory_get_next_file):John (J5) Palmieri2006-09-131-12/+67
| | | | use threadsafe readdir_r instead of readdir
* * dbus-threads.c (dbus_threads_init_default): New method forJohn (J5) Palmieri2006-09-132-1/+394
| | | | | initializing the internal thread implementation (Patch from Alexander Larsson <alexl at redhat dot com>)
* * remove a bunch of todo items from the 1.0 listJohn (J5) Palmieri2006-09-117-15/+10
|
* * dbus/dbus-internal.c: Add dbus_is_verbose so we can have moreJohn (J5) Palmieri2006-09-113-66/+75
| | | | | | | | complex debugging code * dbus/dbus-marshal-basic.c (_dbus_marshal_read_fixed_multi): Move between the test suite ifdefs (_dbus_verbose_bytes): return if verbosity is not enabled
* * dbus/dbus-marshal-recursive-util.c, dbus/dbus-marshal-recursive.c:John (J5) Palmieri2006-09-113-163/+9
| | | | remove DBusMark
* 2006-09-10 Havoc Pennington <hp@redhat.com>Havoc Pennington2006-09-105-2167/+2229
| | | | | | | | | | | | patch mostly by Peter Kümmel, bug #8211 * dbus/dbus-sysdeps-unix.c: new file, which splits out unix-specific stuff in dbus-sysdeps.c * dbus/dbus-sysdeps.c: don't include any UNIX-only headers, and move unix-specific stuff to dbus-sysdeps-unix.c * configure.in: check HAVE_ERRNO_H
* * bus/test-main.c (main): Initialize threading during testsJohn (J5) Palmieri2006-09-081-7/+59
| | | | | | | | | | * dbus/dbus-connection.c (_dbus_connection_new_for_transport): Unlock connection on error (generate_local_error_message): static method for generating an error message when we don't have a message to reply to (_dbus_connection_block_pending_call): Send a disconnect error instead of just a timeout (NULL) when the bus gets disconnected while blocking for a reply.
* * dbus/dbus-connection.c (dbus_connection_dispatch): Properly removeJohn (J5) Palmieri2006-09-081-0/+7
| | | | filters (Patch from Kimmo Hämäläinen <kimmo dot hamalainen at nokia dot com>)
* * dbus/dbus-connection-internal.h: Add prototype forJohn (J5) Palmieri2006-09-071-0/+6
| | | | _dbus_connection_test_get_locks to remove warning
* * dbus/dbus-spawn.c (_dbus_spawn_async_with_babysitter): Make sure weJohn (J5) Palmieri2006-09-071-1/+2
| | | | | close child_err_report_pipe[WRITE_END] on exec (Patch from Kimmo Hämäläinen <kimmo dot hamalainen at nokia dot com>)
* * Clean up close calls to use _dbus_close (Patch fromJohn (J5) Palmieri2006-09-073-39/+39
| | | | Kimmo Hämäläinen <kimmo dot hamalainen at nokia dot com>)
* * doc/TODO:John (J5) Palmieri2006-09-064-20/+68
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Remove pending call locking todo item - dbus_connection_open now holds hard ref. Remove todo item - do proper locking on _dbus_bus_check_connection_and_unref and handle DBUS_BUS_STARTER. Remove todo item - Warn on closing of a shared connection. Remove todo item * bus/bus.c, bus/connection.c, bus/dispatch.c, dbus/dbus-bus.c, dbus/dbus-connection.c: Use the dbus_connection_close_internal so we don't get the warning when closing shared connections * test/test-service.c, test/test-shell-service.c: Applications don't close shared connections themselves so we unref instead of close * test/test-utils.c (test_connection_shutdown): Close the connection * dbus/dbus-bus.c (_dbus_bus_check_connection_and_unref): Changed to _dbus_bus_check_connection_and_unref_unlocked since we only call this method on a locked connection. Make sure we call _dbus_connection_unref_unlocked instead of dbus_connection_unref also. Handle DBUS_BUS_STARTER correctly * dbus/dbus-connection.c (connection_record_shared_unlocked): Mark as shared and hard ref the connection (connection_forget_shared_unlocked): Remove the hard ref from the connection (_dbus_connection_close_internal_and_unlock): New internal function which takes a locked connection and unlocks it after closing it (_dbus_connection_close_internal): New internal function which acts like the origonal dbus_connection_close method by grabbing a connection lock and calling _dbus_connection_close_internal_and_unlock (dbus_connection_close): Public close method, warns when the app trys to close a shared connection
* * doc/TODO, various source files: Audited todo's and FIXME's andJohn (J5) Palmieri2006-09-0610-21/+20
| | | | | | prepended the ones we should be looking at with 1.0. Those prepended with 1.0? need clerification or might not be needed for 1.0
* * dbus/dbus-pending-call.c: Add some checks so we don't crash onJohn (J5) Palmieri2006-09-051-0/+10
| | | | | NULL in the public API (Patch from Kimmo Hämäläinen <kimmo dot hamalainen at nokia dot com>)
* 2006-08-29 Havoc Pennington <hp@redhat.com>Havoc Pennington2006-08-302-25/+36
| | | | | | | | | | * test/test-service.c (path_message_func): fix lack of return value * dbus/dbus-sysdeps.c (_dbus_printf_string_upper_bound): fix formatting, remove #ifdef, and fix docs. #ifdef doesn't make any more sense than on anything else in this file. (_dbus_get_tmpdir): add const to return value, and keep the results of the various getenv around in a static variable.
* 2006-08-29 Havoc Pennington <hp@redhat.com>Havoc Pennington2006-08-303-714/+742
| | | | | * dbus/dbus-sysdeps-util.c, dbus/dbus-sysdeps-util-unix.c: change from Ralf Habacker to move UNIX-specific sysdeps into a separate file.
* * dbus/dbus-transport.c: fix invalid deref when checking ifJohn (J5) Palmieri2006-08-251-1/+1
| | | | | a vtable method exists (Patch from Christian Ehrlicher <ch dot ehrlicher at gmx dot de>)
* * configure.in, dbus/Makefile.am, test/name-test/run-test.sh,John (J5) Palmieri2006-08-251-1/+1
| | | | | | test/name-test/Makefile.am: Fixed some issues with getting get dbus to build with builddir != srcdir (Taken from a patch by Bernard Leak <bernard at brenda-arkle dot demon dot co dot uk>)
* * Released 0.92dbus-0.92John (J5) Palmieri2006-08-181-5/+3
| | | | | | | | * dbus/dbus-threads.c (dbus_threads_init): change the documentation to reflect the init late change * bus/bus.c (bus_context_new): Check user before we fork so we can print out an error message a user will be able to see
* Patch provided by Ralf Habacker (ralf dot habacker at freenet dot de)John (J5) Palmieri2006-08-183-1/+8
| | | | | | | | | * dbus/dbus-sysdeps.c, dbus/dbus-threads.c, dbus/dbus-internals.h: Add two more global locks for use on windows platforms. These are unused on non-windows platforms but are not ifdefed out to avoid potential bugs (i.e. the few bytes lost does not warrent the extra maintanence and complexity that having seperate sets of locks would cause)
* * dbus/dbus-threads.c: Add static DBusList *uninitialized_mutex_list andJohn (J5) Palmieri2006-08-168-116/+295
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | static DBusList *uninitialized_condvar_list to support new late initialization threading model. In this model threads can be initialized even after the D-Bus API has been used but still needs to be initialized before the second thread has been started. Mutexes and condvar addresses are stored in the two static lists and are replaced with actuall locks when threads are initalized. (_dbus_mutex_new_at_location): New method for creating a mutex and placing the location into the static list (_dbus_mutex_free_at_location): New method for removing a mutex location from the static list and freeing the mutex (_dbus_condvar_new_at_location): New method for creating a conditional variable and placing the location into the static list (_dbus_condvar_free_at_location): New method for removing a conditional variable location from the static list and freeing the conditional variable (init_uninitialized_locks): Atomic method which goes through the static lists of mutex and condvar location and updates them with actuall locks (init_global_locks): changed to init_locks * dbus/dbus-connection.c: (_dbus_connection_test_get_locks): New method for tests to check connections (_dbus_connection_new_for_transport): Use the new at_location mutex and condvar API (dbus_connection_allocate_data_slot): Pass in the global lock address to _dbus_data_slot_allocator_alloc * dbus/dbus-dataslot.c: (_dbus_data_slot_allocator_alloc): Use the address of the mutex instead of the mutex itself * dbus/dbus-message.c: (dbus_message_allocate_data_slot): Pass in the global lock address to _dbus_data_slot_allocator_alloc * dbus/dbus-pending-call.c: (dbus_pending_call_allocate_data_slot): Pass in the global lock address to _dbus_data_slot_allocator_alloc * dbus/dbus-server.c: (_dbus_server_init_base): Use the new at_location mutex API (dbus_server_allocate_data_slot): Pass in the global lock address to _dbus_data_slot_allocator_alloc * test/name-test/test-threads-init.c: New test case for late thread initialization
* * dbus/dbus-dataslot.c (_dbus_data_slot_allocator_alloc):John (J5) Palmieri2006-08-141-1/+1
| | | | | | | | Change _dbus_abort to _dbus_assert_not_reached because _dbus_abort causes compile problems when asserts are turned off Keeping _dbus_warn for printing out the message so even if asserts are turned off the user gets the messages that something is wrong
* Patches by Kjartan Maraas <kmaraas at gnome dot org>John (J5) Palmieri2006-08-148-9/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * bus/services.c (bus_service_list_queued_owners): Add a pointer cast to fix compiler warning * dbus/dbus-dataslot.c (_dbus_data_slot_list_get): return a NULL instead of FALSE since the return type is not expecting a boolean * dbus/dbus-marshal-basic.c (_dbus_marshal_test): Remove unused variable * dbus/dbus-marshal-recursive-util.c (node_new): return a NULL instead of FALSE since the return type is not expecting a boolean * dbus/dbus-server-debug-pipe.c (_dbus_transport_debug_pipe_new): Send a NULL into _dbus_transport_new_for_fd instead of a FALSE because we are expecting a pointer not a boolean * dbus/dbus-sysdeps-util.c (_dbus_get_tmpdir): add void as the parameter so some compilers don't complain * dbus/dbus-transport-unix.c (_dbus_transport_new_for_domain_socket, _dbus_transport_new_for_tcp_socket): Send a NULL into _dbus_transport_new_for_fd instead of a FALSE because we are expecting a pointer not a boolean * test/shell-test.c (test_command_line): cast the second argument to _dbus_list_append to avoid compiler warnings * test/test-names.c (main): remove unused variable * test/test-service.c (check_hello_from_self_reply): Initialize echo_message and echo_reply to NULL * test/test-shell-service.c (handle_echo): Remove unused variable and cast the third parameter passed to dbus_connection_get_object_path_data to avoid compiler warrnings * test/name-test/test-names.c (clear_message_queue): Remove unused function * test/name-test/test-pending-call-dispatch.c: Fix format string in printf
* * dbus/dbus-bus.c:John (J5) Palmieri2006-08-141-12/+0
| | | | | | | * test/name-test/test-names.c: Remove test method dbus_bus_connection_get_unique_name because we already have public method dbus_bus_get_unique_name that does the exact same thing
* * fix typos in Julio's previous patch which cause make check to failJohn (J5) Palmieri2006-08-111-4/+4
|
* * fix mismatching paren typoJohn (J5) Palmieri2006-08-111-1/+1
|
* * dbus/dbus-address.c (_dbus_address_test): Revert leaking strcmp.John (J5) Palmieri2006-08-101-1/+1
| | | | | | In any case it was wrong since this is a test checking to see if address parsing is correct. There was no need to get the true tmp directory.
* * dbus/dbus-macros.h: Revert the addition of stddef.hJohn (J5) Palmieri2006-08-101-2/+0
| | | | as we should not be adding it to library headers
* * dbus/dbus-signature.c:John (J5) Palmieri2006-08-101-13/+27
| | | | | Fix the unit tests so that they work if assertions are not enabled. (patch from Julio M. Merino Vidal <jmmv at NetBSD dot org>)
* * forgot to save the dbus/dbus-macros.h file for a previous checkinJohn (J5) Palmieri2006-08-101-0/+2
|
* * dbus/dbus-sysdeps.c:John (J5) Palmieri2006-08-103-1/+31
| | | | | | | | * dbus/dbus-address.c: * bus/activation.c: * test/shell-test.c: don't hardcode tmp directory (patch from Dave Meikle <dmk at davemeikle dot co dot uk>)
* * Change an assert to a detailed warning and abort.John (J5) Palmieri2006-08-091-2/+5
| | | | | | When allocator->lock != mutex it indicates that the user has failed to initalize threads before using the D-Bus library. This warning helps the user identify the issue and fix their app.
* These are all patches from Kjartan Maraas <kmaraas at gnome dot org>John (J5) Palmieri2006-08-081-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | with cleanups of bugs found from Coverity reports: * dbus/dbus-sysdeps-util.c (_dbus_write_pid_file): close the file on error to avoid a leak * bus/expirelist.c (bus_expire_list_test): Check for NULL on dbus_new0 * bus/activation.c (update_directory): remove dead code * bus/config-parser.c (merge_service_context_hash, start_selinux_child): Fix some leaks * bus/bus.c (process_config_every_time): Fixed a leak * bus/desktop-file.c (parse_key_value): Fixed leak * bus/selinux.c (bus_selinux_id_table_insert): Fixed leak
* * dbus/dbus-object-tree.c (_dbus_object_subtree_new):John (J5) Palmieri2006-08-081-5/+0
| | | | remove dead code
* * dbus/dbus-sysdeps.h:John (J5) Palmieri2006-08-083-5/+7
| | | | | | | | | | | | | | | | * dbus/dbus-sysdeps.c: * dbus/dbus-string.c: s/_dbus_printf_length/_dbus_printf_string_upper_bound to comform with GLib's function which does the same thing * configure.in: * bus/Makefile.am: * bus/dir-watch-default.c: * bus/dir-watch-dnotify.c: * bus/dir-watch-kqueue.c: Add kqueue directory watching for freebsd and split the directory watching code into seperate files per method/arch (patches from Timothy Redaelli <drizzt at gufi dotorg>)
* * dbus/dbus-sysdeps.h:John (J5) Palmieri2006-08-083-2/+20
| | | | | | * dbus/dbus-sysdeps.c: add _dbus_printf_length (patch from Peter Kümmel <syntheticpp at gmx dot net>)
* * dbus/dbus-internals.c: unistd.h is not usedJohn (J5) Palmieri2006-08-081-2/+2
| | | | | | (_dbus_verbose_real): only enable verbose printing if DBUS_VERBOSE environment var is set to '1' (patch from Peter Kümmel <syntheticpp at gmx dot net>
* * configure.in: add -Wdeclaration-after-statementJohn (J5) Palmieri2006-08-044-191/+387
| | | | | | | | | | | * dbus/dbus-connection.c: change all the pending call stuff to reflect the fact that pending call operations use the connection lock * dbus/dbus-pending-call.c: add locking here * dbus/dbus-errors.c (struct DBusRealError): don't make the name field const consistent with how message field is done
* * s/D-BUS/D-Bus/gJohn (J5) Palmieri2006-08-0336-77/+77
|
* * dbus/dbus-object-tree.c: Patch by Benjamin OtteJohn (J5) Palmieri2006-08-031-1/+1
| | | | | <in7y118 at public.uni-hamburg dot de> - fix invalid read/write reported by valgrind
* * dbus/dbus-connection.c:John (J5) Palmieri2006-07-221-1/+3
| | | | | | | (_dbus_connection_attach_pending_call_unlocked): (connection_timeout_and_complete_all_pending_calls_unlocked): Make sure we set timeout_added on pending calls to FALSE when we remove the timeout from the connection
* * Removed some extra bindings stuff lingering around (thanks timo)John (J5) Palmieri2006-07-211-1/+1
| | | | | | * dbus-pendingcall.c (_dbus_pending_call_new): s/dbus_connection_ref/_dbus_connection_ref_unlocked fixes assertion when we tried to take a lock on an already locked connection