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.

obexd: Add handler for SIGUSR2

This signal enables debug for obed so just do the same for obex-client

obexd: Use signalfd to handle unix signals

obexd: Use signalfd to handle unix signals

Writing anything to syslog in the signal handler can cause a deadlock
with any ongoing syslog write.

This is also aligned with what BlueZ and other projects has been doing.

obexd: Fix codying style

Fix lines that are over 80 columns

obexd: Wait for abort completion before Transfer.Cancel returns

A new callback is given to g_obex_cancel_transfer() which will be
called when abort completes and then only "Cancel" method reply will
be sent to user

obexd: Remove transfer from queue before callback

It is safer to remove the transfer from the internal queue (including
session->p) before calling the transfer callback. This makes sure the
callback will not manipulate the session in a way that the transfer is
removed more than once.

This was previously protected with session->p->id != 0 checks, but once
the new callbacks have been adopted in session API, this logic can be
removed.

obexd: Remove deprecated part of session API

Once the modules are using the new callback style, the session API can
be simplified and the old functions to access session->p removed.

obexd: Use new session callback style in modules

The session API now provides the transfer object in the callback, so
the modules can directly access the transfer object.

obexd: Give transfer pointer in session callbacks

Operations involving a transfer object will receive a pointer to such
transfer in the callback.

Note that the ownership of this object is not changed in any way,
meaning that the session is still responsible for it. However this
pointer can be useful during the execution of the callback, in order to
access data members of the transfer.

obexd: Avoid GObex dependency from transfer.h

This workaround makes it possible to include transfer.h from the
modules, without adding a dependency to GObex.

obexd: Minor buffer access API changes

Trivial changes in buffer getters in both session and transfer,
regarding the access of transfer parameters:
- const qualifiers added, to avoid unwanted frees
- Buffers are now returned as void* instead of guint8*

obexd: Emit TransferStarted signal only if stream start is success

Even if obex_put_stream_start() fails TransferStarted signal will be
emitted where as transfer start is actually failed.

obexd: Return request id when generating a request in session API

This is more consistent with other functions and allow the caller to
cancel the request using obc_session_cancel.

obexd: Fix not propagating GError in session API functions

The errors should be properly forward to the caller and not just convert
to generic error.

obexd: Remove file in case of error

If the transfer operation is GET and it has not complete by the time its
freed remove the file as its contents maybe corrupted/incomplete.

obexd: open file during transfer creation

This simplify the API a bit by not having to call obc_transfer_set_file
to open the file.

In addition to that split transfer creation/registration function so
GET/PUT can have more specific logic and different paramenters.

obexd: transfers take gobex when starting

gobex api should not be used by a transfer until it is started. This
seems more explicit if the pointer is not passed during creation.

obexd: transfer api merges put and get

A new enum type is used to distinguish put and get transfers.

This is more convenient since it is done when registering the transfer,
and not when it is actually started. The main benefits would be:
    - Some actions can be taken during creation, such as opening files.
    - session.c gets simplified.
    - The size of a put transfer can be exposed in D-Bus, while queued.
    - The transfer operation (put or get) can be exposed in D-Bus.

None of these D-Bus changes are included in this patch.

obexd: remove unused field

The errorcode field is set but never used, so it can safely be removed.
In addition there is no need for such a field, because errors can be
propagated using the available callback.

obexd: Remove buffer based transfer

Simplify the code by using temporary files and eliminates reallocations.

obexd: Fix possible memory leak on pbap module

16 bytes in 1 blocks are definitely lost in loss record 26 of 146
   at 0x4A075B2: realloc (vg_replace_malloc.c:525)
   by 0x3B5104B76D: g_realloc (in /lib64/libglib-2.0.so.0.3000.2)
   by 0x3B51064A96: ??? (in /lib64/libglib-2.0.so.0.3000.2)
   by 0x3B51065156: g_string_insert_len (in /lib64/libglib-2.0.so.0.3000.2)
   by 0x3B510305BC: ??? (in /lib64/libglib-2.0.so.0.3000.2)
   by 0x3B51031BE7: g_build_filename (in /lib64/libglib-2.0.so.0.3000.2)
   by 0x416FEE: pbap_select (pbap.c:254)
   by 0x406CCE: process_message (object.c:224)
   by 0x3B5301D9A0: ??? (in /lib64/libdbus-1.so.3.5.6)
   by 0x3B5300F92F: dbus_connection_dispatch (in /lib64/libdbus-1.so.3.5.6)

obexd: Fix regression when calling obc_session_shutdown within callback

Commit c07ddfbd019d3545cce2d7ec694143cdc55a2167 introduced the freeing of
the active pending request on obc_session_shutdown without checking if
the request was already processed/terminated.

obexd: Remove glib-helper.h support

obexd: Remove unused struct session_callback

obexd: Remove headers declared but not defined

obexd: Use goto for avoiding code duplication

obexd: Fix memory leak during session connect

obexd: Output parameters for GetMessagesListing

This sets the parameters returned from MAP backend for
GetMessagesListing to be used in application parameters header sent in
response.

obexd: Input parameters for GetMessagesListing

This adds support for input application parameters header given in
GetMessagesListing request.

obexd: Mark filter strings const

This structure is used for giving filters as input argument to
messages_get_messages_listing(). There is no need for the string members
to be modifiable in this context. And making them const plays well with
map_ap_get_string(), from which the values of these members are going to
be retrieved.

The string pointers here are simply references to data held in map_ap_t
*inparams.