Skip to content

Fix pthread returns#444

Merged
chrissie-c merged 1 commit intoClusterLabs:masterfrom
chrissie-c:fix-pthread-errs
Aug 11, 2021
Merged

Fix pthread returns#444
chrissie-c merged 1 commit intoClusterLabs:masterfrom
chrissie-c:fix-pthread-errs

Conversation

@chrissie-c
Copy link
Copy Markdown
Contributor

pthread calls do not set errno, they return the error directly

pthread calls do not set errno, they return the error directly
Copy link
Copy Markdown
Member

@jfriesse jfriesse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks sensible so ACK

@chrissie-c chrissie-c merged commit a60ca50 into ClusterLabs:master Aug 11, 2021
@chrissie-c
Copy link
Copy Markdown
Contributor Author

Thanks for the review

bmwiedemann pushed a commit to bmwiedemann/openSUSE that referenced this pull request Nov 18, 2021
https://build.opensuse.org/request/show/931807
by user yan_gao + dimstar_suse
- Update to version 2.0.4+20211112.a2691b9 (v2.0.4):
- poll: Don't log in a signal handler (gh#ClusterLabs/libqb##447)
- Fix pthread returns (gh#ClusterLabs/libqb#444)
- doxygen2man: print structure descriptions (gh#ClusterLabs/libqb#443)
- Implement heap based timer list (gh#ClusterLabs/libqb#439) (forwarded request 931806 from yan_gao)
kraj pushed a commit to YoeDistro/meta-openembedded that referenced this pull request Nov 22, 2021
The most important fix in this release is that we no longer log errors inside
the signal handler in loop_poll.c
This could cause an application hang in some circumstances.

Changelog is as follows:
doxygen2man: print structure descriptions
(ClusterLabs/libqb#443)

Fix pthread returns
(ClusterLabs/libqb#444)

poll: Don't log in a signal handler
(ClusterLabs/libqb#447)

Bump library version for v2.0.4

Implement heap based timer list
(ClusterLabs/libqb#439)

build: Fix undefined pthread reference.
(ClusterLabs/libqb#440)

Signed-off-by: Wang Mingyu <wangmy@fujitsu.com>
Signed-off-by: Khem Raj <raj.khem@gmail.com>
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.

2 participants