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

linux SIGRTMAX=64 is inclusive #336

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

linux SIGRTMAX=64 is inclusive #336

derekbruening opened this issue Nov 28, 2014 · 4 comments

Comments

@derekbruening
Copy link
Contributor

From [email protected] on August 11, 2010 13:06:03

this is a known bug in DR but hasn't been fixed since hasn't cause problems yet, but filing in issue database since now I see it on python:

Linux RT signals actually include 64 as a valid signal number.
The Linux kernel subtracts one from signal numbers before using as
array or bitmask refs, to keep its structs an even 64 size.
This is very confusing and various info is contradictory,
but it seems that there are 33 RT signals, numbered 32..64.


running 64-bit python on my FC12 box hits this issue

<Application python (5887). Internal Error Internal DynamoRIO Error: /work/dr/tot/opensource/core/linux/signal.c:1686 sig < MAX_SIGNUM
(Error occurred @5975 frags)

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

@derekbruening
Copy link
Contributor Author

From [email protected] on May 11, 2011 11:18:14

Issue 456 has been merged into this issue.

@derekbruening
Copy link
Contributor Author

From [email protected] on May 11, 2011 11:20:29

Owner: [email protected]

@derekbruening
Copy link
Contributor Author

From [email protected] on May 11, 2011 11:33:35

the complication is w/ 0-based vs 1-based signal numbers (b/c 0 unused) which is done for sigsets but not elsewhere so the fix requires more than just bumping the constant (else I would have done that instead of adding the comment): needs two different constants or changes to the loops or sthg.

@derekbruening
Copy link
Contributor Author

From [email protected] on October 25, 2011 12:44:37

This was fixed in r992 , and I can run CPython under DynamoRIO.

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