On 15 July 2016 at 11:14, David Henningsson <di...@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
_______________________________________________
pulseaudio-discuss mailing list
pulseaudio-discuss@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss

Reply via email to