summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* doc: Update NEWS for 7.1 releasev7.1stable-7.xArun Raghavan2015-10-301-0/+18
|
* alsa-ucm: fix pa_device_port.profiles leakTanu Kaskinen2015-10-301-1/+0
| | | | | pa_device_port.profiles is initialized in pa_device_port_new(), so no need to do that in alsa-ucm.c.
* null-sink: correct update of timestamp when state changes to RUNNING or IDLEGeorg Chini2015-10-301-2/+4
| | | | | | | | u->timestamp should be reset to "now" when the sink state changes from INIT or SUSPENDED to IDLE or RUNNING. Thanks to Olivier Delbeke for reporting the bug and testing the patch. Bug report is here: http://lists.freedesktop.org/archives/pulseaudio-discuss/2015-October/024553.html
* alsa-mixer: Ignore volume with unexpected number of channelsDavid Henningsson2015-10-301-3/+3
| | | | | | | | | | | | | | We currently only support one and two channels for volumes, and bail out otherwise. This makes Xonar users unhappy because they have a volume with eight channels, and bailing out means they don't have a path/port at all. This way they will at least have a port, which will in turn make the gnome/unity UI behave better. BugLink: https://bugs.freedesktop.org/show_bug.cgi?id=84983 BugLink: https://bugzilla.gnome.org/show_bug.cgi?id=745017 Signed-off-by: David Henningsson <david.henningsson@canonical.com>
* pstream: Fix use-after-free in srb_callbackDavid Henningsson2015-10-301-1/+10
| | | | | | | | | | We need to guard the pstream with an extra ref to ensure it is not destroyed at the time we check whether or not the srbchannel is destroyed. Reported-by: Takashi Iwai <tiwai@suse.de> BugLink: http://bugzilla.opensuse.org/show_bug.cgi?id=950487 Signed-off-by: David Henningsson <david.henningsson@canonical.com>
* shm: Warn on not being able to open shm filesDavid Henningsson2015-10-301-3/+7
| | | | | | | With the exception of when trying to clean up shm files, it's useful to warn if opening them fails, regardless of reason. Signed-off-by: David Henningsson <david.henningsson@canonical.com>
* build-sys: fix the default bash completion directoryTanu Kaskinen2015-10-301-1/+1
| | | | | | If bashcompletiondir was empty, the check didn't catch that. As a result, the symlinks that were supposed to be generated in the completion directory were created in the root directory.
* alsa-mixer: Add "Dock Line Out" jack handling to analog-output-speaker-alwaysTanu Kaskinen2015-10-301-0/+4
| | | | | The jack is already in analog-output-speaker, so let's have the same handling in analog-output-speaker-always too.
* alsa-mixer: Recognize "Dock Line Out" jackTakashi Iwai2015-10-302-0/+12
| | | | | | | Some machines provide "Dock Line Out Jack" control that should be handled like a normal line out. Signed-off-by: Takashi Iwai <tiwai@suse.de>
* alsa-mixer: Add "Mic Boost" and "Line Boost" to analog-input.confDavid Henningsson2015-10-301-0/+6
| | | | | | | | | I'm not sure how much they are needed nowadays with the latest changes to the subset elimination (I found this while researching a bug on an older PA version), but I guess they could be added for consistency at least. Signed-off-by: David Henningsson <david.henningsson@canonical.com>
* pulseaudio.1: document that systemd notification only works without ↵Felipe Sateler2015-10-301-1/+3
| | | | daemonization
* daemon: Use Type=notify for systemd serviceFelipe Sateler2015-10-302-0/+10
| | | | | This ensures systemd knows when pulseaudio finishes loading, thus never marking pulseaudio as active if startup fails.
* Update NEWS for 7.0 releasev7.0Arun Raghavan2015-09-241-0/+5
|
* i18n: Update Slovak translationDušan Kazik2015-09-231-202/+261
|
* tests: Fix test-suite failure on AlphaMichael Cree2015-09-171-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Pulseaudio fails to build on the Alpha architecture due to a failure in the volume-test of the test suite. I had reported this to the Debian bug tracker [1] but the maintainer has asked that I forward the patch to this mail list. The failure in volume-test occurs because it is compiled with -ffast-math which implies -ffinite-math-only of which the gcc manual states that it optimizes for floating-point arithmetic with the assumption that arguments and results are not NaNs or +/-infinity, and futher notes that it may result in incorrect output. On the Alpha platform that is somewhat an understatement as the use of non-finite floating-point arithmetic with -ffinite-math-only results in a floating-point exception and the termination of the program. The volume-test converts volumes into decibels (so a zero volume becomes a negative infinity) and proceeds to add two volumes (in decibels), thus does arithmetic with non-finite floating point numbers despite being compiled with -ffast-math! I attach a patch that protects against the arithmetic with non-finite numbers for your consideration. With that patch the test-suite passes on Alpha. Cheers Michael. [1] https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=798248
* lfe-filter: Deal with empty input chunksArun Raghavan2015-09-171-1/+1
| | | | | | | | | | | | | | | | | It is possible that we get a zero-length memchunk to work with. Specifically, this happens the resampler (which is called before the lfe-filter) consumes all the input data, but does not (yet) produce any output data. Reproduced using: pulseaudio --resample-method=soxr-mq pactl load-module module-null-sink sink_name=lfe_test channels=3 channel_map=front-left,front-right,lfe paplay --raw /dev/zero --rate=48000 -d lfe_test Thanks to the original reporter for the backtrace: Bug: https://bugs.launchpad.net/ubuntu/+source/pulseaudio/+bug/1496577
* modules: Fix entry leak in module-card-restorePeter Meerwald2015-09-161-1/+4
| | | | | | CID 1323585 entry_read() allocates an entry which must be freed
* core: Add missing return on protocol errorPeter Meerwald2015-09-161-1/+3
| | | | | | CID 1323582 when a protocol error occors, better bail out :)
* daemon: No need to check optarg, -p requires argumentPeter Meerwald2015-09-161-1/+1
| | | | | | | CID 1323589 getopt() makes sure that we have an argument for -p remove the broken check for optarg being set
* loopback: Fix the obviously-wrong "buffer+=buffer" logicAlexander E. Patrakov2015-09-131-1/+1
| | | | | | | | | Originally pointed out by Georg Chini. Calculating buffer = buffer + (send_counter - recv_counter) in one branch and buffer = 2 * buffer - (recv_counter - send_counter) looks very obviously wrong. In other words, before the patch, the contribution from the previous lines was double-counted.
* build-sys: add sb-omni-surround-5.1.conf to Makefile.amv6.99.2Tanu Kaskinen2015-09-121-1/+2
| | | | This is needed to get the file included in the release tarballs.
* build-sys: bump sonamesTanu Kaskinen2015-09-121-1/+1
| | | | There were a couple of bug fixes to libpulse since 6.99.1.
* sink, source: Do not dereference freed memory when freeing the next eventsFelipe Sateler2015-09-122-2/+4
| | | | Coverity IDs: 1138197, 1138196
* util: pa_get_fqdn: always free addrinfoFelipe Sateler2015-09-121-2/+7
| | | | | | Also fix the return error code check of getaddrinfo call Coverity ID: 1323587
* i18n: Update Polish translationPiotr Drąg2015-09-121-656/+752
|
* Fix return code check of pa_append_to_home_dirFelipe Sateler2015-09-111-1/+1
| | | | | | It returns negative on error, zero on success. Found by coverity
* build-sys: fix order of libraries installationZavadovsky Yan2015-09-101-1/+1
| | | | | | | | | | | Libraries from modlibexec_LTLIBRARIES list require not only libpulsecommon but also libpulse and libpulsecore from lib_LTLIBRARIES list. This patch fix race in 'make -j X install' (with X is 2 and more) when building/installing inside chroot placed on RAM-disk(tmpfs). Signed-off-by: Zavadovsky Yan <zavadovsky.yan@gmail.com>
* module: Always remove freed module from modules_pending_unloadDavid Henningsson2015-09-071-2/+3
| | | | | | | | | pa_module_free is called from more than one place, not all of these places correctly removed the module from the modules_pending_unload array, potentially causing a dangling pointer in that array. Signed-off-by: David Henningsson <david.henningsson@canonical.com>
* core-util: include xlocale.h when using strtod_l()Tanu Kaskinen2015-09-071-0/+1
| | | | | | | | | | | | | | Based on some googling, strtod_l() is defined in xlocale.h on BSD. Glibc seems to define it in stdlib.h, but only if GNU extensions are enabled (otherwise the function won't be available). So, this patch should fix the use of strtod_l() on BSDs, but on other systems things may or may not be still broken. The original patch author is Jakob Fink <jfink@gmx.at>. He sent this patch to the freebsd-gnome mailing list: http://lists.freebsd.org/pipermail/freebsd-gnome/2015-April/032138.html BugLink: https://bugs.freedesktop.org/show_bug.cgi?id=90285
* build-sys: check strtod_l instead of strtof_lTanu Kaskinen2015-09-072-4/+4
| | | | | | | | | | We have no strtof_l calls in the code, so it doesn't make sense to check that function's availability. We have one strtod_l call, so let's check that instead. I don't know if this change makes any practical difference. I just wondered why we had HAVE_STRTOF_L ifdefs in core-util.c for code that didn't use strtof_l.
* alsa-mixer: sb-omni-surround-5.1.conf: remove analog-surround-40 mappingTanu Kaskinen2015-08-281-7/+0
| | | | | It was reported that the 4.0 mode doesn't work properly: http://thread.gmane.org/gmane.comp.audio.pulseaudio.general/23677/focus=23904
* alsa-mixer: sb-omni-surround-5.1.conf: rename analog-stereo to ↵Tanu Kaskinen2015-08-281-2/+3
| | | | | | | analog-stereo-output The mapping is only useful for output, and this renaming makes the name symmetric with the input mapping.
* alsa-mixer: Add support for Creative SoundBlaster Omni Surround 5.1 USB ↵Nazar Mokrynskyi2015-08-284-0/+93
| | | | | | | sound card Mic and Line in inputs were not working at all, since they use the hw:x,1,0 device, which is not supported in the default configuration.
* update NEWS for 7.0v6.99.1Tanu Kaskinen2015-08-271-0/+48
|
* build-sys: bump sonamesTanu Kaskinen2015-08-271-1/+1
| | | | | | | In this release cycle, libpulse had some internal code changes, but no ABI changes. libpulse-simple and libpulse-mainloop-glib had no changes whatsoever.
* alsa: Clarify potentially confusing device port data fieldArun Raghavan2015-08-251-1/+1
| | | | | The meaning of the code is unchanged here, but it is clearer that the PA_DEVICE_PORT_DATA field is a struct ucm_port pointer.
* alsa-mixer: Drop redundant conditional freesArun Raghavan2015-08-251-5/+2
| | | | | The dynarrays are allocated unconditionally, so the free need not be conditional.
* ucm: Add support for "JackHWMute"Tanu Kaskinen2015-08-214-0/+134
| | | | | | JackHWMute is used to list devices that get forcibly muted by a particular jack. We mark ports unavailable based on that information.
* ucm: Create only one jack object per kcontrolTanu Kaskinen2015-08-213-20/+27
| | | | | | | | | Previously the UCM code created one jack object per device name (which is not the same thing as creating one jack object per device, because the UCM device namespace is scoped on per-verb basis, so devices in different verbs may have the same name). I think it's conceptually cleaner to create one jack object per alsa kcontrol. I plan to do similar refactoring on the traditional mixer code later.
* alsa: Move UCM port availability updating to the mixer codeTanu Kaskinen2015-08-216-17/+112
| | | | | | | | | | | | | | | | Previously module-alsa-card assigned to pa_alsa_jack.plugged_in directly, and then did the port availability updating manually. The idea of pa_alsa_jack_set_plugged_in() is to move the availability updating to the mixer infrastructure, where it really belongs. Similarly, pa_alsa_jack.has_control was previously modified directly from several places. The has_control field affects the port availability, and pa_alsa_jack_set_has_control() takes care of updating the availability. For now, pa_alsa_jack_set_plugged_in() and pa_alsa_jack_set_has_control() only update the port availability when using UCM. My plan is to adapt the traditional mixer code later.
* alsa: Add associations between jacks, UCM devices and UCM portsTanu Kaskinen2015-08-214-2/+90
| | | | | These associations will be used by subsequent UCM jack detection refactoring work.
* ucm: Don't create separate input and output jacksTanu Kaskinen2015-08-212-20/+19
| | | | | | The UCM spec doesn't support separate input and output jacks, so it's redundant to have separate input and output jacks in pa_alsa_ucm_device.
* alsa: Add pa_alsa_jack_new() and pa_alsa_jack_free()Tanu Kaskinen2015-08-213-25/+39
| | | | | This reduces code duplication in alsa-mixer.c and alsa-ucm.c. No functional changes.
* alsa-mixer: Add "Front Line Out" and "Rear Line Out"David Henningsson2015-08-213-0/+32
| | | | | | | | | | | | "Front Line Out" was found in the wild on one of the machines we enable. I figured I could just as well add "Rear Line Out" too, because that's just as likely to show up. As a reminder, "Front Line Out" means "a line out jack physically located on the front side", where as "Line Out Front" means "a line out jack playing back front left and front right channels in a channel map". Signed-off-by: David Henningsson <david.henningsson@canonical.com>
* build-sys: make error message visibleTanu Kaskinen2015-08-211-1/+1
| | | | | | | | This helps figuring out why bootstrap.sh is failing... Directing the error message to /dev/null was very strange. I don't know what the original motivation might have been. My guess is that it was added unintentionally.
* alsa-mixer: Add descriptions for analog-stereo-input and ↵Tanu Kaskinen2015-08-211-2/+7
| | | | | | | | analog-stereo-output mappings These mapping names are used in sb-omni-surround-5.1.conf, which needs to use separate mappings for input and output, since they are associated with different alsa devices.
* pulse: Document client libraries logging behaviorAhmed S. Darwish2015-08-171-0/+33
| | | | | | | | | | | Document how to modify the client libraries logging behvaior using any of the PA-specific environment variables. Using the PULSE_LOG_* environment variables makes debugging and tracing PA applications quite easy, thus the need for an official documentation text. Signed-off-by: Ahmed S. Darwish <darwish.07@gmail.com>
* Removed exclamation marks from user-visible messages.Deepak Srivastava2015-08-144-9/+9
| | | | | | | | | | <EP-E358F00C1D9A449EAE69225B9D2530F8> According to rationale-"http://techbase.kde.org/Projects/Usability/HIG/Exclamation_points" as suggested in reported bug. Component: misc BugLink: https://bugs.freedesktop.org/show_bug.cgi?id=78563 Signed-off-by: Deepak Srivastava <srivastava.d@samsung.com>
* patcl: Error msg from pactl not quite accurate for 'mute' commandsDeepak Srivastava2015-08-121-4/+4
| | | | | | | | | | <EP-E358F00C1D9A449EAE69225B9D2530F8> Updated the error string for mute commands in pactl. If someone forget to specify the parameter on the commandline the resulting error message isn't quite right. BugLink: https://bugs.freedesktop.org/show_bug.cgi?id=90117 Signed-off-by: Deepak Srivastava <srivastava.d@samsung.com>
* pacmd: Placeholders describing command line options should not have spacesManish Sogi2015-08-121-1/+1
| | | | | | | <EP-F69A421D31544C56BDE97C590307C014> Buglink: https://bugs.freedesktop.org/show_bug.cgi?id=78565 Signed-off-by: Manish Sogi <manish.sogi@samsung.com>