Windows: renaming a files generates a KeyError in filesystem_manager

Bug #810781 reported by Natalia Bidart
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ubuntu One Client
Fix Released
High
Manuel de la Peña

Bug Description

While testing this branch lp:~mandel/ubuntuone-client/pyinotify-non-unicode/, I tried to move a file named

~/Ubuntu One/logs.7z

to

~/Ubuntu One/logs-2011-07-12.7z

and syncdaemon crashed internally. See traceback in the next comment.

Revision history for this message
Natalia Bidart (nataliabidart) wrote :

Full trace is http://pastebin.ubuntu.com/644409/

Specific error is:

2011-07-14 22:20:07,326 - ubuntuone.SyncDaemon.EQ - DEBUG - push_event: FS_FILE_
MOVE, kwargs: {'path_from': '\\\\?\\C:\\Users\\Test\\Ubuntu One\\log.7z', 'path_
to': '\\\\?\\C:\\Users\\Test\\Ubuntu One\\log-2011-07-12.7z'}
2011-07-14 22:20:07,329 - ubuntuone.SyncDaemon.sync - ERROR - unhandled exceptio
n
Traceback (most recent call last):
  File "C:\Users\Test\src\unicodes\ubuntuone\syncdaemon\sync.py", line 305, in o
n_event
    **kwargs)
  File "C:\Users\Test\src\unicodes\ubuntuone\syncdaemon\fsm\fsm.py", line 110, i
n on_event
    event_name, parameters, args))
  File "C:\Users\Test\src\unicodes\ubuntuone\syncdaemon\sync.py", line 223, in l
evel_log
    self.log(lvl, message, *args, **kwargs)
  File "C:\Users\Test\src\unicodes\ubuntuone\syncdaemon\sync.py", line 253, in l
og
    "relpath of: %r", self.key['path'])
  File "C:\Users\Test\src\unicodes\ubuntuone\syncdaemon\sync.py", line 98, in __
getitem__
    return self.get(key)
  File "C:\Users\Test\src\unicodes\ubuntuone\syncdaemon\sync.py", line 77, in ge
t
    mdid = self.get_mdid()
  File "C:\Users\Test\src\unicodes\ubuntuone\syncdaemon\sync.py", line 60, in ge
t_mdid
    mdid = self.fs._idx_path[self.keys["path"]]
KeyError: '\\\\?\\C:\\Users\\Test\\Ubuntu One\\log.7z'

tags: added: u1-zomg-windows
Changed in ubuntuone-client:
importance: Undecided → High
status: New → Triaged
assignee: nobody → Manuel de la Pena (mandel)
Revision history for this message
Manuel de la Peña (mandel) wrote :

Seems that the long path prefix is being added in the file notifications, easy fix, will add it in the diff merge proposals...

Revision history for this message
Manuel de la Peña (mandel) wrote :

This bug is fixed since the adition of the decorators in the os_helper code.

Changed in ubuntuone-client:
status: Triaged → Fix Committed
dobey (dobey)
Changed in ubuntuone-client:
milestone: none → 1.7.2
tags: added: u1-windows
dobey (dobey)
Changed in ubuntuone-client:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.