[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: GN_OP_OnSMS in nk6510
From: |
Bastien Nocera |
Subject: |
Re: GN_OP_OnSMS in nk6510 |
Date: |
Wed, 04 Jul 2007 00:22:30 +0100 |
On Tue, 2007-07-03 at 23:25 +0200, Pawel Kot wrote:
> Hi,
>
> On 7/3/07, Bastien Nocera <address@hidden> wrote:
> > On Tue, 2007-07-03 at 22:56 +0200, Pawel Kot wrote:
> > > Hi,
> > >
> > > On 7/3/07, Bastien Nocera <address@hidden> wrote:
> > > > Program received signal SIGSEGV, Segmentation fault.
> > > > 0x00002aaaaaae4292 in NK6510_GetSMS (data=0x622be8, state=0x622810) at
> > > > nk6510.c:1238
> > > > 1238 data->raw_sms->number =
> > > > data->sms_folder->locations[data->raw_sms->number - 1];
> > >
> > > Could you check what data->raw_sms->number is before that line?
> >
> > Message: SMS Folder status received
> > Message: Number of Entries: 20
> > Message: IDs of Entries : 20, 19, 18, 17, 16, 15, 14, 13, 12, 11, 10, 9, 8,
> > 7, 6, 5, 4, 3, 2, 1,
> >
> > Breakpoint 2, NK6510_GetSMS (data=0x622be8, state=0x622810) at nk6510.c:1236
> > 1236 if (error != GN_ERR_NONE) return error;
> > (gdb) p data->raw_sms->number
> > $1 = 0
>
> Will take a deeper look but here's the sanity check:
And another sanity check, earlier this time. And we also avoid going
past the last memory type.
--
Bastien Nocera <address@hidden>
gnokii-sanity-check-sms-get.patch
Description: Text Data
- GN_OP_OnSMS in nk6510, Bastien Nocera, 2007/07/03
- Re: GN_OP_OnSMS in nk6510, Pawel Kot, 2007/07/03
- Re: GN_OP_OnSMS in nk6510, Bastien Nocera, 2007/07/03
- Re: GN_OP_OnSMS in nk6510, Pawel Kot, 2007/07/03
- Re: GN_OP_OnSMS in nk6510, Bastien Nocera, 2007/07/03
- Re: GN_OP_OnSMS in nk6510, Pawel Kot, 2007/07/03
- Re: GN_OP_OnSMS in nk6510, Bastien Nocera, 2007/07/03
- Re: GN_OP_OnSMS in nk6510, Pawel Kot, 2007/07/03
- Re: GN_OP_OnSMS in nk6510, Bastien Nocera, 2007/07/03
- Re: GN_OP_OnSMS in nk6510, Pawel Kot, 2007/07/03
- Re: GN_OP_OnSMS in nk6510,
Bastien Nocera <=
- Re: GN_OP_OnSMS in nk6510, Pawel Kot, 2007/07/03
- Re: GN_OP_OnSMS in nk6510, Bastien Nocera, 2007/07/03
Re: GN_OP_OnSMS in nk6510, Bastien Nocera, 2007/07/04
Re: GN_OP_OnSMS in nk6510, Pawel Kot, 2007/07/05
Re: GN_OP_OnSMS in nk6510, Bastien Nocera, 2007/07/05
Re: GN_OP_OnSMS in nk6510, Bastien Nocera, 2007/07/13