* Matthew Toseland <[EMAIL PROTECTED]> [2008-01-28 14:13:40]:
> On Sunday 27 January 2008 13:40, [EMAIL PROTECTED] wrote:
> > Author: nextgens
> > Date: 2008-01-27 13:40:56 +0000 (Sun, 27 Jan 2008)
> > New Revision: 17318
> >
> > Modified:
> > trunk/freenet/src/freenet/crypt/DSAPrivateKey.java
> > Log:
> > doh!
> >
> > Modified: trunk/freenet/src/freenet/crypt/DSAPrivateKey.java
> > ===================================================================
> > --- trunk/freenet/src/freenet/crypt/DSAPrivateKey.java 2008-01-27
> > 13:32:37
> UTC (rev 17317)
> > +++ trunk/freenet/src/freenet/crypt/DSAPrivateKey.java 2008-01-27
> > 13:40:56
> UTC (rev 17318)
> > @@ -35,7 +35,7 @@
> > BigInteger tempX;
> > do {
> > tempX = new NativeBigInteger(256, r);
> > - } while (tempX.compareTo(g.getQ()) > -1 ||
> tempX.equals(BigInteger.ZERO));
> > + } while (tempX.compareTo(g.getQ()) > -1 ||
> tempX.compareTo(BigInteger.ZERO) < 1);
>
> I'm pretty sure we don't want it to equal zero. And new
> NativeBigInteger(numbits, random) will never return a negative number.
>
> So this patch is incorrect.from irc: [21:57] < toad_> | nextgens: i take it back
signature.asc
Description: Digital signature
_______________________________________________ Devl mailing list [email protected] http://emu.freenetproject.org/cgi-bin/mailman/listinfo/devl
