aboutsummaryrefslogtreecommitdiffstats
path: root/erts/emulator/beam/erl_time_sup.c
diff options
context:
space:
mode:
authorMikael Pettersson <[email protected]>2010-06-30 14:31:04 +0200
committerBjörn Gustavsson <[email protected]>2010-08-27 12:21:33 +0200
commit90108371943ace300f1dcf1543545a40be035a4a (patch)
treeea5f34ab7bad21635f0397e394dbbc9f39bcfd7a /erts/emulator/beam/erl_time_sup.c
parent980773b6f40133feff45b0ad9e811174b85034d9 (diff)
downloadotp-90108371943ace300f1dcf1543545a40be035a4a.tar.gz
otp-90108371943ace300f1dcf1543545a40be035a4a.tar.bz2
otp-90108371943ace300f1dcf1543545a40be035a4a.zip
fix native code crash when calling unloaded module with on_load function
As reported in erlang-bugs, the following sequence of events crashes the VM: 1. Module M1 is loaded and in native mode. 2. Module M2 is not loaded, in emulated mode, and has an on_load function. 3. M1 calls some function in M2. This works. 4. M1 again calls some function in M2. This segfaults. The reason for the crash is that when the beam loader fixes up export entries after a successful on_load function call, it erroneously clears the ->code[3] field in that module's export entries. This is redundant (no code in beam relies on ->code[3] being NULL), inconsistent with modules without on_load functions (there ->code[3] remains a valid beam instruction after the module is loaded), and breaks native code which needs the old ->address value in an export entry to remain valid after a module load step (before the load ->address points to ->code[3], after the load ->address points to the real code but uses of the old ->address value remain so ->code[3] must remain valid). Thus the fix for the crash is to simply not clear ->code[3]. This patch fixes R14A and should also fix R13B04. (There does exist a performance bug in this area, but it is unrelated to the on_load feature so will be fixed separately.)
Diffstat (limited to 'erts/emulator/beam/erl_time_sup.c')
0 files changed, 0 insertions, 0 deletions