[pulseaudio-discuss] [PATCH] sink: Increase max sink inputs per sink

Arun Raghavan arun.raghavan at collabora.co.uk
Fri Sep 20 06:17:42 PDT 2013


On Fri, 2013-09-20 at 14:55 +0200, David Henningsson wrote:
> On 09/20/2013 02:06 PM, Arun Raghavan wrote:
> > On Fri, 2013-09-20 at 09:29 +0200, David Henningsson wrote:
> >> On 09/20/2013 05:44 AM, Arun Raghavan wrote:
> >>> We're hitting the 32 sink-input limit quite often with increasing use of
> >>> PA by web browsers, so let's increase this limit.
> >>
> >> Does this increase the risk of us being the target of attacks; i e, if a
> >> web browser deliberately starts 255 streams, would we risk getting
> >> killed by using too much rtprio cpu time?
> > 
> > As Tanu says, this doesn't open up a new vector, really.
> > 
> >> Btw; is there a specific web browser you're looking at?
> > 
> > webkitgtk+ uses PulseAudio via GStreamer and Firefox has a PulseAudio
> > backend that maps each stream to a PA stream. Chrome/Chromium also has
> > as PA backend, but I don't recall off the top of my head if they do
> > their own mixing.
> 
> Thanks, patch acked from me.

Thanks for the review, pushed now.

-- Arun



More information about the pulseaudio-discuss mailing list