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

Sync problems. Experimental. DO NOT MERGE #2204

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

evgenyz
Copy link
Contributor

@evgenyz evgenyz commented Mar 13, 2025

Don't merge it!

  • E: oscap: DESC_WUNLOCK failed to unlock a mutex: Permission denied
  • E: oscap: DESC_WUNLOCK failed to unlock a mutex: Success
  • malloc_consolidate(): invalid chunk size (followed by Segmentation fault)

There are a lot of possible memory overruns reported by helgrind
coming from rbt_* operations without this flag.
evgenyz added 5 commits March 21, 2025 10:30
We should not re-initialize thread_barrier, it is an UB.
This is implementation dependent, frown upon and triggering
for sanitizers.
We should make sure that receiver gets something (error should do it),
even if the worker thread suddenly terminates (e.g. thread_exit).
SEAP_cmd_exec function waiting branch was prone to spurious wakeups.
Use pthread_cond_broadcast __SEAP_cmd_sync_handler to make sure
all possible listeners are notified.
These log records were introduced in anticipation of the
PCRE -> PCRE2 migration problems. We can get rid of them now.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant