-
Notifications
You must be signed in to change notification settings - Fork 146
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
Comments
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 |
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!) |
Moved to cyrusimap/cyrus-sasl#584 |
No description provided.
The text was updated successfully, but these errors were encountered: