Mailing-List: contact c-dev-help@axis.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Apache AXIS C Developers List" <c-dev@axis.apache.org>
Received-SPF: pass (nike.apache.org: domain of alex.mantaut@intraway.com
 designates 74.125.82.173 as permitted sender)
MIME-Version: 1.0
Date: Mon, 30 Jan 2012 11:50:36 -0300
Message-ID: 
 <CA+wVFzvo_AWy6bx-KsP3eKjmmy3k08dE5dm4p+Vb4HB6+b3Khg@mail.gmail.com>
Subject: Asynchronous call fixes
From: Alex Mantaut <alex.mantaut@intraway.com>
To: c-dev@axis.apache.org
Content-Type: multipart/mixed; boundary=0016e6de01671d304404b7bff808

--0016e6de01671d304404b7bff808
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable

Dear Axis2/C devs,

I've started developing an application using axis2/C asynchronous
calls and I found several bugs...=A0 After a lot of diving trough the
axis code I think I found some of the problems, and implemented simple
fixes for them...

I would like to send those patches to you to see if the bug fixes are
correct or is there a better solution for those issues? I've runned
the project tests and did some memory leak check and the patch fixes
the memory leaks from asynchronous calls...

Also it seems that axis2/c crashes in several places while creating
and using multiple svc_clients, mostly due the saturation of file
descriptors, and because in some cases the use of files isn't
validated... I've tried to find a solution to that as well but there
were many places that needed to be modified, I found some but I need
to cleanup my code before sending a patch...

If the bug fixes are correct I would love to create a jira ticket in
order to incorporate the code to trunk...

Thank you all for your great work,
Best regards


PS: I'm sorry if this mail arrives to you twice, but I keep getting an
"Undelivered Mail Returned to Sender"
--
--
Mantaut Alex
Intraway Corp.

+54 (11) 6040-4000
MSN: alex.mantaut@intraway.com

Visit our website at=A0http://www.intraway.com
Proud to be an ISO 9001:2008 certified company

--0016e6de01671d304404b7bff808
Content-Type: text/x-patch; charset=US-ASCII; name="axis_async.patch"
Content-Disposition: attachment; filename="axis_async.patch"
Content-Transfer-Encoding: base64
X-Attachment-Id: f_gy1m5o8v0

