Re: [PATCH crypto 2/2] crypto: caam - add allocation failure handling in SPRINTFCAT macro

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Monday, April 28, 2014 at 11:53:23 PM, Herbert Xu wrote:
> On Mon, Apr 28, 2014 at 09:28:41PM +0200, Marek Vasut wrote:
> > On Monday, April 28, 2014 at 12:24:41 PM, Herbert Xu wrote:
> > > On Fri, Apr 18, 2014 at 01:01:42PM +0300, Horia Geanta wrote:
> > > > GFP_ATOMIC memory allocation could fail.
> > > > In this case, avoid NULL pointer dereference and notify user.
> > > > 
> > > > Cc: <stable@xxxxxxxxxxxxxxx> # 3.2+
> > > > Cc: Kim Phillips <kim.phillips@xxxxxxxxxxxxx>
> > > > Signed-off-by: Horia Geanta <horia.geanta@xxxxxxxxxxxxx>
> > > 
> > > Patch applied to the crypto tree.  Thanks.
> > 
> > Hi Herbert,
> > 
> > Don't you instead want to apply my 11-patch series from
> > Message-Id: <1398362722-16430-1-git-send-email-marex@xxxxxxx>
> > Subject: [PATCH 00/11] crypto: caam: Error reporting fixes
> > which actually fixes this SPRINTFCAT() horror properly please ?
> 
> This patch is going into the current release while your patches
> are better suited for the next merge window.

I agree with you that we're rather late in the RC now, but I also believe this 
patch is just hiding the real bug and that we should fix the root cause of the 
problem instead of adding a plaster over it.

I also believe that while my 11-patch series is big, it should go into -stable 
eventually, since it fixes the real problem.

Best regards,
Marek Vasut
--
To unsubscribe from this list: send the line "unsubscribe linux-crypto" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Kernel]     [Gnu Classpath]     [Gnu Crypto]     [DM Crypt]     [Netfilter]     [Bugtraq]

  Powered by Linux