obexd: Update the file offset to the beginning after writing to the file

When the transfer file is opened in O_RDWR mode, just after the contents are
written to the file, the file offset has to be set to the beginning of the
file. If not subsequent read fails. This patch fixes this.

obexd: Make obc_transfer_put to accept NULL as filename

In that case contents should be valid so the modules can use
obc_transfer_put to create temporary files.

obexd: Avoid extra copies while passing apparam to transfer

By passing directly the reference to GObexApparam it is no longer
necessary to use intermediate buffers to pass data around.

obexd: Fix not sending parameters to get message in map module

Attachment and charset are mandatory, so Message.Get now takes an
additional boolean parameter which the user application should set
if it wants the attachments to be downloaded, charset is always set
to UTF8.

obexd: Add MessageAccess.ListFilterFields

obexd: Rename MessageAccess method GetMessagesListing to ListMessages

In addition to that add missing parsing of the filters

obexd: Rename MessageAccess method GetFolderListing to ListFolders

In addition to that add missing parsing of the filters.

obexd: Use larger timeout first packet in transfers

This align the client timeout for GET operations with g_obex_get_req_pkt
which does use 60 seconds to cover for extra interaction during the first
packet.

obexd: Unify callbacks for operations that only report status

This removes duplicated code.

obexd: Add set delete status function

This function will inform MAP server to modify
the delete status of a given message.

obexd: Add set read status function

This function will inform MAP server to modify the
read status of a given message.

obexd: Add support for MessageStatus

Add driver to support x-bt/messageStatus mimetype

obexd: Move common code to pull_phonebook

obexd: Remove deprecated methods from PhonebookAccess

SetFormat, SetOrder, SetFilter and GetFilter methods are no longer
necessary as other methods now take them as parameters to avoid round
trips.

obexd: Add filters to PhonebookAccess.List and PhonebookAccess.Search

This avoid D-Bus round trips and is more aligned with what has been
proposed for MessageAccess interface.

obexd: Add filters to PhonebookAccess.Pull

This avoid D-Bus round trips and is more aligned with what has been
proposed for MessageAccess interface.

obexd: Add filters to Phonebook.PullAll

This avoid D-Bus round trips and is more aligned with what has been
proposed for MessageAccess interface.

obexd: Fix crash on map module

gboolean is expected to hold 0/1. But it is holding int return from
strcasecmp which crashes DBusMessage at:

_dbus_return_val_if_fail (*bool_p == 0 || *bool_p == 1, FALSE);

Trace:
0  0x00007ffff7328d95 in __GI_raise (sig=6) at raise.c:64
1  0x00007ffff732a2ab in __GI_abort () at abort.c:93
2  0x00007ffff78d0655 in _dbus_abort () at dbus-sysdeps.c:94
3  0x00007ffff78c75f1 in _dbus_warn_check_failed at dbus-internals.c:289
4  0x00007ffff78ba28b in dbus_message_iter_append_basic at dbus-message.c:2538
5  0x00000000004201c3 in append_variant at client/dbus.c:44
6  0x000000000042024e in obex_dbus_dict_append at client/dbus.c:65
7  0x000000000041dcc9 in parse_read at client/map.c:423
8  0x000000000041dfa7 in msg_element at client/map.c:518
9  0x00007ffff7b323b9 in emit_start_element at gmarkup.c:986
10 0x00007ffff7b33b44 in g_markup_parse_context_parse at gmarkup.c:1323
11 0x000000000041e1ad in message_listing_cb at client/map.c:586
12 0x000000000041744c in session_terminate_transfer client/session.c:743
13 0x00000000004174d7 in session_notify_complete at client/session.c:758
14 0x000000000041755a in transfer_complete at client/session.c:778
15 0x000000000041f57b in xfer_complete at client/transfer.c:521
16 0x000000000040efdf in transfer_complete at gobex/gobex-transfer.c:102
17 0x000000000040f418 in transfer_response at gobex/gobex-transfer.c:221
18 0x000000000040b320 in handle_response at gobex/gobex.c:948
19 0x000000000040bbc1 in incoming_data at gobex/gobex.c:1191
20 0x00007ffff7b2f94a in g_main_dispatch (context=0x62f130) at gmain.c:2515
21 g_main_context_dispatch (context=0x62f130) at gmain.c:3052
22 0x00007ffff7b2fd10 in g_main_context_iterate at gmain.c:3123
23 g_main_context_iterate at gmain.c:3060
24 0x00007ffff7b3010a in g_main_loop_run (loop=0x62e1b0) at gmain.c:3317
25 0x000000000041527d in main at client/main.c:175

