Re: SMPP implementation in current CVS build

Alex Judd wrote:
> 
> Illogical and strange but thanks for looking at it (and the fix).
> 
> We don't check for host as a default config value either so you get an
> octstr error if you don't specify it. Quick diff fix for that.
> 
> # cvs diff -u gw/smsc_smpp.c
> Index: gw/smsc_smpp.c
> ===================================================================
> RCS file: /home/cvs/gateway/gw/smsc_smpp.c,v
> retrieving revision 1.50
> diff -u -r1.50 smsc_smpp.c
> --- gw/smsc_smpp.c      2002/01/28 15:12:13     1.50
> +++ gw/smsc_smpp.c      2002/01/29 10:27:57
> @@ -829,6 +827,10 @@
> 
>      /* Check that config is OK */
>      ok = 1;
> +    if (host == NULL) {
> +        error(0,"SMPP: Configuration file doesn't specify host");
> +        ok = 0;
> +    }
>      if (username == NULL) {
>         error(0, "SMPP: Configuration file doesn't specify username.");
>         ok = 0;
> #

patch applied. Thanks.

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