From mboxrd@z Thu Jan 1 00:00:00 1970 From: Takashi Iwai Date: Fri, 14 Sep 2012 09:45:04 +0000 Subject: Re: [patch] ALSA: compress_core: integer overflow in snd_compr_allocate_buffer() Message-Id: List-Id: References: <20120905123217.GD6128@elgon.mountain> <20120906145912.GK19410@mwanda> <1347614682.1943.35.camel@vkoul-udesk3> <1347614934.1943.38.camel@vkoul-udesk3> In-Reply-To: <1347614934.1943.38.camel@vkoul-udesk3> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Vinod Koul Cc: alsa-devel@alsa-project.org, kernel-janitors@vger.kernel.org, Pierre-Louis Bossart , Jesper Juhl , Namarta Kohli , Dan Carpenter At Fri, 14 Sep 2012 14:58:54 +0530, Vinod Koul wrote: > > On Fri, 2012-09-14 at 14:54 +0530, Vinod Koul wrote: > > On Fri, 2012-09-14 at 11:06 +0200, Takashi Iwai wrote: > > > > So, we actually should have two distinct fixes. > > > > > > I took Dan's patch now for next branch. But it's still better to > > > filter weird parameters in the caller side, too. > > > > > > Vinod, care to write such a patch and submit? > > Yes am already on it :) > Btw I saw that you took Dan's fix for flags: > ALSA: compress_core: fix open flags test in snd_compr_open() > > Did you apply anything for this? You mean anything else? So far only two patches from Dan. Takashi