[pulseaudio-discuss] Access control

[Wim Taymans]

On 15 July 2016 at 11:14, David Henningsson <diwic at ubuntu.com> wrote:

>
>
> On 2016-07-15 11:05, Wim Taymans wrote:
>
>> Hi guys,
>>
>> I'm having another look at the access control patches. I revived my old
>> patches and found some trouble with the async stuff that I fixed here:
>>
>> https://cgit.freedesktop.org/~wtay/pulseaudio/log/?h=access-hooks <
>> https://cgit.freedesktop.org/%7Ewtay/pulseaudio/log/?h=access-hooks>
>>
>
> FWIW, I also remember fixing a bug or two before I added your patches to
> Ubuntu. The way it looks in Ubuntu now is here:
>
>
> http://anonscm.debian.org/cgit/pkg-pulseaudio/pulseaudio.git/tree/debian/patches?h=ubuntu
>
> (see patches 0406, 0407 and 0408)
>

It looks ok, you fixed the length in _copy and the reading of the command
and tag when resuming the async operation.

Interestingly you added the pa_creds to the pa_client. Any reason not to
upstream this?

Wim
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/pulseaudio-discuss/attachments/20160715/d8683841/attachment.html>