Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1265146

Re: [PATCH] crypto: sun4i-ss: add missing statesize

Show all headers | View raw

[Multipart message — attachments visible in raw view] - view raw

On Fri, Nov 06, 2015 at 12:56:39PM +0800, Herbert Xu wrote:
> On Thu, Nov 05, 2015 at 08:07:19AM -0800, Maxime Ripard wrote:
> > 
> > On Thu, Nov 05, 2015 at 08:48:57AM +0100, LABBE Corentin wrote:
> > > sun4i-ss implementaton of md5/sha1 is via ahash algorithms.
> > > A recent change make impossible to load them without giving statesize.
> > 
> > Which one?
> 
> We recently disabled ahash drivers that do not declare statesize
> because it can lead to a crash when the driver is used through
> algif.

"Recently" is relative and really doesn't help.

Having the commit ID that made this change is an absolute reference,
and really helps to identify when that behaviour changed.

> Not declaring statesize is a bug anyway but the fact that it
> is exported through algif makes it much worse.
> 
> > > This patch specifiy statesize for sha1 and md5.
> > > 
> > > Signed-off-by: LABBE Corentin <clabbe.montjoie@gmail.com>
> > > Cc: stable@vger.kernel.org
> > 
> > Please also add a Fixes tag (and the stable version it applies to).
> 
> I don't see the point for a fixes tag as it would simply refer
> to the original patch-set that added the driver.

What's the problem with that?

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH] crypto: sun4i-ss: add missing statesize LABBE Corentin <clabbe.montjoie@gmail.com> - 2015-11-05 08:50 +0100
  Re: [PATCH] crypto: sun4i-ss: add missing statesize Chen-Yu Tsai <wens@csie.org> - 2015-11-05 11:40 +0100
  Re: [PATCH] crypto: sun4i-ss: add missing statesize Maxime Ripard <maxime.ripard@free-electrons.com> - 2015-11-05 17:10 +0100
    Re: [PATCH] crypto: sun4i-ss: add missing statesize Herbert Xu <herbert@gondor.apana.org.au> - 2015-11-06 06:00 +0100
      Re: [PATCH] crypto: sun4i-ss: add missing statesize Maxime Ripard <maxime.ripard@free-electrons.com> - 2015-11-08 19:00 +0100
        Re: [PATCH] crypto: sun4i-ss: add missing statesize Christoph Biedl <linux-kernel.bfrz@manchmal.in-ulm.de> - 2015-11-08 19:30 +0100
          Re: [PATCH] crypto: sun4i-ss: add missing statesize Herbert Xu <herbert@gondor.apana.org.au> - 2015-11-09 05:00 +0100
            Re: [PATCH] crypto: sun4i-ss: add missing statesize Maxime Ripard <maxime.ripard@free-electrons.com> - 2015-11-11 23:30 +0100

csiph-web