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

Make init_mechlist thread safe #2813

Closed
vlmarek opened this issue Jun 26, 2019 · 4 comments
Closed

Make init_mechlist thread safe #2813

vlmarek opened this issue Jun 26, 2019 · 4 comments
Assignees

Comments

@vlmarek
Copy link

vlmarek commented Jun 26, 2019

No description provided.

@vlmarek
Copy link
Author

vlmarek commented Jun 26, 2019

Hi,

We are doing some housekeeping in the changes we use in cyrus-sasl in Solaris.

https://github.com/oracle/solaris-userland/tree/master/components/cyrus-sasl/patches

And I have noticed that we have in the pipeline a fix which never got to be used. It seems simple enough, but I can't judge if this is something you would consider having. If yes, I'm happy to try to port it to current version and create a pull request. If no I will just drop it because we don't need it internally anymore.

That said I don't even see lib/client.c in your code so probably this is pointless ...

Thank you
__
Vlad

@vlmarek
Copy link
Author

vlmarek commented Jun 26, 2019

116-race-condition.txt

@elliefm elliefm self-assigned this Aug 19, 2019
@elliefm
Copy link
Contributor

elliefm commented Aug 19, 2019

Hi, thanks for the patch. The right place for this is the cyrus-sasl repository, https://github.com/cyrusimap/cyrus-sasl -- do you want to re-open this issue there? If not, I will eventually, but if you do it yourself then github will give you the notifications/etc.

(I don't know sasl well enough to comment on the patch myself, but if it's in the right place it can attract the right eyes!)

@elliefm
Copy link
Contributor

elliefm commented Nov 15, 2019

Moved to cyrusimap/cyrus-sasl#584

@elliefm elliefm closed this as completed Nov 15, 2019
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

No branches or pull requests

2 participants