introspect: Fix ABI break introduced by b98a2e1

The size of pa_card_profile_info cannot change even if it just a field
appended to end because each entry is appended to a contiguous memory
and accessed by offset this may lead clients to access invalid data.

To fix a new struct called pa_card_profile_info2 is introduced and shall
be used for now on while pa_card_profile_info shall be considered
deprecated but it is still mantained for backward compatibility.

A new field called profiles2 is introduced to pa_card_info, this new field
is an array of pointers to pa_card_profile_info2 so it should be possible
to append new fields to the end of the pa_card_profile_info2 without
breaking binary compatibility as the entries are not accessed by offset.

alsa-mixer: Add internal surround speaker elements

These kcontrol names have started to show up lately, in
combination with surround internal speakers.

BugLink: https://bugs.launchpad.net/bugs/1236965
Signed-off-by: David Henningsson <david.henningsson@canonical.com>

console-kit: Remove logging of every message on the bus

tests: Extract PA_CPU_TEST* macros to separate header

Signed-off-by: Peter Meerwald <pmeerw@pmeerw.net>

daemon conf: "high-priority" setting isn't respected

The code got removed by accident during the cleanup in commit 9c438bcac671. So
this patch is needed to bring it back and make things work like documented.

zsh-completion: Add --monitor-stream completion to pacat

simple: Improve pa_simple_read() documentation

There was a question in IRC about whether pa_simple_read() blocks or
not. It's already documented on the simple API overview page, but it's
good to say it also in the function reference. As a bonus, I added
some additional details to the documentation too.

bluetooth: Fix variable constness

The string points to memory inside a DBusMessage, so we don't own the string.

bluetooth: Remove adapter_remove_all()

The function was redundant, because all it did was call adapter_free()
for each adapter in the hashmap, and that can be delegated to
pa_hashmap when freeing or emptying it.

zsh-completion: Fix pacat device completion

The pacat completion didn't complete the right devices for the --record
and --playback flags.

This patch fixes this and makes the device completion for pacat easily
expandable.

bash-completion: Add --monitor-stream completion to pacat

modargs: Add pa_modargs_get_sample_rate()

return sample rate from the "rate" module argument

Signed-off-by: Peter Meerwald <pmeerw@pmeerw.net>

modargs: Add pa_modargs_get_resample_method()

Signed-off-by: Peter Meerwald <p.meerwald@bct-electronic.com>

mainloop, glib-mainloop: time_restart could cause incorrect event ordering

This fixes a bug where calling time_restart can leave the current event
in the cache, even though the restart scheduled the event in the future.
This would cause the event to get executed more frequently than it should.

tests: Resampler output memchunk may be reset (zero)

Signed-off-by: Peter Meerwald <p.meerwald@bct-electronic.com>

memblock: Fix typo in description of pa_memblock_unref_fixed(), cleanup

Signed-off-by: Peter Meerwald <p.meerwald@bct-electronic.com>

Updated Brazilian Portuguese translation

Signed-off-by: Rafael Ferreira <rafael.f.f1@gmail.com>

bluetooth: Track discovery modules by index

Previously module-bluez5-discover and module-bluez4-discover were being
tracked using their pa_module pointer. But during daemon shutdown these
modules are unloaded before module-bluetooth-discover, leaving stale
pointers in module-bluetooth-discover's userdata. To avoid this problem
this commit makes module-bluetooth-discover keep track of
module-bluez5-discover and module-bluez4-discovery by their indexes.

build-sys: Print CPPFLAGS in configure

I needed this for verifying that the conditional defining of
_FORTIFY_SOURCE works.

build-sys: Don't define _FORTIFY_SOURCE when building with -O0

sndfile-util: fix format for 24bit depth wav files

PA_SAMPLE_24NE generated in pa_sndfile_read_sample_spec is not
handled in pa_sndfile_readf and writef function. paplay/parecord
used to get aborted for 24bit depth wav files

alsa: Fix crash when loading bare ALSA sink/source

module-alsa-{sink,source}.c call pa_alsa_{sink,source}_new with
mapping set to NULL. Guard against this, like the rest of the
function does.

module-alsa-card does not use NULL, so this went unnoticed so far.

build-sys: Fix building with Android toolchain

Signed-off-by: Anton Lundin <glance@acc.umu.se>

tunnel-sink-new: Fix a possible crash

