Opened 13 years ago

Closed 13 years ago

Last modified 11 years ago

#2668 closed defect (fixed)

passing path to checkout to update() causes data file to be destroyed

Reported by: sascha_silbe Owned by: sascha_silbe
Priority: Unspecified by Maintainer Milestone:
Component: Sugar Version: Git as of bugdate
Severity: Critical Keywords: 11.2.0
Cc: Distribution/OS: OLPC
Bug Status: Assigned


If we call datastore.get() and pass the file name back into datastore.update() for the same entry with transfer_ownership set to False, the data file will get destroyed.
This is because source and destination file are identical (hard links) and we overwrite the destination file in AsyncCopy.start() without unlinking it first.

Change History (6)

comment:2 Changed 13 years ago by erikos


  • open Record and take a photo
  • switch to the Journal and wait for the picture entry to appear
  • go back to Record
  • rename the photo
  • close Record
  • open the photo in Imageviewer / open the Record session and show the picture

comment:4 Changed 13 years ago by erikos

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

Did land, closing. QA happens on

comment:5 Changed 11 years ago by dnarvaez

  • Component changed from sugar-datastore to Sugar

comment:6 Changed 11 years ago by dnarvaez

  • Milestone 0.92 deleted

Milestone 0.92 deleted

Note: See TracTickets for help on using tickets.