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

posix ostimer.c functions not using semaphore #94

Closed
skliper opened this issue Sep 30, 2019 · 6 comments
Closed

posix ostimer.c functions not using semaphore #94

skliper opened this issue Sep 30, 2019 · 6 comments

Comments

@skliper
Copy link
Contributor

skliper commented Sep 30, 2019

Similar to Trac #92, the posix version of ostimer.c is not properly protecting its OS_timer_table via a mutex.

{{{
pthread_mutex_t OS_timer_table_mut;
}}}

Many of the issues seen in Trac #92 are in the posix version as well.
Full branch coverage unit tests, in Trac #45, were used to fix Trac #92 (VxWorks). The posix ostimer.c will likely need similar coverage tests to ensure this is fixed properly.

@skliper skliper self-assigned this Sep 30, 2019
@skliper
Copy link
Contributor Author

skliper commented Sep 30, 2019

Imported from trac issue 71. Created by abrown4 on 2015-07-08T13:19:36, last modified: 2019-07-17T16:58:53

@skliper
Copy link
Contributor Author

skliper commented Sep 30, 2019

Trac comment by abrown4 on 2015-07-08 13:22:45:

Not all methods in the RTEMS version are protected... that may need its own issue as well.

@skliper
Copy link
Contributor Author

skliper commented Sep 30, 2019

Trac comment by jphickey on 2015-07-08 16:14:19:

Note that this is already all fixed under trac #28

@skliper
Copy link
Contributor Author

skliper commented Sep 30, 2019

Trac comment by jphickey on 2019-03-26 08:56:51:

Adding this to the group of bugs that are fixed by using the "NG" architecture

@skliper
Copy link
Contributor Author

skliper commented Sep 30, 2019

Trac comment by jhageman on 2019-07-01 15:35:44:

Marked for CCB review - either close as last-gen only, wontfix or fixed by next gen already

@skliper
Copy link
Contributor Author

skliper commented Sep 30, 2019

Trac comment by jhageman on 2019-07-17 16:58:53:

CCB 7/17/2019 - fixed by next gen, closing last gen issue as wontfix

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