When the creation of u->thread fails, then pa_thread_mq_done() in
pa__done() will crash, because pa_thread_mq_init() was never called.
Allocating the thread_mq object separately, instead of embedding it
in the userdata struct, allows pa__done() to call pa_thread_mq_done()
only when necessary.

tunnel-sink-new: remove switch-default from state change callbacks

Using default sections for switch(state) in state change callbacks will prevent
useful compiler warnings for non-handled cases

alsa-mixer: Drop all unused paths, not only unsupported paths

This is a cleaner solution, because it also removes paths that are
being removed because they are subsets of other paths.

Otherwise, the lingering paths could cause jack detection related
assertion failures.

BugLink: https://bugs.freedesktop.org/show_bug.cgi?id=69676
Reported-and-tested-by: Kalev Lember <kalevlember@gmail.com>
Signed-off-by: David Henningsson <david.henningsson@canonical.com>

i18n: Update bluetooth bits in POTFILES

default/system.pa: Do not load module-dbus-protocol

The author of this module, Tanu Kaskinen, has said that this module
"is not suitable for general use". Also, it is still causing crashes
on card removal (see bug 69871).

Qpaeq, and possibly other tools, use this module - but they can load
the module manually if they still wish to use it.

Signed-off-by: David Henningsson <david.henningsson@canonical.com>

alsa-ucm: Fix bad rate check

Need to check the rate value for being > 0 not the return value of
pa_atou().

alsa-ucm: Don't allow Playback/CaptureRate to be 0

This was a review comment on the original patch that I missed fixing.

bluetooth: Add a comment about messing with the IDLE suspend cause

hashmap: Add a key+value iterator

bluetooth: Revive module-bluetooth-discover

Create a wrapper module called module-bluetooth-discover to avoid
breaking backward-compatibility of default.pa. This wrapper may
eventually be dropped altoghether with BlueZ 4 support.

cli-command: Use pa_module_exists() in .ifexists

module: Create pa_module_exists()

This new function checks if a certain module name is available in the
system.

bluetooth: Fail to load driver if discovery module is not loaded

For quite some time now the device driver module doesn't work well
without the discovery module, so for the BlueZ 5 support we'll prevent
the device driver module to be loaded if the discovery module is not
loaded.

bluetooth: Implement get_n_used() for module-bluez5-device

bluetooth: Handle changes to BlueZ 5 transports state

bluetooth: Process source messages for BlueZ 5 cards

bluetooth: Process sink messages for BlueZ 5 cards

bluetooth: Create I/O thread function for BlueZ 5 cards

Create the thread function, the render and push functions for A2DP, the
process message function for communication between the I/O thread and
the main thread, and other helper functions related to them.

bluetooth: Set card profile for BlueZ 5 cards

bluetooth: Start / stop I/O thread for BlueZ 5 cards

bluetooth: Create source for BlueZ 5 cards

bluetooth: Create sink for BlueZ 5 cards

bluetooth: Initialize profiles for BlueZ 5 cards

Initialized the currently active profile, configure and acquire the
transport.

bluetooth: Create BlueZ 5 card profile for each audio UUID

bluetooth: Create BlueZ 5 card ports

bluetooth: Create BlueZ 5 card

bluetooth: Unload module-bluez5-device on device's disconnection

bluetooth: Get BlueZ 5 device object

Get the remote device information stored in pa_bluetooth_discovery. This
also creates the mandatory parameter 'path' for module-bluez5-device,
which is used to inform the object path of the remote device in BlueZ on
the module load.

bluetooth: Create module-bluez5-device

bluetooth: Track devices in module-bluez5-discover

bluetooth: Handle PropertiesChanged for org.bluez.MediaTransport1

bluetooth: Handle PropertiesChanged for org.bluez.Device1

bluetooth: Protect from a misbehaving bluetoothd

bluetoothd always send the GetManagedObjects() reply messages with the
objects array argument following an in-depth order starting from the
root. That means parents will always be known at the time their children
objects are parsed, if clients parse the objects in the same order they
appear in the array, as we do in PulseAudio.

This commit tries to protect PulseAudio in the case bluetoothd changes
that behavior for some reason. It hasn't been tested, since this
situation never occurs.

bluetooth: Parse BlueZ 5 device properties

This code is based on previous work by Mikel Astiz.

bluetooth: Handle PropertiesChanged for org.bluez.Adapter1

bluetooth: Register endpoints with BlueZ 5 adapter

bluetooth: Parse BlueZ 5 adapter properties

