Ticket #533 (assigned defect)

Opened 1 year ago

Last modified 2 months ago

bug reading phone book entries where the entry's name contains a \n

Reported by: cstender Assigned to: andrewb (accepted)
Priority: normal Milestone: 0.41
Component: Plugin: moto Version: 0.32
Severity: normal Keywords:
Cc: bg@suse.de

Description (Last modified by felixmoeller)

After bug #531 seems to be fixed I tried to sync the V3i again and now I run into another bug. It seems that moto get all changes but isn't able to commit them to the engine. 

[1186561183.788550]                                             [SENSITIVE] <-- OK
[1186561183.792043]                                             >>>>>>>  osync_context_report_osyncerror(0x8149980, 0x8146ee0:(Traceback (most recent call last):
  File "/usr/lib/opensync/python-plugins/motosync.py", line 2272, in get_changes
    for change in self.access.list_changes(self.objtype):
  File "/usr/lib/opensync/python-plugins/motosync.py", line 2101, in list_changes
    entries = [PhoneContactMoto(d) for d in self.comms.read_contacts()]
  File "/usr/lib/opensync/python-plugins/motosync.py", line 1016, in read_contacts
    ret.extend(self.__parse_results('MPBR', data))
  File "/usr/lib/opensync/python-plugins/motosync.py", line 1117, in __parse_results
    assert(part[-1] == '"')
AssertionError
))

Change History

08/08/07 11:20:07 changed by andrewb

  • status changed from new to assigned.

The problem here is the contact in slot 17: 

+MPBR: 17,"meinrad.ege@web.de
meinrad.ege@ids.allianz.com",128,0045006700650020004D00650069006E007200610064,6,0,255,0,0,1,255,1,4,,0,0,,,,,,,,"",""

You have two email addresses separated by a newline, which screws up my parsing of the returned lines from the phone. moto-sync shouldn't fail in this situation, and I'll leave this ticket open until I fix the parsing code to handle newlines in strings, but in the meantime you can work around this problem by cleaning up that contact (it should be two separate contacts).

08/08/07 14:07:49 changed by cstender

Okay, this seems to work. Thanks. I removed the trace-logs, if you need them for debugging, please drop a note.

10/16/07 00:20:14 changed by dgollub

  • component changed from OpenSync to Plugin: moto.

11/04/07 15:28:15 changed by andrewb

  • summary changed from moto-plugin can't commit changes to bug reading phone book entries where the entry's name contains a \n.

07/30/08 18:48:19 changed by tuju

  • milestone changed from 0.40 to 0.41.

08/09/08 21:27:45 changed by felixmoeller

  • description changed.