Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Skip to content

Commit b6cfede

Browse files
committed
Fix typo in pg_srand48 (srand48 in older branches).
">" should be ">>". This typo results in failure to use all of the bits of the provided seed. This might rise to the level of a security bug if we were relying on srand48 for any security-critical purposes, but we are not --- in fact, it's not used at all unless the platform lacks srandom(), which is improbable. Even on such a platform the exposure seems minimal. Reported privately by Andres Freund.
1 parent 7c0e1a2 commit b6cfede

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/port/erand48.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -92,7 +92,7 @@ srand48(long seed)
9292
{
9393
_rand48_seed[0] = RAND48_SEED_0;
9494
_rand48_seed[1] = (unsigned short) seed;
95-
_rand48_seed[2] = (unsigned short) (seed > 16);
95+
_rand48_seed[2] = (unsigned short) (seed >> 16);
9696
_rand48_mult[0] = RAND48_MULT_0;
9797
_rand48_mult[1] = RAND48_MULT_1;
9898
_rand48_mult[2] = RAND48_MULT_2;

0 commit comments

Comments
 (0)