bluetooth: Parse BlueZ 5 D-Bus interfaces

Parse the arguments of the InterfacesAdded signal and the
GetManagedObjects() reply.

This code is based on previous work by Mikel Astiz.

bluetooth: Handle InterfacesAdded and InterfacesRemoved

This code is based on previous work by Mikel Astiz.

bluetooth: Create a function to remove only one adapter object

bluetooth: Get managed objects

Get objects from the BlueZ 5 object manager. This code is based on
previous work by Mikel Astiz.

bluetooth: Add utility function to send D-Bus messages with reply

bluetooth: Implement org.bluez.MediaEndpoint1.ClearConfiguration()

bluetooth: Implement org.bluez.MediaEndpoint1.SelectConfiguration()

bluetooth: Implement org.bluez.MediaEndpoint1.SetConfiguration()

bluetooth: Register MediaEndpoint1 objects with D-Bus

bluetooth: Create pa_bluetooth_transport for BlueZ 5 support

Create the pa_bluetooth_transport structure to store information about
the bluetooth transport and utility functions to manipulate this
structure. The acquire() and release() operations are function pointers
in the pa_bluetooth_transport structure to make possible for different
transport backends to provide different implementations of these
operations. Thre is also a userdata field for the transport backend
provide data for the acquire/release functions.

This commit also creates a new function
pa_bluetooth_device_any_transport_connected() to check if there is any
audio connection between the host and a remote device.

bluetooth: Remove all devices and adapters when org.bluez goes away

bluetooth: Create pa_bluetooth_device for BlueZ 5 support

Create a structure pa_bluetooth_device to store information about the
bluetooth device and utility functions to manipulate this structure.

bluetooth: Create pa_bluetooth_adapter for BlueZ 5 support

bluetooth: Create infrastrucure for bluetooth hooks

Create a pa_bluetooth_hook enum type and API to get pa_hooks from the
opaque pa_bluetooth_discovery struct.

bluetooth: Track org.bluez for BlueZ 5

bluetooth: Create pa_bluetooth_discovery for BlueZ 5

pa_bluetooth_discovery is the struct that holds information about known
Bluetooth audio devices and other information about the Bluetooth stack.

This commit also creates bluez5-util.[ch], which will hold a lot of
utility functions to help with the BlueZ 5 support.

bluetooth: Create stub for module-bluez5-discover

build: Add support for enabling/disabling BlueZ 5 support

build: Make the build of bluetooth modules BlueZ 4 specific

bluetooth: Remove module-bluetooth-proximity

module-bluetooth-proximity has not worked for quite a while, since it
uses pre-BlueZ4 APIs. Nobody complained since then, which is a good
indication that it doesn't have much users. Even the original commit
message refers to it more as a toy than as something of great use: "add
new fun module that automatically mutes your audio devices when you
leave with your bluetooth phone, and unmutes when you come back"

Removing it we completely remove the dependency on libbluetooth.

bluetooth: Prefix BlueZ 4 card profiles enum with PA_BLUEZ4_

Also typedef the enum as pa_bluez4_profile_t.

bluetooth: Register BlueZ 4 endpoints in a separate object path

We need diferent object paths for BlueZ 4 and BlueZ 5 endpoints to avoid
crashing the daemon in the case both modules are loaded in parallel.

bluetooth: Name BlueZ 4 symbols accordingly

We need diferent symbol prefixing for the current BlueZ 4 support and
the new BlueZ 5 support which is about to enter the codebase, to avoid
symbol clashing and crashing the daemon in the case both modules are
loaded at the same time.

