Jörg Pommnitz wrote:
>
> I think the right patch is:
>
> if ((server == NULL) || (emi2_handle_smscreq (conn, *server) < 0)) {
> return; /* reopen the connection */
> }
>
> instead of your commit:
>
> if ((*server !=NULL) && (emi2_handle_smscreq (conn, *server) < 0)) {
> return; /* reopen the connection */
> }
>
> This way we will reopen the connection when the server connection
> is closed.
I don't think the additional if statement had to do something with the
if statement logic. Andreas simply fixed the NULL pointer assertion
checking problem if emi2_handle_smscreq() gets called with server ==
NULL, which obviously throughs the assertion panic.
Heads up if someone has objections here, please!
Stipe
[EMAIL PROTECTED]
-------------------------------------------------------------------
Wapme Systems AG
Münsterstr. 248
40470 Düsseldorf
Tel: +49-211-74845-0
Fax: +49-211-74845-299
E-Mail: [EMAIL PROTECTED]
Internet: http://www.wapme-systems.de
-------------------------------------------------------------------
wapme.net - wherever you are
