Warning: Can't synchronize with repository "(default)" (No changeset 96d22ec3fa3ef6de3ea8dc0d7d398adc9aa071cf in the repository). Look in the Trac log for more information.

Ticket #53 (closed defect: fixed)

Opened 5 years ago

Last modified 5 years ago

Commas in Names confuse ogsmd

Reported by: daniel Owned by: daniel
Priority: critical Milestone: milestone2
Component: framework/ogsmd Version:
Keywords: Cc:

Description (last modified by daniel) (diff)

When the name of my contact includes a comma - like this +CPBR: 2,"**********",145,"Moeckel,Marek"

then RetrievePhonebook? fails because the split picks up that additional comma

Traceback (most recent call last):
  File "/usr/lib/python2.5/site-packages/framework/subsystems/ogsmd/gsm/channel.py", line 218, in _readyToRead
    self.readyToRead( data )
  File "/usr/lib/python2.5/site-packages/framework/subsystems/ogsmd/gsm/channel.py", line 354, in readyToRead
    self.parser.feed( data, not self.q.empty() )
  File "/usr/lib/python2.5/site-packages/framework/subsystems/ogsmd/gsm/parser.py", line 127, in feed
    nextstate = self.state( b, haveCommand )
  File "/usr/lib/python2.5/site-packages/framework/subsystems/ogsmd/gsm/parser.py", line 172, in state_inline_r
    return self.solicitedLineCompleted()
  File "/usr/lib/python2.5/site-packages/framework/subsystems/ogsmd/gsm/parser.py", line 201, in solicitedLineCompleted
    self.response( self.lines )
  File "/usr/lib/python2.5/site-packages/framework/subsystems/ogsmd/gsm/channel.py", line 443, in _handleResponseToRequest
    self.handleResponseToRequest( request, response )
  File "/usr/lib/python2.5/site-packages/framework/subsystems/ogsmd/gsm/channel.py", line 382, in handleResponseToRequest
    ok_cb( reqstring.strip(), response )
  File "/usr/lib/python2.5/site-packages/framework/subsystems/ogsmd/gsm/decor.py", line 45, in logIt
    result = fn( *args, **kwargs )
  File "/usr/lib/python2.5/site-packages/framework/subsystems/ogsmd/modems/abstract/mediator.py", line 555, in responseFromChannel
    index, number, ntype, name = self._rightHandSide( entry ).split( ',' )
ValueError: too many values to unpack

Change History

comment:1 Changed 5 years ago by daniel

  • Description modified (diff)

comment:2 Changed 5 years ago by daniel

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

Okay, solved the problem with a split( ",", 3 )

This is a safe solution *only* if we can expect the number field to not contain a comma, ever. I believe that is the case, but we should investigate that further.

comment:3 Changed 5 years ago by daniel

Fixed in [c11b95784df0fbf76264c039e57baeaa1b393e00] which also fixes the same problem in RetrieveEntry?.

Note: See TracTickets for help on using tickets.