This commit replaces all pa_bluetooth_ and pa_bt_ (for consistency)
prefixes with pa_bluez4_, for both lower-case and upper-case, what was
done with the following sed commands:

 $ sed -i s/pa_bluetooth_/pa_bluez4_/g src/modules/bluetooth/*bluez4*
 $ sed -i s/PA_BLUETOOTH_/PA_BLUEZ4_/g src/modules/bluetooth/*bluez4*
 $ sed -i s/pa_bt_/pa_bluez4_/g src/modules/bluetooth/*bluez4*
 $ sed -i s/PA_BT_/PA_BLUEZ4_/g src/modules/bluetooth/*bluez4*

bluetooth: Rename shared struct to make it BlueZ 4 specific

bluetooth: Rename bluetooth modules to BlueZ 4

bluetooth: Update modules description and copyright

The current set of bluetooth modules only support up to BlueZ 4. Since
the BlueZ API when through a big change with the release of BlueZ 5 the
modules will be forked into a new set for BlueZ 5.

This commit also fixes the spelling of Bluetooth (it's a trademark which
should always be spelled with capital B) and the spelling of my name,
and also update the copyright note dates throughout the Bluetooth
modules.

Revert "bluetooth: Detect BlueZ 5"

This reverts commit 2247b187393ef68879162faee8b422da9f5e556f.

This is part of the reversion of BlueZ 5 support so it can be added back
in a separate set of modules. This makes the code easier to maintain and
decrease PulseAudio's binary size.

Revert "bluetooth: Parse the tree returned by ObjectManager"

This reverts commit c4bd51a34547ed0991ff3c7219c2d5de49669ca8.

This is part of the reversion of BlueZ 5 support so it can be added back
in a separate set of modules. This makes the code easier to maintain and
decrease PulseAudio's binary size.

Revert "bluetooth: Fix error checking style"

This reverts commit d9ed42c40f744b273c8d466e95dd4a06e41364b9.

This is part of the reversion of BlueZ 5 support so it can be added back
in a separate set of modules. This makes the code easier to maintain and
decrease PulseAudio's binary size.

Revert "bluetooth: BlueZ 5 interface rename to org.bluez.MediaEndpoint1"

This reverts commit d22ea7ff7630bef83a3bf8c26cc56c196dec33a1.

This is part of the reversion of BlueZ 5 support so it can be added back
in a separate set of modules. This makes the code easier to maintain and
decrease PulseAudio's binary size.

Revert "bluetooth: BlueZ 5 interface rename to org.bluez.Media1"

This reverts commit 61e8fd8854308cb2d41e6313f25ae7131c912ea8.

This is part of the reversion of BlueZ 5 support so it can be added back
in a separate set of modules. This makes the code easier to maintain and
decrease PulseAudio's binary size.

Revert "bluetooth: BlueZ 5 interface rename to org.bluez.MediaTransport1"

This reverts commit cfb96b2530690df113e1aceb5b82fc166f9d2cfd.

This is part of the reversion of BlueZ 5 support so it can be added back
in a separate set of modules. This makes the code easier to maintain and
decrease PulseAudio's binary size.

Revert "bluetooth: Support ObjectManager interface add/remove"

This reverts commit 114edb0696ce979ad10f5616067afda1b76ae4e2.

This is part of the reversion of BlueZ 5 support so it can be added back
in a separate set of modules. This makes the code easier to maintain and
decrease PulseAudio's binary size.

Revert "bluetooth: Support Properties.PropertiesChanged signal"

This reverts commit 235611a7d13ab251689f5b480183d7f39e043a2e.

This is part of the reversion of BlueZ 5 support so it can be added back
in a separate set of modules. This makes the code easier to maintain and
decrease PulseAudio's binary size.

Revert "bluetooth: Parse media transport's properties"

This reverts commit 2f79fb580ad583b2492b567d32630e7e03883855.

This is part of the reversion of BlueZ 5 support so it can be added back
in a separate set of modules. This makes the code easier to maintain and
decrease PulseAudio's binary size.

Revert "bluetooth: Support media transport's State property"

This reverts commit 6fdf2b05b8867d4597c497243b469ecd0d8c2426.

This is part of the reversion of BlueZ 5 support so it can be added back
in a separate set of modules. This makes the code easier to maintain and
decrease PulseAudio's binary size.

Revert "bluetooth: Update to new BlueZ 5 transport acquire/release API"

This reverts commit 9615def4b96f0bab365ddc03f6c003f382a54752.

This is part of the reversion of BlueZ 5 support so it can be added back
in a separate set of modules. This makes the code easier to maintain and
decrease PulseAudio's binary size.

Revert "bluetooth: Support transport auto-release"

This reverts commit 0e4c16e120f95b122c7139deb046192927d1b6c7.

This is part of the reversion of BlueZ 5 support so it can be added back
in a separate set of modules. This makes the code easier to maintain and
decrease PulseAudio's binary size.

build-sys: Support building pulseaudio as a submodule

When pulseaudio is a submodule, .git is a file.

Signed-off-by: Anton Lundin <glance@acc.umu.se>

mutex: Fix compiler warning when priority inheritance is not supported

'r' is unused in the case where prioritiy inheritance is not detected at
configure time.