Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RANK ORDER VIOLATION (inline sample client on any app) #1028

Closed
derekbruening opened this issue Nov 28, 2014 · 2 comments
Closed

RANK ORDER VIOLATION (inline sample client on any app) #1028

derekbruening opened this issue Nov 28, 2014 · 2 comments

Comments

@derekbruening
Copy link
Contributor

From [email protected] on December 16, 2012 14:26:36

we should do pre-release tests with both debug build and
release! I'm hitting a number of issues that way.


DynamoRIO Notice: d:\derek\dr\test\hello64.exe(15308)

rank order violation change_linking_lock(recursive)@......\git\src\core\link.c:108 acquired after fragment_delete_mutex(mutex)@......\git\src\core\fragment.c:2075 in tid:1444

dr_mark_trace_head() grabs the locks in the wrong order

Child-SP RetAddr Call Site

00 000000001e20d6c8 00000000153db3b6 ntdll!ZwRaiseHardError+0xa
01 000000001e20d6d0 0000000015394fbd dynamorio!nt_messagebox+0x176 [d:\derek\dr\git\src\core\win32\ntdll.c @ 3382]
02 000000001e20d770 0000000015174038 dynamorio!debugbox+0x5d [d:\derek\dr\git\src\core\win32\os.c @ 3734]
03 000000001e20d7a0 000000001516f8d3 dynamorio!notify+0x298 [d:\derek\dr\git\src\core\utils.c @ 1925]
04 000000001e20e010 00000000151707f2 dynamorio!deadlock_avoidance_lock+0x4a3 [d:\derek\dr\git\src\core\utils.c @ 595]
05 000000001e20e0c0 00000000151705dd dynamorio!mutex_trylock+0x92 [d:\derek\dr\git\src\core\utils.c @ 901]
06 000000001e20e100 0000000015171880 dynamorio!mutex_lock+0x7d [d:\derek\dr\git\src\core\utils.c @ 839]
07 000000001e20e140 000000001537fe83 dynamorio!acquire_recursive_lock+0x40 [d:\derek\dr\git\src\core\utils.c @ 983]
08 000000001e20e170 0000000072001388 dynamorio!dr_mark_trace_head+0x153 [d:\derek\dr\git\src\core\x86\instrument.c @ 5790]
09 000000001e20e220 000000001537244e inline!event_basic_block+0xc8 [d:\derek\dr\git\src\api\samples\inline.c @ 252]
0a 000000001e20e260 000000001530d337 dynamorio!instrument_basic_block+0x42e [d:\derek\dr\git\src\core\x86\instrument.c @ 1391]
0b 000000001e20e350 00000000152fb983 dynamorio!client_process_bb+0x167 [d:\derek\dr\git\src\core\x86\interp.c @ 2483]
0c 000000001e20e500 0000000015311674 dynamorio!build_bb_ilist+0x5203 [d:\derek\dr\git\src\core\x86\interp.c @ 3494]
0d 000000001e20ebd0 000000001514a3fc dynamorio!build_basic_block_fragment+0x424 [d:\derek\dr\git\src\core\x86\interp.c @ 4417]
0e 000000001e20ee70 000000001e18324b dynamorio!dispatch+0xb9c [d:\derek\dr\git\src\core\dispatch.c @ 190]

Original issue: http://code.google.com/p/dynamorio/issues/detail?id=1028

@derekbruening
Copy link
Contributor Author

From [email protected] on December 16, 2012 12:00:30

the client.custom_traces test that would have caught this has been disabled for issue #120 . never got around to re-enabling. issue #65 would also have caught this earlier.

@derekbruening
Copy link
Contributor Author

From [email protected] on December 17, 2012 08:06:33

This issue was closed by revision r1791 .

Status: Fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant