Bug report: NRE on Bind when SystemType is not set up

Smpp v3.4 client

Moderator: alt

Locked
Jabberwok
Posts: 6
Joined: Tue Aug 04, 2009 11:56 am

Bug report: NRE on Bind when SystemType is not set up

Post by Jabberwok » Tue Aug 11, 2009 10:02 am

Hi there!

Now i have sources and can point exact lines in files where there are errors :)

Steps to reproduce:

Code: Select all

smppClient.Connect(config.HostName, config.Port);
pduBindResp bindResp = smppClient.Bind(config.SystemId, config.Password);
and SystemType is not set up.

in bindResp there will be CommandStatus.SMPPCLIENT_UNKNOWNERROR
and there will be OnError.

With sources it is obvious:
in class pduBind when you try to set SystemType to null it fails with NRE, since you try to check it's length.

i suggest to set SystemType of SmppClient to string.Empty not null, or throw more explaining exception.
alt
Site Admin
Posts: 988
Joined: Tue Apr 25, 2006 9:45 am

Post by alt » Thu Aug 13, 2009 9:35 pm

Thank you again, Jabberwok.
Fixed.

Everybody can download new build.

Who has source code license, please write me an email. I'll send update.
Don't forget to include purchasing information.
Locked