SW5kZXg6IHNyYy9jb3JlL2NsaWVudGFwaS9zdmNfY2xpZW50LmMKPT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gc3Jj
L2NvcmUvY2xpZW50YXBpL3N2Y19jbGllbnQuYwkocmV2aXNpb24gMTIzNjc3NikKKysrIHNyYy9j
b3JlL2NsaWVudGFwaS9zdmNfY2xpZW50LmMJKHdvcmtpbmcgY29weSkKQEAgLTgwNSw2ICs4MDUs
NyBAQAogICAgIGNvbnN0IGF4aW9tX25vZGVfdCAqIHBheWxvYWQsCiAgICAgYXhpczJfY2FsbGJh
Y2tfdCAqIGNhbGxiYWNrKQogeworICAgIGF4aXMyX2NhbGxiYWNrX2luY3JlbWVudF9yZWYoY2Fs
bGJhY2ssZW52KTsKICAgICBheGlzMl9tc2dfY3R4X3QgKm1zZ19jdHggPSBOVUxMOwogICAgIEFY
SVMyX1RSQU5TUE9SVF9FTlVNUyB0cmFuc3BvcnRfaW5fcHJvdG9jb2w7CiAgICAgYXhpczJfYm9v
bF90IHFuYW1lX2ZyZWVfZmxhZyA9IEFYSVMyX0ZBTFNFOwpAQCAtODQyLDYgKzg0Myw3IEBACiAg
ICAgICAgIHJldHVybjsKICAgICB9CiAKKwogICAgIGF4aXMyX29wX2NsaWVudF9zZXRfY2FsbGJh
Y2soc3ZjX2NsaWVudC0+b3BfY2xpZW50LCBlbnYsIGNhbGxiYWNrKTsKICAgICBheGlzMl9vcF9j
bGllbnRfYWRkX291dF9tc2dfY3R4KHN2Y19jbGllbnQtPm9wX2NsaWVudCwgZW52LCBtc2dfY3R4
KTsKIApAQCAtODcwLDcgKzg3Miw5IEBACiAgICAgICAgIGF4aXMyX29wX2NsaWVudF9zZXRfY2Fs
bGJhY2tfcmVjdihzdmNfY2xpZW50LT5vcF9jbGllbnQsIGVudiwgc3ZjX2NsaWVudC0+Y2FsbGJh
Y2tfcmVjdik7CiAgICAgfQogCi0gICAgYXhpczJfb3BfY2xpZW50X2V4ZWN1dGUoc3ZjX2NsaWVu
dC0+b3BfY2xpZW50LCBlbnYsIEFYSVMyX0ZBTFNFKTsKKyAgICBpZihheGlzMl9vcF9jbGllbnRf
ZXhlY3V0ZShzdmNfY2xpZW50LT5vcF9jbGllbnQsIGVudiwgQVhJUzJfRkFMU0UpICE9IEFYSVMy
X1NVQ0NFU1MpCisgICAgCXJldHVybjsKKwogICAgIGF4aXMyX3N2Y19jbGllbnRfc2V0X2h0dHBf
aW5mbyhzdmNfY2xpZW50LCBlbnYsIG1zZ19jdHgpOwogICAgIHN2Y19jbGllbnQtPmF1dGhfZmFp
bGVkID0gYXhpczJfbXNnX2N0eF9nZXRfYXV0aF9mYWlsZWQobXNnX2N0eCwgZW52KTsKICAgICBz
dmNfY2xpZW50LT5yZXF1aXJlZF9hdXRoX2lzX2h0dHAgPSBheGlzMl9tc2dfY3R4X2dldF9yZXF1
aXJlZF9hdXRoX2lzX2h0dHAobXNnX2N0eCwgZW52KTsKQEAgLTg4NCw2ICs4ODgsNyBAQAogICAg
ICAgICBheHV0aWxfcW5hbWVfZnJlZSgoYXh1dGlsX3FuYW1lX3QgKilvcF9xbmFtZSwgZW52KTsK
ICAgICAgICAgb3BfcW5hbWUgPSBOVUxMOwogICAgIH0KKyAgICBheGlzMl9jYWxsYmFja19mcmVl
KGNhbGxiYWNrLCBlbnYpOwogfQogCiBBWElTMl9FWFRFUk4gdm9pZCBBWElTMl9DQUxMCkluZGV4
OiBzcmMvY29yZS9jbGllbnRhcGkvb3BfY2xpZW50LmMKPT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gc3JjL2NvcmUv
Y2xpZW50YXBpL29wX2NsaWVudC5jCShyZXZpc2lvbiAxMjM2Nzc2KQorKysgc3JjL2NvcmUvY2xp
ZW50YXBpL29wX2NsaWVudC5jCSh3b3JraW5nIGNvcHkpCkBAIC00Myw3ICs0Myw3IEBACiAgICAg
YXhpczJfYm9vbF90IGNvbXBsZXRlZDsKIAogICAgIC8qIHRvIGhvbGQgdGhlIGxvY2FsbHkgY3Jl
YXRlZCBhc3luYyByZXN1bHQgKi8KLSAgICBheGlzMl9hc3luY19yZXN1bHRfdCAqYXN5bmNfcmVz
dWx0OworICAgIC8qYXhpczJfYXN5bmNfcmVzdWx0X3QgKmFzeW5jX3Jlc3VsdDsqLwogCiAgICAg
YXhpczJfY2FsbGJhY2tfcmVjdl90ICpjYWxsYmFja19yZWN2OwogCkBAIC0xMDEsNyArMTAxLDcg
QEAKICAgICBvcF9jbGllbnQtPmNhbGxiYWNrID0gTlVMTDsKICAgICBvcF9jbGllbnQtPmNvbXBs
ZXRlZCA9IEFYSVMyX0ZBTFNFOwogICAgIG9wX2NsaWVudC0+cmV1c2UgPSBBWElTMl9GQUxTRTsK
LSAgICBvcF9jbGllbnQtPmFzeW5jX3Jlc3VsdCA9IE5VTEw7CisgICAgLypvcF9jbGllbnQtPmFz
eW5jX3Jlc3VsdCA9IE5VTEw7Ki8KICAgICBvcF9jbGllbnQtPmNhbGxiYWNrX3JlY3YgPSBOVUxM
OwogCiAgICAgb3BfY2xpZW50LT5vcHRpb25zID0gb3B0aW9uczsKQEAgLTUwMSw3ICs1MDEsMTAg
QEAKICAgICAgICAgICAgICAgICAiT3AgY2xpZW50IGV4ZWN1dGUgZmFpbGVkIGR1ZSB0byBlbmdp
bmUgY3JlYXRpb24gZmFpbHVyZS4iKTsKICAgICAgICAgICAgIHJldHVybiBBWElTMl9GQUlMVVJF
OwogICAgICAgICB9Ci0gICAgICAgIGF4aXMyX2VuZ2luZV9zZW5kKGVuZ2luZSwgZW52LCBtc2df
Y3R4KTsKKworICAgICAgICBpZihheGlzMl9lbmdpbmVfc2VuZChlbmdpbmUsIGVudiwgbXNnX2N0
eCkgIT0gQVhJUzJfU1VDQ0VTUykKKyAgICAgICAgCXJldHVybiBBWElTMl9GQUlMVVJFOworCiAg
ICAgICAgIGF4aXMyX2VuZ2luZV9mcmVlKGVuZ2luZSwgZW52KTsKICAgICB9CiAgICAgZWxzZSAv
KiBTYW1lIGNoYW5uZWwgd2lsbCBiZSB1c2VkIGlycmVzcGVjdGl2ZSBvZiBtZXNzYWdlIGV4Y2hh
bmdlIHBhdHRlcm4uICovCkBAIC02NDEsMTAgKzY0NCwxMCBAQAogICAgIGlmKCFvcF9jbGllbnQp
CiAgICAgICAgIHJldHVybjsKIAotICAgIC8qaWYob3BfY2xpZW50LT5jYWxsYmFjaykKKyAgICBp
ZihvcF9jbGllbnQtPmNhbGxiYWNrKQogICAgIHsKICAgICAgICAgYXhpczJfY2FsbGJhY2tfZnJl
ZShvcF9jbGllbnQtPmNhbGxiYWNrLCBlbnYpOwotICAgIH0qLworICAgIH0KIAogICAgIGlmKG9w
X2NsaWVudC0+b3BfY3R4KQogICAgIHsKQEAgLTY4OCw4ICs2OTEsOCBAQAogICAgIHsKICAgICAg
ICAgcmV0dXJuIE5VTEw7CiAgICAgfQotCiAgICAgdGhfZW52ID0gYXh1dGlsX2luaXRfdGhyZWFk
X2VudihhcmdzX2xpc3QtPmVudik7CisgICAgYXhpczJfY2FsbGJhY2tfaW5jcmVtZW50X3JlZihh
cmdzX2xpc3QtPmNhbGxiYWNrLHRoX2Vudik7CiAKICAgICBvcF9jdHggPSBheGlzMl9vcF9jdHhf
Y3JlYXRlKHRoX2VudiwgYXJnc19saXN0LT5vcCwgYXJnc19saXN0LT5vcF9jbGllbnQtPnN2Y19j
dHgpOwogICAgIGlmKCFvcF9jdHgpCkBAIC03MDYsMjUgKzcwOSwzOCBAQAogICAgICAqIGluIHRo
ZSBzaW5nbGUgY2hhbm5lbCBub24gYmxvY2tpbmcgY2FzZSB3aGljaCwgaW1wbHkgdGhpcyBpcyB0
d28gd2F5IG1lc3NhZ2UgYnkgZGVzaWduLgogICAgICAqLwogCi0gICAgLyogSGVyZSBhZnRlciB0
aGUgY29kZSBpcyBhIHN1YnNldCBvZiB3aGF0IGNhbGxiYWNrIHJlY2VpdmVyIGRvIGluIGR1YWwg
Y2hhbm5lbCBjYXNlLiovCi0gICAgYXhpczJfb3BfY2xpZW50X2FkZF9tc2dfY3R4KGFyZ3NfbGlz
dC0+b3BfY2xpZW50LCB0aF9lbnYsIHJlc3BvbnNlKTsKLSAgICBhcmdzX2xpc3QtPm9wX2NsaWVu
dC0+YXN5bmNfcmVzdWx0ID0gYXhpczJfYXN5bmNfcmVzdWx0X2NyZWF0ZSh0aF9lbnYsIHJlc3Bv
bnNlKTsKLQotICAgIGlmKGFyZ3NfbGlzdC0+Y2FsbGJhY2spCisgICAgaWYocmVzcG9uc2UpCiAg
ICAgewotICAgICAgICBheGlzMl9jYWxsYmFja19pbnZva2Vfb25fY29tcGxldGUoYXJnc19saXN0
LT5jYWxsYmFjaywgdGhfZW52LAotICAgICAgICAgICAgYXJnc19saXN0LT5vcF9jbGllbnQtPmFz
eW5jX3Jlc3VsdCk7CisJCS8qIEhlcmUgYWZ0ZXIgdGhlIGNvZGUgaXMgYSBzdWJzZXQgb2Ygd2hh
dCBjYWxsYmFjayByZWNlaXZlciBkbyBpbiBkdWFsIGNoYW5uZWwgY2FzZS4qLworCQlheGlzMl9v
cF9jbGllbnRfYWRkX21zZ19jdHgoYXJnc19saXN0LT5vcF9jbGllbnQsIHRoX2VudiwgcmVzcG9u
c2UpOwogCi0gICAgICAgIGF4aXMyX2NhbGxiYWNrX3NldF9jb21wbGV0ZShhcmdzX2xpc3QtPmNh
bGxiYWNrLCB0aF9lbnYsIEFYSVMyX1RSVUUpOworCQlheGlzMl9hc3luY19yZXN1bHRfdCAqIGFz
eW5jX3Jlc3VsdCA9IGF4aXMyX2FzeW5jX3Jlc3VsdF9jcmVhdGUodGhfZW52LCByZXNwb25zZSk7
CisKKwkJaWYoYXJnc19saXN0LT5jYWxsYmFjaykKKwkJeworCQkJYXhpczJfY2FsbGJhY2tfaW52
b2tlX29uX2NvbXBsZXRlKGFyZ3NfbGlzdC0+Y2FsbGJhY2ssIHRoX2VudiwKKwkJCQlhc3luY19y
ZXN1bHQpOworCisJCQlheGlzMl9jYWxsYmFja19zZXRfY29tcGxldGUoYXJnc19saXN0LT5jYWxs
YmFjaywgdGhfZW52LCBBWElTMl9UUlVFKTsKKwkJfQorCSAgICAvKiBDbGVhbiB1cCBtZW1vcnkg
Ki8KKwkgICAgYXhpczJfYXN5bmNfcmVzdWx0X2ZyZWUoYXN5bmNfcmVzdWx0LCB0aF9lbnYpOwor
CiAgICAgfQorICAgIGVsc2UKKyAgICB7CisgICAgCWlmKGFyZ3NfbGlzdC0+Y2FsbGJhY2spCisg
ICAgCXsKKyAgICAJCS8qVE9ETyBzZWVrIGV4Y2VwdGlvbiBjb2RlKi8KKyAgICAJCWF4aXMyX2Nh
bGxiYWNrX3JlcG9ydF9lcnJvcihhcmdzX2xpc3QtPmNhbGxiYWNrLCB0aF9lbnYsMSk7CisgICAg
CX0KKyAgICB9CiAKLSAgICAvKiBDbGVhbiB1cCBtZW1vcnkgKi8KLSAgICBheGlzMl9hc3luY19y
ZXN1bHRfZnJlZShhcmdzX2xpc3QtPm9wX2NsaWVudC0+YXN5bmNfcmVzdWx0LCB0aF9lbnYpOwot
CiAgICAgYXhpczJfb3BfY3R4X2ZyZWUob3BfY3R4LCB0aF9lbnYpOwogCiAgICAgdGhfcG9vbCA9
IHRoX2Vudi0+dGhyZWFkX3Bvb2w7CiAKKyAgICBheGlzMl9jYWxsYmFja19mcmVlKGFyZ3NfbGlz
dC0+Y2FsbGJhY2ssIHRoX2Vudik7CiAgICAgQVhJUzJfRlJFRSh0aF9lbnYtPmFsbG9jYXRvciwg
YXJnc19saXN0KTsKIAogICAgIGlmKHRoX2VudikKQEAgLTE0NTAsNCArMTQ2Niw0IEBACiAgICAg
YXhpczJfYm9vbF90IHJldXNlKQogewogICAgIG9wX2NsaWVudC0+cmV1c2UgPSByZXVzZTsKLX0K
XCBObyBuZXdsaW5lIGF0IGVuZCBvZiBmaWxlCit9CkluZGV4OiBzcmMvY29yZS9jbGllbnRhcGkv
Y2FsbGJhY2suYwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09Ci0tLSBzcmMvY29yZS9jbGllbnRhcGkvY2FsbGJhY2suYwko
cmV2aXNpb24gMTIzNjc3NikKKysrIHNyYy9jb3JlL2NsaWVudGFwaS9jYWxsYmFjay5jCSh3b3Jr
aW5nIGNvcHkpCkBAIC0yMiw2ICsyMiw4IEBACiBzdHJ1Y3QgYXhpczJfY2FsbGJhY2sKIHsKIAor
ICAgIGludCByZWY7CisKICAgICAvKiogY2FsbGJhY2sgY29tcGxldGU/ICovCiAgICAgYXhpczJf
Ym9vbF90IGNvbXBsZXRlOwogCkBAIC0xMDUsNiArMTA3LDcgQEAKICAgICBjYWxsYmFjay0+b25f
ZXJyb3IgPSBheGlzMl9jYWxsYmFja19vbl9lcnJvcjsKIAogICAgIGNhbGxiYWNrLT5tdXRleCA9
IGF4dXRpbF90aHJlYWRfbXV0ZXhfY3JlYXRlKGVudi0+YWxsb2NhdG9yLCBBWElTMl9USFJFQURf
TVVURVhfREVGQVVMVCk7CisgICAgY2FsbGJhY2stPnJlZiA9IDE7CiAgICAgcmV0dXJuIGNhbGxi
YWNrOwogfQogCkBAIC0yMDgsMTEgKzIxMSw0MCBAQAogICAgIHJldHVybiBBWElTMl9TVUNDRVNT
OwogfQogCitBWElTMl9FWFRFUk4gYXhpczJfc3RhdHVzX3QgQVhJUzJfQ0FMTAorYXhpczJfY2Fs
bGJhY2tfaW5jcmVtZW50X3JlZigKKwlheGlzMl9jYWxsYmFja190ICogY2FsbGJhY2ssCisgICAg
Y29uc3QgYXh1dGlsX2Vudl90ICogZW52KQoreworCWF4dXRpbF90aHJlYWRfbXV0ZXhfbG9jayhj
YWxsYmFjay0+bXV0ZXgpOworCWNhbGxiYWNrLT5yZWYrKzsKKwlheHV0aWxfdGhyZWFkX211dGV4
X3VubG9jayhjYWxsYmFjay0+bXV0ZXgpOworCisgICAgcmV0dXJuIEFYSVMyX1NVQ0NFU1M7Cit9
CisKIEFYSVMyX0VYVEVSTiB2b2lkIEFYSVMyX0NBTEwKIGF4aXMyX2NhbGxiYWNrX2ZyZWUoCiAg
ICAgYXhpczJfY2FsbGJhY2tfdCAqIGNhbGxiYWNrLAogICAgIGNvbnN0IGF4dXRpbF9lbnZfdCAq
IGVudikKIHsKKwlheHV0aWxfdGhyZWFkX211dGV4X2xvY2soY2FsbGJhY2stPm11dGV4KTsKKwlj
YWxsYmFjay0+cmVmLS07CisKKworICAgIGlmKGNhbGxiYWNrLT5yZWYgPiAwKQorICAgIHsKKyAg
ICAJYXh1dGlsX3RocmVhZF9tdXRleF91bmxvY2soY2FsbGJhY2stPm11dGV4KTsKKyAgICAgICAg
cmV0dXJuOworICAgIH0KKworCWF4dXRpbF90aHJlYWRfbXV0ZXhfdW5sb2NrKGNhbGxiYWNrLT5t
dXRleCk7CisKKyAgICBpZihjYWxsYmFjay0+bXNnX2N0eCkKKyAgICB7CisgICAgCWF4aXMyX21z
Z19jdHhfZnJlZShjYWxsYmFjay0+bXNnX2N0eCwgZW52KTsKKyAgICB9CisKICAgICBpZihjYWxs
YmFjay0+bXV0ZXgpCiAgICAgewogICAgICAgICBheHV0aWxfdGhyZWFkX211dGV4X2Rlc3Ryb3ko
Y2FsbGJhY2stPm11dGV4KTsK

--0016e6de01671d304404b7bff808
Content-Type: text/plain; charset=us-ascii


---------------------------------------------------------------------
To unsubscribe, e-mail: c-dev-unsubscribe@axis.apache.org
For additional commands, e-mail: c-dev-help@axis.apache.org
--0016e6de01671d304404b7bff808--