#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: | ||
| Has a Patch: | yes | 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)
Change History (10)
Changed 2 years ago by fishpond
comment:1 Changed 2 years ago by fishpond
- Has a Patch set
comment:2 Changed 2 years ago by fishpond
- Type changed from bug to enhancement
comment:3 Changed 2 years ago by diver
Thanks for the patch! Could you recreate it using git format-patch.
Changed 2 years ago by fishpond
comment:4 Changed 2 years ago by fishpond
Here you go! Also removed the tracing which was still activated in the last patch.
comment:5 Changed 2 years ago by korli
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 Changed 2 years ago by korli
- Resolution set to fixed
- Status changed from new to closed
Edited and applied in hrev45228. Thanks!
comment:7 Changed 2 years ago by korli
- 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 Changed 2 years ago by hamish
It is indeed. I've removed the workaround now. Thanks korli, and thanks fishpond for the patch.
Patch for add-ons/kernel/drivers/random/driver.cpp