fix initstate to make the state buffer usable in setstate
authorSzabolcs Nagy <nsz@port70.net>
Tue, 21 Jan 2014 01:01:35 +0000 (02:01 +0100)
committerSzabolcs Nagy <nsz@port70.net>
Tue, 21 Jan 2014 01:01:35 +0000 (02:01 +0100)
setstate could use the results of previous initstate or setstate
calls (they return the old state buffer), but the documentation
requires that an initialized state buffer should be possible to
use in setstate immediately, which means that initstate should
save the generator parameters in it.

I also removed the copyright notice since it is present in the
copyright file.

src/prng/random.c

index 4ad6205..e250e28 100644 (file)
@@ -1,10 +1,3 @@
-/*
- * random.c - Copyright © 2011 Szabolcs Nagy
- * Permission to use, copy, modify, and/or distribute this code
- * for any purpose with or without fee is hereby granted.
- * There is no warranty.
-*/
-
 #include <stdlib.h>
 #include <stdint.h>
 #include "libc.h"
@@ -12,11 +5,7 @@
 /*
 this code uses the same lagged fibonacci generator as the
 original bsd random implementation except for the seeding
-
-different seeds produce different sequences with long period
-(other libcs seed the state with a park-miller generator
-when seed=0 some fail to produce good random sequence
-others produce the same sequence as another seed)
+which was broken in the original
 */
 
 static uint32_t init[] = {
@@ -98,6 +87,7 @@ char *initstate(unsigned seed, char *state, size_t size) {
                n = 63;
        x = (uint32_t*)state + 1;
        __srandom(seed);
+       savestate();
        UNLOCK(lock);
        return old;
 }