Skip Menu |

This queue is for tickets about the Authen-SASL-Cyrus CPAN distribution.

Report information
The Basics
Id: 2821
Status: new
Priority: 0/
Queue: Authen-SASL-Cyrus

People
Owner: Nobody in particular
Requestors: pfeifer [...] wait.de
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in: 0.07
Fixed in: (no value)

Attachments
Authen-SASL-Cyrus.patch.gz



Subject: The XS code has some problems, e.g. passwd callbacks cause SIGSEGV
Hello, we wanted to use your module to talk to a Cyrus murder complex. In order to do that, we had to overcome a number of problems. The most time was spend because we overlooked this comment in Cyrus.xs: For SASL_CB_PASS, the context is in the SECOND param We solved the problem with a second PerlCallback function calling the first. While we tried to find out why the callbacks where sometimes called with bad perl contexts, we simplified the memory allocation stuff and replaced alloc/free/strdup calls by the perl API macros recommended in the perl documention. A minor problem was to make the module compile with the perl 5.003 we had on our production systems. There was also a dereferencing error we fixed. As our changes where quite extensive (API macros), I do attach not only the patch to your version but also the complete new version. Please consider our changes for the next version of your module. Cheers, Ulrich Pfeifer ------------------------------------------------------------------- Hello Mark, I fixed other bugs and added some regression tests. [...] Ulrich Pfeifer
Download Authen-SASL-Cyrus.patch.gz
application/gzip 6.2k

Message body not shown because it is not plain text.