obexd: Remove redundant include errno.h

obexd: Remove redundant include sys/stat.h

obexd: Add subject length parameter in GetMessageListing

Add subject length parameter to allow MAP server to
create subject as per MAP client length preference.

obexd: Add offset parameter decription

Add offset parameter description for
messages_get_messages_listing() function header

obexd: Fix invalid oflag check

Change logic to check for O_WRONLY, as oflag is
created by an OR operation of O_WRONLY, O_CREAT and
O_TRUNC.
Message update is a PUT function, this check returns
bad response if GET is received.

obexd: Remove map_ap.c

It is no longer needed as MAP plugin is now using GObexApparam API

obexd: Make use of GObexApparam API

obexd: Make use of GObexApparam API

obexd: Port PBAP module to use GObexApparam

obexd: Fix pbap_select using absolute path with known locations

pbap_select has to use absolute path with known location to support
repeatable pbap_select calls. In other way the second call fails.

obexd: Fix possible memory leak in handling of location

path was not being freed before returning, the is also simplified so
the error is now given in the return of the function.

obexd: Fix possible memory leak in bluetooth driver

2,210 (64 direct, 2,146 indirect) bytes in 2 blocks are definitely lost in loss record 150 of 155
   at 0x4A0884D: malloc (vg_replace_malloc.c:263)
   by 0x3B2900F8A1: sdp_create (sdp.c:3656)
   by 0x3B2900FAE0: sdp_connect (sdp.c:4604)
   by 0x415902: adapter_reply (bluetooth.c:421)
   by 0x3B22C0C429: ??? (in /usr/lib64/libdbus-1.so.3.5.6)
   by 0x3B22C0F5E9: dbus_connection_dispatch (in /usr/lib64/libdbus-1.so.3.5.6)
   by 0x4059A7: message_dispatch (mainloop.c:76)
   by 0x32CD64822A: ??? (in /usr/lib64/libglib-2.0.so.0.3200.4)
   by 0x32CD647694: g_main_context_dispatch (in /usr/lib64/libglib-2.0.so.0.3200.4)
   by 0x32CD6479C7: ??? (in /usr/lib64/libglib-2.0.so.0.3200.4)
   by 0x32CD647DC1: g_main_loop_run (in /usr/lib64/libglib-2.0.so.0.3200.4)
   by 0x40567F: main (main.c:175)

obexd: Make handle of location more clear

Add proper defines for supported paths and convert relative
path absolute to align with PBAP paths.

obexd: phonebook: Replace magic strings for phonebook names and folders

obexd: Fix phonebook contacts query

At the moment IRMC fails to connect for phonebook_ebook and
phonebook_tracker usage because the name parameter for phonebook_pull
is not an absolute path.

obexd: Add Message.Get implementation

obexd: Use filter instead of dummy as argument name in MAP

This is aligned with the documentation that uses filter as well.

obexd: Change MessageAccess.GetMessageListing to not return raw xml

This parses the response and return as a list of dictionary where each
entry is a message and its properties,

obexd: Change MessageAccess.GetFolderListing to not return raw xml

This parses the response and return as a list of dictionary where each
entry is a folder and its properties, similar to what
FileTransfer.ListFolder does.

obexd: Fix wrong signature of MessageAccess.SetFolder

D-Bus string signature is 's' not 'string'

obexd: Fix some common spelling errors

Fix spelling errors reported by codespell.

obexd: Fix dead assignments

Values stored are never read.

obexd: Fix error returning in obc_transfer_put

Always set error on failure in obc_transfer_put. This is expected by
callers and will avoid possible NULL pointer dereference.

Also fix improper use of errno variable (calling error may modify it)
and some dead assignments to perr.

obexd: Use g_malloc0 in get_filter_strs

Use g_malloc0 instead of g_try_malloc0 to allocate list. All users
expects to get valid pointer from it anyway. Also size of allocated
memory is small.

obexd: Fix use of uninitialized count variable

If opendir() call fails count would be passed to callback
uninitialized.

obexd: Fix memory leak in bluetooth driver

384 (184 direct, 200 indirect) bytes in 1 blocks are definitely lost in loss record 141 of 157
   at 0x4A0884D: malloc (vg_replace_malloc.c:263)
   by 0x392E217815: ??? (in /usr/lib64/libdbus-1.so.3.5.6)
   by 0x392E218632: dbus_message_new_method_call (in /usr/lib64/libdbus-1.so.3.5.6)
   by 0x413AD3: send_method_call (bluetooth.c:87)
   by 0x414933: manager_reply (bluetooth.c:496)
   by 0x392E20C429: ??? (in /usr/lib64/libdbus-1.so.3.5.6)
   by 0x392E20F5E9: dbus_connection_dispatch (in /usr/lib64/libdbus-1.so.3.5.6)
   by 0x405767: message_dispatch (mainloop.c:76)
   by 0x369E04827A: ??? (in /usr/lib64/libglib-2.0.so.0.3200.3)
   by 0x369E0476E4: g_main_context_dispatch (in /usr/lib64/libglib-2.0.so.0.3200.3)
   by 0x369E047A17: ??? (in /usr/lib64/libglib-2.0.so.0.3200.3)
   by 0x369E047E11: g_main_loop_run (in /usr/lib64/libglib-2.0.so.0.3200.3)

obexd: Fix cancellation of queued transfers

Cancelling queued (not started yet) transfers should not fail. Instead,
they must be removed from the queue, so we just need to call the
transfer callback.

obexd: Remove obc_transfer_abort

Refactor the code to remove function obc_transfer_abort, which is used
only once and is anyway coupled to the D-Bus API.

obexd: Fix possible unreplied D-Bus message

If a previous cancel request is in progress, a second cancel request
should fail.

This by the way fixes unreplied D-Bus messages.

obexd: Fix using org.openobex.Error

In addition use define so it is easier to change this in future

obexd: Use session path as prefix for transfer path

This should make it easier to identify to which session the transfer
belongs.

obexd: Fix not using connection passed to obc_transfer_register

Make use of the connection parameter instead of assuming the connection
will always be to the session bus.

obexd: Rename Transfer interface to org.bluez.obex.Transfer

obexd: Rename MessageAccess interface to org.bluez.obex.MessageAccess

obexd: Rename Synchronization interface to org.bluez.obex.Synchronization

obexd: Rename PhonebookAccess interface to org.bluez.obex.PhonebookAccess

obexd: Rename FileTransfer interface to org.bluez.obex.FileTransfer

obexd: Rename ObjectPush interface to org.bluez.obex.ObjectPush

obexd: Rename session interface to org.bluez.obex.Session

The base is also changed to /org/bluez/obex to follow the new namespace

obexd: Change namespace to org.bluez.obex.client

obexd: Change agent interface to org.bluez.obex.Agent

obexd: Change D-Bus namespace to org.bluez.obex

This changes the namespace from org.openobex to org.bluez.obex

obexd: Update copyright statement

obexd: PhonebookAccess sessions return transfers

Return the D-Bus path of the transfer representing the operation.

obexd: Synchronization sessions return transfers

Return the D-Bus path of the transfer representing the operation.

obexd: FileTransfer sessions return transfers

Return the D-Bus path of the transfer representing the operation.

obexd: ObjectPush sessions return transfers

Return the D-Bus path of the transfer representing the operation.

obexd: Expose D-Bus data in internal transfer API

Expose in transfer API the D-Bus path and properties as should be
returned by transfer-initiating D-Bus methods.

obexd: Support empty filename in obc_transfer_get

Passing an empty string as a filename for obc_transfer_get will be
similar to passing a NULL filename. This means a temporary file will be
created to store the content of the transfer.

NULL and "" are not exactly equivalent though: in case of NULL the file
will be automatically removed immediately after being open, which means
that the transfer initiator should also open the file to prevent it from
being removed (to be used from the modules). In this case, the filename
will not be exposed in D-Bus.

On the other hand, if "" is given, the file will be removed only in case
of error. So after success the transfer initiator should decide whether
the file should be removed or not.

This change is convenient in order to expose the same API in D-Bus.

obexd: Make transfer filename optional

The property might not exist for certain transfers, typically when they
have been initiated internally.

obexd: Make FileTransfer.PutFile asynchronous

There is no reason to have inconsistent behavior between GetFile and
PutFile, in FileTransfer D-Bus API.

Before this change, PutFile reported success immediately after queueing
the transfer, even though the D-Bus signature includes the async flag.

obexd: Expose all transfers in D-Bus

Relying on a internal policy (based on transfer type) to decide if a
transfer should be exposed or not in D-Bus has some limitations. The
simplest possible alternative to this is to expose all transfers in
D-Bus, assuming the overhead is not significant.

