[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Magic Money Comments
Pr0duct Cypher at [email protected] wrote:
>> There is no prototype for pgp_randombyte.
> There is a prototype for it in pgptools.h
Yes, but it needs to say "byte pgp_randombyte(void);" instead of
just "byte pgp_randombyte();".
>> Complete prototypes for (*output) and (*lookup) need to be
>> provided in the function header of pgp_check_sigs and also for a
>> different function pointer (*output) used in PGPKGEN.
> Is this bad? I thought a function pointer didn't really
> care, as long as you called it correctly.
No - it is fine. This is simply a list of what it takes to
eliminate all warnings from Borland. It is certainly ok not
to do this. Even PGP generates quite a few warnings while
compiling.
> MPILIB has not been altered. It is the same as in PGP.
> PLATFORM appears to set the appropriate modmult, but I
> usually define UPTON because SMITH has given me some
> problems in the past.
Yes - I didn't notice all the various knobs and dials in
PLATFORM.H until after I wrote that message. I needed to
define a few more things.
> What are you running, a Cray? My machine takes quite a few
> seconds to run the server.
I am running on a 33 mhz 486. While MPILIB does indeed take
seconds to perform the required arithmetic, it builds its
operations from an O(N^2) multiply algorithm. A "fast"
algorithm would speed things up considerably.
> I hate error checking. If I put in safe read and write
> calls, what do I do if they fail?
Just exiting with an error message would be fine.
> Thanks for pointing out that bug, and especially for
> setting up a Magic Money server.
You are welcome. I'm not sure I have actually set up a
server aside from the purpose of exercising the code for a
few days. Hopefully others will grab the executables and do
something a bit more permanent.
--
Mike Duvos $ PGP 2.3a Public Key available $
[email protected] $ via Finger. $