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.

Sign up for GitHub

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

Jump to bottom

Poco::NamedEvent does not release System V semaphore on Linux #2971

Open
ColovicAleksandar opened this issue Apr 2, 2020 · 3 comments · May be fixed by #4357
Open

Poco::NamedEvent does not release System V semaphore on Linux #2971

ColovicAleksandar opened this issue Apr 2, 2020 · 3 comments · May be fixed by #4357
Assignees
@matejk
Labels
bug platform_specific
Milestone
Release 1.14.0

Comments

@ColovicAleksandar
Copy link

Repro steps: create array of e.g. 256 Poco::NamedEvent instances. You'll get an error e.g.
System exception : cannot create named mutex ... semget() failed, errno=28

The Linux system allows up to 128 semaphore Ids and Poco::NamedEvent does not release semaphore id of the destructed Poco::NamedEvent instance.

I think in the NamedEvent_UNIX.cpp NamedEventImpl::~NamedEventImpl() there should be a call:

#if (POCO_OS == POCO_OS_LINUX)
// remove semaphores
semctl (semid, 0, IPC_RMID);
...
@obiltschnig obiltschnig added this to the Release 1.11.0 milestone May 2, 2020
@github-actions
Copy link

This issue is stale because it has been open for 365 days with no activity.

@github-actions github-actions bot added the stale label Oct 22, 2021
@github-actions
Copy link

This issue was closed because it has been inactive for 60 days since being marked as stale.

@aleks-f aleks-f reopened this Jul 7, 2022
@aleks-f aleks-f modified the milestones: Release 1.12.0, Release 1.13.0 Jul 7, 2022
@aleks-f aleks-f removed the stale label Jul 7, 2022
@github-actions
Copy link

This issue is stale because it has been open for 365 days with no activity.

@github-actions github-actions bot added the stale label Oct 25, 2023
@matejk matejk self-assigned this Dec 15, 2023
matejk added a commit that referenced this issue Dec 18, 2023
@matejk
fix(NamedEvent): Release semaphore ID in dtor when created with semget (
497bf6e 
#2971)
@matejk matejk added bug and removed stale labels Dec 18, 2023
matejk added a commit that referenced this issue Dec 18, 2023
@matejk
fix(NamedEvent): Release semaphore ID in dtor when created with semget (
eecf927 
#2971)
matejk added a commit that referenced this issue Dec 18, 2023
@matejk
fix(NamedEvent): Release semaphore ID in dtor when created with semget (
bee5a2a 
#2971)
matejk added a commit that referenced this issue Dec 22, 2023
@matejk
fix(NamedEvent): Release semaphore ID in dtor when created with semget (
b86b6a1 
#2971)
@matejk matejk modified the milestones: Release 1.13.1, Release 1.14.0 Jan 31, 2024
@matejk matejk linked a pull request Jan 31, 2024 that will close this issue
fix(NamedEvent): Release semaphore ID in dtor when created with semget (fixes #2971) #4357
Draft
matejk added a commit that referenced this issue Mar 9, 2024
@matejk
fix(NamedEvent): Release semaphore ID in dtor when created with semget (
47fb093 
#2971)
matejk added a commit that referenced this issue Mar 26, 2024
@matejk
fix(NamedEvent): Release semaphore ID in dtor when created with semget (
a2b106e 
#2971)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@matejk matejk

Labels
bug platform_specific
Projects
1.14
Status: In Progress
Milestone
Release 1.14.0
4 participants
@obiltschnig @matejk @aleks-f @ColovicAleksandar