mail_open gives an invalid handle to a MAILSTREAM

--===============0974625899767655265==
Content-Type: multipart/alternative; boundary=047d7b6d8432dbc8e004fa68afc8

--047d7b6d8432dbc8e004fa68afc8
Content-Type: text/plain; charset=UTF-8

Hello,

A setup of Asterisk 12 with IMAP-uw 2007e works ok with the default value
of "maxmsg=100" on voicemail.conf

However, if I increase that value to a higher limit (maxmsg=400, for
example), Asterisk crashes when reaching a voicemail count of about 250 to
280 messages.  I have reproduced this issue with different asterisk
versions, see the full Asterisk backtrace and developer comments on
https://issues.asterisk.org/jira/browse/ASTERISK-23759
I saw a similar problem on this email thread:
https://mailman13.u.washington.edu/mailman/htdig/imap-uw/2007-October/001707.html

Looks like the issue is on the double call to mail_open, but the Asterisk
developers say the first calling of mail_open gives back an invalid handle
to a MAILSTREAM. Maybe this is due to the number of messages beign too
large?   So far on all the tests the crash happens when the mailbox has
more than ~250 messages.


The fuction where mail_open is called twice is the following:

########
static int init_mailstream(struct vm_state *vms, int box)
{
        MAILSTREAM *stream = NIL;
        long debug;
        char tmp[256];

        if (!vms) {
                ast_log(LOG_ERROR, "vm_state is NULL!\n");
                return -1;
        }
        ast_debug(3, "vm_state user is:%s\n", vms->imapuser);
        if (vms->mailstream == NIL || !vms->mailstream) {
                ast_debug(1, "mailstream not set.\n");
        } else {
                stream = vms->mailstream;
        }
        /* debug = T;  user wants protocol telemetry? */
        debug = NIL;  /* NO protocol telemetry? */

        if (delimiter == '\0') {                /* did not probe the server
yet */
                char *cp;
#ifdef USE_SYSTEM_IMAP
#include <imap/linkage.c>
#elif defined(USE_SYSTEM_CCLIENT)
#include <c-client/linkage.c>
#else
#include "linkage.c"
#endif
                /* Connect to INBOX first to get folders delimiter */
                imap_mailbox_name(tmp, sizeof(tmp), vms, 0, 1);
                ast_mutex_lock(&vms->lock);
                stream = mail_open (stream, tmp, debug ? OP_DEBUG : NIL);
                ast_mutex_unlock(&vms->lock);
                if (stream == NIL) {
                        ast_log(LOG_ERROR, "Can't connect to imap server
%s\n", tmp);
                        return -1;
                }
                get_mailbox_delimiter(vms, stream);
                /* update delimiter in imapfolder */
                for (cp = vms->imapfolder; *cp; cp++)
                        if (*cp == '/')
                                *cp = delimiter;
        }
        /* Now connect to the target folder */
        imap_mailbox_name(tmp, sizeof(tmp), vms, box, 1);
        ast_debug(3, "Before mail_open, server: %s, box:%d\n", tmp, box);
        ast_mutex_lock(&vms->lock);
        vms->mailstream = mail_open (stream, tmp, debug ? OP_DEBUG : NIL);
        ast_mutex_unlock(&vms->lock);
        if (vms->mailstream == NIL) {
                return -1;
        } else {
                return 0;
        }
}
######################

Any help would be appreciated,

Thank you and best regards,

Alejandro

--047d7b6d8432dbc8e004fa68afc8
Content-Type: text/html; charset=UTF-8
Content-Transfer-Encoding: base64

PGRpdiBkaXI9Imx0ciI+PGRpdj48ZGl2PjxkaXY+PGRpdj5IZWxsbyw8YnI+PC9kaXY+PHA+QSBz
ZXR1cCBvZiBBc3RlcmlzayAxMiB3aXRoIElNQVAtdXcgMjAwN2Ugd29ya3Mgb2sgd2l0aCB0aGUg
ZGVmYXVsdCB2YWx1ZSBvZiAmcXVvdDttYXhtc2c9MTAwJnF1b3Q7IG9uIHZvaWNlbWFpbC5jb25m
PC9wPg0KDQo8cD5Ib3dldmVyLCBpZiBJIGluY3JlYXNlIHRoYXQgdmFsdWUgdG8gYSBoaWdoZXIg
bGltaXQgKG1heG1zZz00MDAsIGZvciANCmV4YW1wbGUpLCBBc3RlcmlzayBjcmFzaGVzIHdoZW4g
cmVhY2hpbmcgYSB2b2ljZW1haWwgY291bnQgb2YgYWJvdXQgMjUwIA0KdG8gMjgwIG1lc3NhZ2Vz
LsKgIEkgaGF2ZSByZXByb2R1Y2VkIHRoaXMgaXNzdWUgd2l0aCBkaWZmZXJlbnQgYXN0ZXJpc2sg
dmVyc2lvbnMsIHNlZSB0aGUgZnVsbCBBc3RlcmlzayBiYWNrdHJhY2UgYW5kIGRldmVsb3BlciBj
b21tZW50cyBvbiA8YSBocmVmPSJodHRwczovL2lzc3Vlcy5hc3Rlcmlzay5vcmcvamlyYS9icm93
c2UvQVNURVJJU0stMjM3NTkiPmh0dHBzOi8vaXNzdWVzLmFzdGVyaXNrLm9yZy9qaXJhL2Jyb3dz
ZS9BU1RFUklTSy0yMzc1OTwvYT48YnI+DQo8L3A+PC9kaXY+PGRpdj5JIHNhdyBhIHNpbWlsYXIg
cHJvYmxlbSBvbiB0aGlzIGVtYWlsIHRocmVhZDrCoCA8YSBocmVmPSJodHRwczovL21haWxtYW4x
My51Lndhc2hpbmd0b24uZWR1L21haWxtYW4vaHRkaWcvaW1hcC11dy8yMDA3LU9jdG9iZXIvMDAx
NzA3Lmh0bWwiPmh0dHBzOi8vbWFpbG1hbjEzLnUud2FzaGluZ3Rvbi5lZHUvbWFpbG1hbi9odGRp
Zy9pbWFwLXV3LzIwMDctT2N0b2Jlci8wMDE3MDcuaHRtbDwvYT48YnI+DQo8YnI+PC9kaXY+PGRp
dj5Mb29rcyBsaWtlIHRoZSBpc3N1ZSBpcyBvbiB0aGUgZG91YmxlIGNhbGwgdG8gbWFpbF9vcGVu
LCBidXQgdGhlIEFzdGVyaXNrIGRldmVsb3BlcnMgc2F5IHRoZSBmaXJzdCBjYWxsaW5nIG9mIG1h
aWxfb3BlbiBnaXZlcyBiYWNrIGFuIGludmFsaWQgaGFuZGxlIHRvIGEgTUFJTFNUUkVBTS4gTWF5
YmUgdGhpcyBpcyBkdWUgdG8gdGhlIG51bWJlciBvZiBtZXNzYWdlcyBiZWlnbiB0b28gbGFyZ2U/
wqDCoCBTbyBmYXIgb24gYWxsIHRoZSB0ZXN0cyB0aGUgY3Jhc2ggaGFwcGVucyB3aGVuIHRoZSBt
YWlsYm94IGhhcyBtb3JlIHRoYW4gfjI1MCBtZXNzYWdlcy48YnI+DQo8L2Rpdj48ZGl2Pjxicj48
YnI+PC9kaXY+VGhlIGZ1Y3Rpb24gd2hlcmUgbWFpbF9vcGVuIGlzIGNhbGxlZCB0d2ljZSBpcyB0
aGUgZm9sbG93aW5nOjxicj48YnI+IyMjIyMjIyM8YnI+c3RhdGljIGludCBpbml0X21haWxzdHJl
YW0oc3RydWN0IHZtX3N0YXRlICp2bXMsIGludCBib3gpPGJyPns8YnI+wqDCoMKgwqDCoMKgwqAg
TUFJTFNUUkVBTSAqc3RyZWFtID0gTklMOzxicj7CoMKgwqDCoMKgwqDCoCBsb25nIGRlYnVnOzxi
cj4NCsKgwqDCoMKgwqDCoMKgIGNoYXIgdG1wWzI1Nl07PGJyPjxicj7CoMKgwqDCoMKgwqDCoCBp
ZiAoIXZtcykgezxicj7CoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqAgYXN0X2xvZyhMT0df
RVJST1IsICZxdW90O3ZtX3N0YXRlIGlzIE5VTEwhXG4mcXVvdDspOzxicj7CoMKgwqDCoMKgwqDC
oMKgwqDCoMKgwqDCoMKgwqAgcmV0dXJuIC0xOzxicj7CoMKgwqDCoMKgwqDCoCB9PGJyPsKgwqDC
oMKgwqDCoMKgIGFzdF9kZWJ1ZygzLCAmcXVvdDt2bV9zdGF0ZSB1c2VyIGlzOiVzXG4mcXVvdDss
IHZtcy0mZ3Q7aW1hcHVzZXIpOzxicj4NCsKgwqDCoMKgwqDCoMKgIGlmICh2bXMtJmd0O21haWxz
dHJlYW0gPT0gTklMIHx8ICF2bXMtJmd0O21haWxzdHJlYW0pIHs8YnI+wqDCoMKgwqDCoMKgwqDC
oMKgwqDCoMKgwqDCoMKgIGFzdF9kZWJ1ZygxLCAmcXVvdDttYWlsc3RyZWFtIG5vdCBzZXQuXG4m
cXVvdDspOzxicj7CoMKgwqDCoMKgwqDCoCB9IGVsc2Ugezxicj7CoMKgwqDCoMKgwqDCoMKgwqDC
oMKgwqDCoMKgwqAgc3RyZWFtID0gdm1zLSZndDttYWlsc3RyZWFtOzxicj7CoMKgwqDCoMKgwqDC
oCB9PGJyPsKgwqDCoMKgwqDCoMKgIC8qIGRlYnVnID0gVDvCoCB1c2VyIHdhbnRzIHByb3RvY29s
IHRlbGVtZXRyeT8gKi88YnI+DQrCoMKgwqDCoMKgwqDCoCBkZWJ1ZyA9IE5JTDvCoCAvKiBOTyBw
cm90b2NvbCB0ZWxlbWV0cnk/ICovPGJyPjxicj7CoMKgwqDCoMKgwqDCoCBpZiAoZGVsaW1pdGVy
ID09ICYjMzk7XDAmIzM5Oykge8KgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoCAvKiBkaWQg
bm90IHByb2JlIHRoZSBzZXJ2ZXIgeWV0ICovPGJyPsKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKg
wqDCoCBjaGFyICpjcDs8YnI+I2lmZGVmIFVTRV9TWVNURU1fSU1BUDxicj4jaW5jbHVkZSAmbHQ7
aW1hcC9saW5rYWdlLmMmZ3Q7PGJyPg0KI2VsaWYgZGVmaW5lZChVU0VfU1lTVEVNX0NDTElFTlQp
PGJyPiNpbmNsdWRlICZsdDtjLWNsaWVudC9saW5rYWdlLmMmZ3Q7PGJyPiNlbHNlPGJyPiNpbmNs
dWRlICZxdW90O2xpbmthZ2UuYyZxdW90Ozxicj4jZW5kaWY8YnI+wqDCoMKgwqDCoMKgwqDCoMKg
wqDCoMKgwqDCoMKgIC8qIENvbm5lY3QgdG8gSU5CT1ggZmlyc3QgdG8gZ2V0IGZvbGRlcnMgZGVs
aW1pdGVyICovPGJyPsKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoCBpbWFwX21haWxib3hf
bmFtZSh0bXAsIHNpemVvZih0bXApLCB2bXMsIDAsIDEpOzxicj4NCsKgwqDCoMKgwqDCoMKgwqDC
oMKgwqDCoMKgwqDCoCBhc3RfbXV0ZXhfbG9jaygmYW1wO3Ztcy0mZ3Q7bG9jayk7PGJyPsKgwqDC
oMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoCBzdHJlYW0gPSBtYWlsX29wZW4gKHN0cmVhbSwgdG1w
LCBkZWJ1ZyA/IE9QX0RFQlVHIDogTklMKTs8YnI+wqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDC
oMKgIGFzdF9tdXRleF91bmxvY2soJmFtcDt2bXMtJmd0O2xvY2spOzxicj7CoMKgwqDCoMKgwqDC
oMKgwqDCoMKgwqDCoMKgwqAgaWYgKHN0cmVhbSA9PSBOSUwpIHs8YnI+DQrCoMKgwqDCoMKgwqDC
oMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgIGFzdF9sb2coTE9HX0VSUk9SLCAmcXVv
dDtDYW4mIzM5O3QgY29ubmVjdCB0byBpbWFwIHNlcnZlciAlc1xuJnF1b3Q7LCB0bXApOzxicj7C
oMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgIHJldHVybiAtMTs8
YnI+wqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgIH08YnI+wqDCoMKgwqDCoMKgwqDCoMKg
wqDCoMKgwqDCoMKgIGdldF9tYWlsYm94X2RlbGltaXRlcih2bXMsIHN0cmVhbSk7PGJyPsKgwqDC
oMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoCAvKiB1cGRhdGUgZGVsaW1pdGVyIGluIGltYXBmb2xk
ZXIgKi88YnI+DQrCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqAgZm9yIChjcCA9IHZtcy0m
Z3Q7aW1hcGZvbGRlcjsgKmNwOyBjcCsrKTxicj7CoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKg
wqDCoMKgwqDCoMKgwqDCoMKgIGlmICgqY3AgPT0gJiMzOTsvJiMzOTspPGJyPsKgwqDCoMKgwqDC
oMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgICpjcCA9
IGRlbGltaXRlcjs8YnI+wqDCoMKgwqDCoMKgwqAgfTxicj7CoMKgwqDCoMKgwqDCoCAvKiBOb3cg
Y29ubmVjdCB0byB0aGUgdGFyZ2V0IGZvbGRlciAqLzxicj4NCsKgwqDCoMKgwqDCoMKgIGltYXBf
bWFpbGJveF9uYW1lKHRtcCwgc2l6ZW9mKHRtcCksIHZtcywgYm94LCAxKTs8YnI+wqDCoMKgwqDC
oMKgwqAgYXN0X2RlYnVnKDMsICZxdW90O0JlZm9yZSBtYWlsX29wZW4sIHNlcnZlcjogJXMsIGJv
eDolZFxuJnF1b3Q7LCB0bXAsIGJveCk7PGJyPsKgwqDCoMKgwqDCoMKgIGFzdF9tdXRleF9sb2Nr
KCZhbXA7dm1zLSZndDtsb2NrKTs8YnI+wqDCoMKgwqDCoMKgwqAgdm1zLSZndDttYWlsc3RyZWFt
ID0gbWFpbF9vcGVuIChzdHJlYW0sIHRtcCwgZGVidWcgPyBPUF9ERUJVRyA6IE5JTCk7PGJyPg0K
wqDCoMKgwqDCoMKgwqAgYXN0X211dGV4X3VubG9jaygmYW1wO3Ztcy0mZ3Q7bG9jayk7PGJyPsKg
wqDCoMKgwqDCoMKgIGlmICh2bXMtJmd0O21haWxzdHJlYW0gPT0gTklMKSB7PGJyPsKgwqDCoMKg
wqDCoMKgwqDCoMKgwqDCoMKgwqDCoCByZXR1cm4gLTE7PGJyPsKgwqDCoMKgwqDCoMKgIH0gZWxz
ZSB7PGJyPsKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoCByZXR1cm4gMDs8YnI+wqDCoMKg
wqDCoMKgwqAgfTxicj59PGJyPiMjIyMjIyMjIyMjIyMjIyMjIyMjIyM8YnI+PGJyPjwvZGl2Pg0K
PGRpdj5BbnkgaGVscCB3b3VsZCBiZSBhcHByZWNpYXRlZCw8YnI+PC9kaXY+PGRpdj48YnI+PC9k
aXY+VGhhbmsgeW91IGFuZCBiZXN0IHJlZ2FyZHMsPGJyPjxicj48L2Rpdj5BbGVqYW5kcm88YnI+
PC9kaXY+DQo=
--047d7b6d8432dbc8e004fa68afc8--

--===============0974625899767655265==
Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

_______________________________________________
Imap-uw mailing list
[email protected]
http://mailman13.u.washington.edu/mailman/listinfo/imap-uw

--===============0974625899767655265==--