The function os::random() uses long but the calculations are for 32 bit random number. The function has been in the code since the beginning.

os::random() also has a _rand_seed which is modified without thread safety. It doesn't hurt anything since most uses for Symbol are called under the SymbolTable_lock.

If more multithreaded uses are added, this could hurt the randomness of the value returned.