On Mon, Nov 21, 2011 at 4:50 PM, Dan Carpenter <dan.carpenter@xxxxxxxxxx> wrote: > We can't call mpi_free() on the elements if the first kzalloc() fails. > > Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> Signed-off-by: Dmitry Kasatkin <dmitry.kasatkin@xxxxxxxxx> > > diff --git a/lib/mpi/mpi-mpow.c b/lib/mpi/mpi-mpow.c > index 4cc7593..7328d0d 100644 > --- a/lib/mpi/mpi-mpow.c > +++ b/lib/mpi/mpi-mpow.c > @@ -73,7 +73,7 @@ int mpi_mulpowm(MPI res, MPI *basearray, MPI *exparray, MPI m) > > G = kzalloc((1 << k) * sizeof *G, GFP_KERNEL); > if (!G) > - goto nomem; > + goto err_out; > > /* and calculate */ > tmp = mpi_alloc(mpi_get_nlimbs(m) + 1); > @@ -129,5 +129,6 @@ nomem: > for (i = 0; i < (1 << k); i++) > mpi_free(G[i]); > kfree(G); > +err_out: > return rc; > } > -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
