Re: smsd crash report

[Chuck]

Hi,

I poked around in my core dumps with gdb and believe that I have isolated the 
problem.  Below is the code immediately before my crash in nk6510.c:
                if (!data->message_center) {
                        dprintf("SMSC received but not expected\n");
                        e = GN_ERR_INTERNALERROR;
                }

All previous error conditions have a break statement after setting e to the 
error code, but this one does not.  I believe it should look like this:
                if (!data->message_center) {
                        dprintf("SMSC received but not expected\n");
                        e = GN_ERR_INTERNALERROR;
                        break;
                }

chuck

On Mon, May 23, 2011 at 11:18:47AM -0700, Chuck wrote:
> Hi,
> 
> I sent you the full log in a private email after I confirmed that both 0.6.27 
> and 0.6.25 crashed with a similar message to recent versions.
> 
> Thanks,
> chuck
> 
> On Thu, May 19, 2011 at 07:54:55AM +0200, Pawel Kot wrote:
> > Hi,
> > 
> > On Wed, May 18, 2011 at 20:31, Chuck <address@hidden> wrote:
> > > I have a new rebuilt 0.6.29 smsd that is statically linked, so this 
> > > should be more informative and the correct program version. ?By the way, 
> > > with 0.6.29 I get "gn_sms_get returned error 16: The given memory 
> > > location is invalid." before each "Sending SMS:" line, except for the 
> > > first message sent.
> > 
> > Thanks for that. Looks very weird to me but at least I didn't break it
> > recently :) I will have a deeper look at it over weekend or next week.
> > Meanwhile, would it be able for you to check also few more earlier
> > versions (2 or 3) to see whether they crash the same way? If it was
> > working some time ago it will be easier to fix. I'd appreciate also if
> > you could send the full log file (can be in the priv email).
> > 
> > thanks,
> > -- 
> > Pawel Kot