Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#9416 closed enhancement (fixed)

dev/random needs to be writable

Reported by: fishpond Owned by: nobody
Priority: normal Milestone: R1
Component: Drivers Version: R1/Development
Keywords: random, OpenJDK Cc: hamish
Blocked By: Blocking:
Platform: All

Description

dev/random currently is read-only. On other OSs, writing data to it is used for seeding or increasing the entropy. OpenJDK's NativePRNG class uses dev/random writes and will thus throw exceptions on re-seeding. With the attached patch, I'm able to run Netbeans 7.3beta2 with only few problems remaining.

Attachments (2)

driver.patch (981 bytes ) - added by fishpond 12 years ago.
Patch for add-ons/kernel/drivers/random/driver.cpp
0001-Added-capability-to-accept-writes-to-dev-random.patch (2.4 KB ) - added by fishpond 12 years ago.

Download all attachments as: .zip

Change History (10)

by fishpond, 12 years ago

Attachment: driver.patch added

Patch for add-ons/kernel/drivers/random/driver.cpp

comment:1 by fishpond, 12 years ago

patch: 01

comment:2 by fishpond, 12 years ago

Type: bugenhancement

comment:3 by diver, 12 years ago

Thanks for the patch! Could you recreate it using git format-patch.

comment:4 by fishpond, 12 years ago

Here you go! Also removed the tracing which was still activated in the last patch.

comment:5 by korli, 12 years ago

Patch is acceptable as is, but it seems like C style, whereas the file is C++. This should be improved at a later time.

comment:6 by korli, 12 years ago

Resolution: fixed
Status: newclosed

Edited and applied in hrev45228. Thanks!

comment:7 by korli, 12 years ago

Cc: hamish added

Hamish, this could be of interest for you in relation to http://hg.openjdk.java.net/haiku/haiku/jdk/rev/140b6d549a5e

comment:8 by hamish, 12 years ago

It is indeed. I've removed the workaround now. Thanks korli, and thanks fishpond for the patch.

Note: See TracTickets for help on using tickets.