[Bug go/52341] New: crypto/rand FAILs before Solaris 11

[Bug go/52341] New: crypto/rand FAILs before Solaris 11

[ro at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52341

             Bug #: 52341
           Summary: crypto/rand FAILs before Solaris 11
    Classification: Unclassified
           Product: gcc
           Version: 4.7.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: go
        AssignedTo: ian@airs.com
        ReportedBy: ro@gcc.gnu.org
              Host: *-*-solaris2.{8,9,10}
            Target: *-*-solaris2.{8,9,10}
             Build: *-*-solaris2.{8,9,10}


Before Solaris 11, the libgo crypto/rand test FAILs like this:

--- FAIL: rand.TestRead (0.00 seconds)
    ???:1: Read(buf) = 1040, %!s(<nil>)
FAIL
FAIL: crypto/rand

There's a limit on /dev/urandom here, while not properly documented in
random(7D), it can be found in the OpenSolaris sources and is alluded to in
Solaris 11 random(7D):

     The limitation per read for /dev/random is 1040  bytes.  The
     limit for /dev/urandom is (128 * 1040).

The 1040 byte limit also applies to /dev/urandom before Solaris 11.

  Rainer

[Bug go/52341] crypto/rand FAILs before Solaris 11

[ian at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52341

--- Comment #2 from ian at gcc dot gnu.org <ian at gcc dot gnu.org> 2012-04-25 04:41:23 UTC ---
Author: ian
Date: Wed Apr 25 04:41:13 2012
New Revision: 186804

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=186804
Log:
    PR go/52341

crypto/rand: Use io.ReadFull when reading from /dev/urandom.

Modified:
    branches/gcc-4_7-branch/libgo/go/crypto/rand/rand_test.go

[Bug go/52341] crypto/rand FAILs before Solaris 11

[ian at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52341

--- Comment #1 from ian at gcc dot gnu.org <ian at gcc dot gnu.org> 2012-04-25 04:41:05 UTC ---
Author: ian
Date: Wed Apr 25 04:40:49 2012
New Revision: 186803

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=186803
Log:
    PR go/52341

crypto/rand: Use io.ReadFull when reading from /dev/urandom.

Modified:
    trunk/libgo/go/crypto/rand/rand_test.go

[Bug go/52341] crypto/rand FAILs before Solaris 11

[ian at airs dot com]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52341

Ian Lance Taylor <ian at airs dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
            Version|4.7.0                       |4.7.1
         Resolution|                            |FIXED

--- Comment #3 from Ian Lance Taylor <ian at airs dot com> 2012-04-25 04:41:51 UTC ---
Should be fixed now.