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

No type checking of GIDs in stubs classes #258

Closed
brycelelbach opened this Issue Jul 10, 2012 · 3 comments

Comments

Projects
None yet
1 participant
@brycelelbach
Member

brycelelbach commented Jul 10, 2012

[reported by blelbach] [Trac time Tue Nov 8 18:14:28 2011] One of our common use patterns is to use the prefix/locality GID to remotely access the runtime support services of a locality. We do this by calling into the runtime support stub class directly. Unfortunately, no type checking is performed here. If the gid passed in does not point to a runtime support component, the component is accessed anyways. This is very bad as it can lead to data corruption!

@brycelelbach

This comment has been minimized.

Show comment
Hide comment
@brycelelbach

brycelelbach Jul 10, 2012

Member

[comment by hkaiser] [Trac time Tue Nov 8 19:15:19 2011] That's being check in action_manager, well it would be if Bryce had not commented out the corresponding tests in rev 1210972 :-P.

Member

brycelelbach commented Jul 10, 2012

[comment by hkaiser] [Trac time Tue Nov 8 19:15:19 2011] That's being check in action_manager, well it would be if Bryce had not commented out the corresponding tests in rev 1210972 :-P.

@brycelelbach

This comment has been minimized.

Show comment
Hide comment
@brycelelbach

brycelelbach Jul 10, 2012

Member

[comment by blelbach] [Trac time Mon Jan 23 20:18:34 2012] Oh :p.

Member

brycelelbach commented Jul 10, 2012

[comment by blelbach] [Trac time Mon Jan 23 20:18:34 2012] Oh :p.

@brycelelbach

This comment has been minimized.

Show comment
Hide comment
@brycelelbach

brycelelbach Jul 10, 2012

Member

[comment by blelbach] [Trac time Tue Feb 28 14:32:27 2012] r7208

Member

brycelelbach commented Jul 10, 2012

[comment by blelbach] [Trac time Tue Feb 28 14:32:27 2012] r7208

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