-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Martin Preuss schrieb:
> Hi,
> 
> On Wednesday 12 July 2006 15:17, Archimedes Submerged wrote:
> [...]
>> -  *rbuf = (GWEN_BUFFER *)0;
> [...]
> 
> I will not include this patch, because in case of an error this variable is 
> supposed to be unchanged. So actually this line should be added to the 
> caller's code...

I think there should be something changed in that file in the lines 1149ff:

  rv=AO_Provider_SendAndReceive(pro, u, msg, strlen(msg), &rbuf);
  if (rv) {
    DBG_ERROR(AQOFXCONNECT_LOGDOMAIN,
              "Error exchanging getStatements-request (%d)", rv);
    GWEN_Buffer_free(rbuf);
    AO_Context_free(ctx);
    return rv;
  }

in other words, the GWEN_Buffer_free(rbuf) is called in any case, even
if rbuf wasn't initialized. Initializing rbuf=0 would make this more
clear by a failed assertion, but is probably not yet sufficient.

Christian
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.2.1 (MingW32)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iQCVAwUBRLT5umXAi+BfhivFAQIeDgQAsr0WLeyWVV4pU2tWyK0hZn+Dqp/3ZkXf
RzPX2btl6Bbh17KICue5RwUl+QrObmZeNjpHd7moVq2Wq9k94pR4wWSa6/UVRRjU
f41Mfq6EkvgCgzQp7DpgNXdbvJx760oj+m7kjDo6x3s62DCFc/lk8kbvq3yQj/V+
HTonKCzS+SU=
=3AAG
-----END PGP SIGNATURE-----


-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
Aqbanking-devel mailing list
Aqbanking-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/aqbanking-devel

Reply via email to