Ticket #332 (closed defect: fixed)

Opened 3 years ago

Last modified 3 years ago

SMS wasn't sent properly, after that the channel is "stuck"

Reported by: PaulFertser Owned by: mickey
Priority: major Milestone: milestone5.5
Component: framework/ogsmd Version:
Keywords: Cc:

Description


Attachments

m779551db.txt Download (2.3 KB) - added by PaulFertser 3 years ago.
ogsmd debug log

Change History

Changed 3 years ago by PaulFertser

ogsmd debug log

comment:1 Changed 3 years ago by jluebbe

The problem doesn't seem to be the \r without \n, this works well here and in other methods. Instead the PDU data seems to be missing...

Line 12 and 16 of your log:

2009.01.27 16:51:14 ogsmd.channel DEBUG    <MiscChannel via /dev/pts/2>: sending 11 bytes: 'AT+CMGW=34\r'
2009.01.27 16:51:24 ogsmd.channel DEBUG    <MiscChannel via /dev/pts/2>: TIMEOUT 'AT+CMGW=34' => ???

This is sent by SimStoreMessage? in ogsmd/modems/abstract/mediator.py. You tried to store or send an SMS? It should have sent the PDU right after the \r, the same as with SmsSendMessage?.

The 34 is the length calculated form the PDU string so it is definiately not empty. Could you add debug prints to SimStoreMessage? which print the pdu string and the command string generated from it before it is enqueued?

comment:2 Changed 3 years ago by PaulFertser

  • Summary changed from Command sent to MiscChannel lacks \n, after that the channel is "stuck" to SMS wasn't sent properly, after that the channel is "stuck"

Yes, it makes sense. I sent some sms's that day and one sms didn't appear in the list at all.

Unfortunately, i don't remember the exact contents, so it won't be easy to reproduce. Probably just code analisys can suggest what can go wrong there?

There were no funky unicode chars, just plain ASCII.

comment:3 Changed 3 years ago by mickey

  • Status changed from new to assigned

You're right, there are potential problems in sending of multiline commands. I need to make these more failure-proof after MS5.

comment:4 Changed 3 years ago by stefan

  • Milestone milestone5 deleted

comment:5 Changed 3 years ago by mickey

  • Milestone set to milestone5.5

comment:6 Changed 3 years ago by daniel

Has this occured again with a recent frameworkd?

comment:7 Changed 3 years ago by PaulFertser

No, but i can't say it was easy to reproduce before. ;)

I don't know if Mickey's already fixed "potential problems in sending of multiline commands", so i guess it's up to him to close this ticket once he is ok with it.

comment:8 Changed 3 years ago by daniel

Mickey?

comment:9 Changed 3 years ago by mickey

Will fix for ms5.5

comment:10 Changed 3 years ago by mickey

  • Status changed from assigned to closed
  • Resolution set to fixed

[9c746079d020f483257c64200cd0e18ca8586f13] should improve this. Please reopen with a log, if you ever see this again.

Note: See TracTickets for help on using tickets.