obexd: Remove obsolete authentication code

After the removal of the agent, the implementation of the session can be
simplified by removing all authentication-related code.

obexd: Remove internal transfer progress report

The new D-Bus API uses signals to report the progress updates, so the
internal progress callback is not needed any more.

obexd: Remove unused functions in transfer API

After the removal of the agent these functions are not useful any more.

obexd: Remove D-Bus agent

The authorization mechanism is entirely removed from the session, and
thus transfers are automatically started (once popped from the queue)
without confirmation and without any name/filename change.

obexd: Use transfer owner instead of agent

The security checks in the transfers' D-Bus API will consider check for
the transfer owner's path (session owner) instead of the agent path.

obexd: Add transfer event-reporting signals

These signals replace the old agent-based notification mechanism.

obexd: Add progress property to transfer

The number of transferred bytes is exposed in D-Bus using a specific
property for this purpose.

Internally, the value of this property does not necessarily match the
internal progress counter. In order to avoid D-Bus overhead, the
property will be updated once per second.

obexd: Add D-Bus helper library

No functionality changes. This is just about avoiding duplicated code.

obexd: Replace parameter dict with conventional ones

obexd: Move GetCapabilities to session API

obexd: Replace SendFiles with SendFile

The function is now asynchronous, since it will return only when the
transfer has been finished.

obexd: Wrap OPP into specific session type

obexd: Use constant instead of NULL variable

The filename will always be NULL for capability-requesting sessions, so
it doesn't make much sense to use such field.

obexd: Remove left over glib-helper.h support

obexd: Simplify error-handling code

Refactor error-handling code to avoid duplicated code.

obexd: Fix NULL dereference in case of error

obc_session_queue assumes that the given transfer is not NULL, so this
must be checked explicitly.

obexd: Do not set signature and reply in GDBus tables

Use GDBUS_* macros, so signature and reply fields are not set in each
method/signal.

obexd: Convert GDBus methods to use macro helpers

With these macro helpers we can separate in/out arguments and use their
own vector.

obexd: Constify GDBus signal tables

Constify signal tables with the following command:

find . -name '*.[ch]' -exec \
         sed -i 's/\(GDBusSignalTable .* =\)/const \1/g' {} \;

obexd: Constify GDBus method tables

Constify method tables with the following command:

find . -name '*.[ch]' -exec \
         sed -i 's/\(GDBusMethodTable .* =\)/const \1/g' {} \;

obexd: Fix not checking if GoepL2capPsm contains a valid PSM

In case the PSM is not valid ignore it so we are still able to connect
to RFCOMM.

obexd: Fix not setting port to 0 when listen fails

With some only kernels auto assigning psm seem to fail, leaving the psm
with wrong value:

obexd[2930]: plugins/bluetooth.c:start() listening on channel 9
obexd[2930]: bluetooth: unable to listen in psm 65535
obexd[2930]: plugins/bluetooth.c:start() listening on channel 10
obexd[2930]: bluetooth: unable to listen in psm 65535

obexd: Use stack memory for application parameters in pbap module

The maximum amount needed is known and sufficiently small to be in the
stack which is much simpler to deal with.

obexd: Add obc_transfer_set_params to set application parameters

The parameters are optional and only used in a few occasions so it
doesn't make sense to have that directly in obc_transfer_get and
obc_transfer_put.

obexd: Create transfers in modules

After this patch the modules are responsible for creating the transfers,
and these objects must be queued using the session API.

This way the transfer initiator has full access to the transfer object,
in case for example it wants to access some member variable.

obexd: Flip parameter order in transfer API

This minor change makes the transfer API more consistent with the
parameter-order used in the session API.

obexd: Buffer-passing changes in transfer API

Transfer API now takes const buffers (both params and contents) and
internally copies the memory as necessary. This new API is safer to use,
which is convenient if the modules would start using it directly.

obexd: Transfer API splits create and register

The transfer-creating functions (obc_transfer_get and obc_transfer_put)
no longer register the transfer automatically.

This separation makes it possible that the modules would create the
transfers and then pass the object to the session, which would be
responsible for the registration.

obexd: Split internal obc_transfer_register()

The creation process has been internally split into two steps: creation
and D-Bus registration. This is easier to understand and it also allows
to expose these two-steps in the transfer API.

obexd: Fix possible double free of params

obc_transfer_get() and obc_transfer_put() should only assume ownership
of the given params only in case of success. Otherwise some erros might
result in a double free of such memory.